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

Makefile is not very friendly to RPM spec file builds in Fedora and the like #70

Open
treveradams opened this issue Jul 17, 2013 · 4 comments

Comments

@treveradams
Copy link

I am not sure what is going on, but if CFLAGS is defined in a SPEC file make command, the CFLAGS in the makefile are completely ignored, causing a failure to build.

If possible, these should just be prefixed to the CFLAGS in the Makefile.

@diegonehab
Copy link
Contributor

What does passing the CFLAGS in the SPEC file do? Does it define the additional variable in the invocation of make? If so, this is the standard behavior: the variables defined in the makefile will be ignored. How do you do with Lua? Do you use MYCFLAGS etc?

@diegonehab
Copy link
Contributor

I added MYCFLAGS and MYLDFLAGS. Please let me know if these are enough.

@treveradams
Copy link
Author

I am just learning Lua. I imagine they will be enough. However, as I was just needing some functionality that was not yet in Fedora, I have asked those who actually do the official packaging to look here and follow up. If you would like to follow up with them, https://bugzilla.redhat.com/show_bug.cgi?id=985573 is the URL for the Fedora bug (this report is one of two patches in that report).

Thank you for the help!

@ewestbrook
Copy link
Contributor

Does anything further need to be done here?

We're reviewing open items in preparation for a release. If action is needed here, please add a comment. Otherwise, this issue will be closed on or after 24-Feb-2019.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants