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

configure.ac forces sysconfdir to be /etc, breaking Nix/Guix installs #41

Open
xelxebar opened this issue Apr 12, 2021 · 0 comments
Open

Comments

@xelxebar
Copy link

Overview

Line 144 in configure.ac is a little strange:

test "$sysconfdir" = '${prefix}/etc' && sysconfdir=/etc

It strips the prefix from sysconfdir. I'm not sure what the intent is here, but this breaks installations anywhere other than the root directory. In particular, Guix and Nix will issue a build failure, since the build daemon doesn't have permission to create things under /etc.

Proposal

Perhaps the && was intended to be || or something? Better yet, maybe we want to set sysconfdir via an autoconf macro instead?

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

No branches or pull requests

1 participant