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

Relative links #117

Open
ebourg opened this issue Aug 6, 2013 · 2 comments
Open

Relative links #117

ebourg opened this issue Aug 6, 2013 · 2 comments
Milestone

Comments

@ebourg
Copy link
Collaborator

ebourg commented Aug 6, 2013

The Debian Policy requires to create relative links unless the link points to another top level directory:

http://www.debian.org/doc/debian-policy/ch-files.html#s10.5

I suggest to normalize automatically the links specified by the user and make them relative if required. For example a link named /usr/share/java/foo.jar and pointing to /usr/share/java/foo-1.0.jar would point instead to ./foo-1.0.jar

@tmortagne
Copy link
Collaborator

I'm not sure about this, it's more a best practice than a technical requirement and there could be use case where you want your link to be absolute for some reason. It's not like it was impossible to indicate a relative link in the pom.xml. I would let users deal with that.

@ebourg
Copy link
Collaborator Author

ebourg commented Aug 7, 2013

Lintian prints a warning for non relative links:

http://lintian.debian.org/tags/symlink-should-be-relative.html

That's not strictly required to make a working package, but I think we should do our best to produce lintian clean packages.

An argument for relative links I believe is for chrooted file systems. If you access the files outside of the chroot you should still be able to follow the links. This is not possible with absolute links.

@tcurdt tcurdt added this to the 2.0 milestone Feb 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants