add SYSCONFDIR variable to allow FHS-compliant builds#210
Conversation
There was a problem hiding this comment.
Thanks. A few minor corrections.
The SYSCONFDIR should also be reflected to the preprocessor, like in the line DEFINES+=PREFIX=\\\"$$PREFIX\\\" in this pro file.
Consequently, this line should utilize the SYSCONFDIR.
With this changes I am happy to merge.
BTW: What (distro)package are you about to create?
|
I just made the corrections, thanks. I made a package for Void Linux, it builds with this patch and it works great. I will make the PR for the void package very soon but what I have so far is here: void-linux/void-packages@243c289. |
|
Thanks for putting this together. Happy to accept the follow-up PR for void Linux whenever it is ready. |
|
I noticed some of the CI tests were failing, because I didn't set the new |
|
Thanks for the heads up. I was too impatient to wait for the tests to finish, it escaped me too. Nvm and no worries :
This is from top of my head. If you are stuck it can also be I missed sth. Just shout in any case. |
|
Thanks a lot for the guidance, I just made the PR. |
This change separates the installation prefix (PREFIX) from the system configuration directory by introducing a new
SYSCONFDIRvariable.Currently, the
skyscraper.profile installs configuration files under$${PREFIX}/etc/skyscraper, which can result in non-FHS-compliant paths such as /usr/etc/skyscraper. This makes it hard to package skyscraper for most distros without a patch. By separating the/etcinstallation prefix from the binary prefix we can build an an FHS-compliant build without a patch.With this change
SYSCONFDIRdefaults to$${PREFIX}/etc, preserving original behavior for local builds.SYSCONFDIRto/etc(or any other desired path) without modifying the.profile.$${PREFIX}/binas before.