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

Can you add a "DESTDIR" #5

Closed
SolarAquarion opened this issue May 25, 2018 · 13 comments
Closed

Can you add a "DESTDIR" #5

SolarAquarion opened this issue May 25, 2018 · 13 comments

Comments

@SolarAquarion
Copy link

No description provided.

@uhlin
Copy link
Owner

uhlin commented May 26, 2018

For Windows? If not Windows, it's possible to specify a PREFIX when issuing make, like this: 'PREFIX=/opt make install'. If a prefix isn't supplied it defaults to /usr/local. That however will install the program in /usr/local/bin and the manual page in /usr/local/man/man1. Is this what you are asking for?

@SolarAquarion
Copy link
Author

SolarAquarion commented May 26, 2018 via email

@uhlin
Copy link
Owner

uhlin commented May 26, 2018

Not sure how you mean. Can you provide a diff? Or if you don't can for some reason I can do it but I only need to understand better.

@SolarAquarion
Copy link
Author

@uhlin
Copy link
Owner

uhlin commented May 27, 2018

But doesn't specifying a PREFIX kind of do that? 'PREFIX=/path/to/dest make install'. I'll look into it further.

@SolarAquarion
Copy link
Author

SolarAquarion commented May 27, 2018 via email

@uhlin
Copy link
Owner

uhlin commented May 27, 2018

Ah, ok. Think I understand now. Will fix it.

@uhlin
Copy link
Owner

uhlin commented May 27, 2018

Hmm. On the other hand isn't it just to include the destdir as part of the prefix? I might be wrong. All directories in the PREFIX should be created automatically if non-existent...

@SolarAquarion
Copy link
Author

SolarAquarion commented May 27, 2018 via email

@uhlin
Copy link
Owner

uhlin commented May 27, 2018

Pushed a new commit regarding DESTDIR. Hopefully it does what you want...

@uhlin
Copy link
Owner

uhlin commented May 28, 2018

Can you confirm if it does what you want now?

@SolarAquarion
Copy link
Author

It works wonderfully and now i'm going to push it to the aur

@uhlin
Copy link
Owner

uhlin commented May 28, 2018

Very good to hear. Then I close this thread now.

@uhlin uhlin closed this as completed May 28, 2018
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

2 participants