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

Three dangling symlinks in tree #105

Closed
dagwieers opened this issue Jun 8, 2012 · 3 comments
Closed

Three dangling symlinks in tree #105

dagwieers opened this issue Jun 8, 2012 · 3 comments
Assignees
Labels
Milestone

Comments

@dagwieers
Copy link
Contributor

The current master branch has 2 dangling symlinks:

[dag@moria rear]$ make validate
find . -name '*.sh' -exec bash -n {} \;
bash: ./usr/share/rear/format/OBDR/default/04_set_NETFS_requirements.sh: No such file or directory
bash: ./usr/share/rear/format/OBDR/default/04_set_tape_device.sh: No such file or directory

These symlinks link towards the prep/OBDR stage where similar files no longer exist:

[dag@moria rear]$ ls -l ./usr/share/rear/format/OBDR/default/04_set_NETFS_requirements.sh ./usr/share/rear/format/OBDR/default/04_set_tape_device.sh
lrwxrwxrwx. 1 dag dag 55 Jun  6 20:13 ./usr/share/rear/format/OBDR/default/04_set_NETFS_requirements.sh -> ../../../prep/OBDR/default/04_set_NETFS_requirements.sh
lrwxrwxrwx. 1 dag dag 48 Jun  6 20:13 ./usr/share/rear/format/OBDR/default/04_set_tape_device.sh -> ../../../prep/OBDR/default/04_set_tape_device.sh
@ghost ghost assigned jhoekx Jun 8, 2012
@dagwieers
Copy link
Contributor Author

I added a validation check for dangling symlinks to the Makefile, which now blocks every usage:

Locating dangling symlinks
find -L . -type l
find: File system loop detected; `./usr/share/rear/skel/default/lib/tls' is part of the same file system loop as `./usr/share/rear/skel/default/lib'.
./usr/share/rear/format/OBDR/default/04_set_NETFS_requirements.sh
./usr/share/rear/format/OBDR/default/04_set_tape_device.sh

So this needs to be fixed ASAP.

@dagwieers
Copy link
Contributor Author

Can anyone tell me what the /lib/tls/ symlink is for ? It points to itself so I doubt it can work like this.

jhoekx added a commit that referenced this issue Jun 11, 2012
The symlinks are there to set TAPE_DEVICE.

This is a partial fix for #105.
@dagwieers
Copy link
Contributor Author

@schlomo @gdha Does any of you remember what the /lib/tls symlink was for ? I'd like to remove it as it does not seem to serve any purpose and blocks both make validate as well as the make dist process.

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

No branches or pull requests

2 participants