-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat(action): additional options to specify link guidelines #3
Comments
Dear Zach Gover,
|
Dear Zach Gover,
Thank you for your interest in the project. |
@ololx What is your position on option to use relative paths vs absolute? |
@zgover Could you be so kind as to clarify what you mean? Note that the root folder of the disk (where installed OS X) is not written and is indicated by writing a special character "slash" (/) at the beginning of the path. |
@ololx let me demonstrate using Target file: Absolute exampleTo make an absolute soft symlinkWorking dir: ln -s "$(pwd)/test.txt" "$(pwd)/example-dir/test-abs.txt" Relative exampleTo make a relative soft symlinkWorking dir: ln -s ./../test.txt ./test-rel.txt Final resultSo now the PurposeThe general purpose of this is to make symbolic links within network drives (e.g. SMB, NFS, etc.,). While currently it is fixed to absolute paths, therefore when creating symlinks on an SMB share to a target which is also on the same share the resulting symlink is below ⇩⬇︎ (the destination fails if you're not on machine which created it because the absolute path is using the path of the currently mounted network volume) Current result (network share)Working dir: lrwxr-xr-x 1 zgover staff 32 Aug 21 00:21 test.txt -> /Volumes/ReadyNAS/zgover/data/example-dir/test.txt |
@zgover This is really interesting case. I think the main problem here is the different name of the mounted disk on another machine? or you want to replace directories with ../ when is created the link in same super directory with target file or directory? I added this point into TODO for the considered case - https://github.com/ololx/quick-symlink/blob/main/TODO.md:
|
@ololx cool let's keep the issue open until resolved to track progress in PR's and commits. |
Hi, @zgover, Could you be so kind to watch the v0.8.0 release and tell me about following: |
Request of additional actions
Thoughts or reservations?
The text was updated successfully, but these errors were encountered: