-
Notifications
You must be signed in to change notification settings - Fork 241
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
exec_prefix is set to / for prefix=/usr #1229
Comments
I've initially mentioned this in #1224. Also I do not know enough autotools-fu to fix this. |
Thanks!
I don't know autotools either, but I guess I can do something (having read many times the GNU coding standards about directory variables). I'll write the obvious fix, and will ask for review from distro people. |
alejandro-colomar
added a commit
to alejandro-colomar/shadow
that referenced
this issue
Feb 24, 2025
…directory variables Closes: <shadow-maint#1229> Reported-by: Chris Hofstaedtler <[email protected]> Cc: Sam James <[email protected]> Signed-off-by: Alejandro Colomar <[email protected]>
alejandro-colomar
added a commit
to alejandro-colomar/shadow
that referenced
this issue
Feb 26, 2025
…directory variables According to Sam, if we just remove the definitions, autotools will provide the right values. Note that this changes the default from /usr to /usr/local, which is the right value according to the GNU conding standards (and also what the BSDs prefer, according to one conversation I had with Ingo Schwarze from OpenBSD). Closes: <shadow-maint#1229> Link: <https://www.gnu.org/software/autoconf/manual/autoconf-2.72/html_node/Default-Prefix.html> Reported-by: Chris Hofstaedtler <[email protected]> Cc: Sam James <[email protected]> Signed-off-by: Alejandro Colomar <[email protected]>
alejandro-colomar
added a commit
to alejandro-colomar/shadow
that referenced
this issue
Mar 3, 2025
…directory variables According to Sam, if we just remove the definitions, autotools will provide the right values. Note that this changes the default from /usr to /usr/local, which is the right value according to the GNU conding standards (and also what the BSDs prefer, according to one conversation I had with Ingo Schwarze from OpenBSD). Closes: <shadow-maint#1229> Link: <https://www.gnu.org/software/autoconf/manual/autoconf-2.72/html_node/Default-Prefix.html> Reported-by: Chris Hofstaedtler <[email protected]> Cc: Sam James <[email protected]> Signed-off-by: Alejandro Colomar <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
configure.ac contains these "hacks":
At least the last of the three lines should IMO go away. src/Makefile.am has these lines:
I agree with the comment. IMO all of this (exec_)prefix trickery should go away.
The text was updated successfully, but these errors were encountered: