-
Notifications
You must be signed in to change notification settings - Fork 713
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
Fix the Makefile to work with the install instructions in the README #259
Conversation
Hello LeLobster,
just a comment regarding DESTDIR!
LeLobster writes:
[...]
Also removed all the references to `DESTDIR` because that was not used at all and did nothing.
[...]
This is often used in order to install software in a staging directory
and in some package systems all packages actually need to support
DESTDIR. I think it is better to keep it.
|
I did not know that. EDIT -> No idea why I named it |
On second thought, after I did some reading about the Makefile standards on How about just adding a check if those dirs are passed on the command line and assign them to |
Thoughts @inkarkat? It looks good to me at first glance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me.
…odotxt#259) * Fix the Makefile to work with the install instructions in the README * Put back DEST_DIR * Add simple ifdef checks to see if the user has set custom dirs via the commandline * Update README to match Makefile
Before submitting a pull request, please make sure the following is done:
master
.fixes #XX
reference to the issue that this pull request fixes.First time working with a Makefile but all I did was fix it so the instructions in the README actually work.
INSTALL_DIR, CONFIG_DIR & BASH_COMPLETION
were not used in the original Makefile, instead there werebindir, sysconfdir & datarootdir
.Also removed all the references to
DESTDIR
because that was not used at all and did nothing.The default install location is still the same as I've changed nothing besides naming.