Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fallback symlink command doesn't work #990

Merged
merged 2 commits into from
Mar 20, 2022
Merged

Fallback symlink command doesn't work #990

merged 2 commits into from
Mar 20, 2022

Conversation

dflock
Copy link
Contributor

@dflock dflock commented Mar 20, 2022

If run as-is, the current symlink command will create a broken link:

$ sudo ln -s  ./bin/libasciidoc /usr/bin/libasciidoc
$ file /usr/bin/libasciidoc
/usr/bin/libasciidoc: broken symbolic link to ./bin/libasciidoc

This command works and produces a working absolute link:

$ sudo ln -s "$PWD/bin/libasciidoc" /usr/bin/libasciidoc

If run as-is, the current symlink command will create a broken link:

```
$ sudo ln -s  ./bin/libasciidoc /usr/bin/libasciidoc
$ file /usr/bin/libasciidoc
/usr/bin/libasciidoc: broken symbolic link to ./bin/libasciidoc
```

This command works and produces a working absolute link:

```console
$ sudo ln -s "$PWD/bin/libasciidoc" /usr/bin/libasciidoc
```
@xcoulon
Copy link
Member

xcoulon commented Mar 20, 2022

oh thanks @dflock, i did not notice that error in the doc 🤦‍♂️
however, I'm getting this error on my mac:

$ ln -s  ./bin/libasciidoc /usr/bin/libasciidoc
ln: /usr/bin/libasciidoc: Operation not permitted

but it works with sudo ln -s $PWD/bin/libasciidoc /usr/local/bin/libasciidoc instead.
Depending on your OS and if the command above also works for you, could you update the PR accordingly?

@xcoulon xcoulon merged commit 9f96927 into bytesparadise:master Mar 20, 2022
@xcoulon
Copy link
Member

xcoulon commented Mar 20, 2022

thanks for this PR @dflock! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants