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

Renamed CPP to CXX in Makefiles #1052

Merged
merged 1 commit into from Jun 16, 2017

Conversation

caringi
Copy link

@caringi caringi commented Jun 16, 2017

Renamed all variables in Makefiles that have 'CPP' in the name to 'CXX':

"CFLAGS enables the addition of switches for the C compiler, while
CXXFLAGS is meant to be used when invoking a C++ compiler. Similarly, a
variable CPPFLAGS exists with switches to be passed to the C or C++
preprocessor." (https://en.wikipedia.org/wiki/CFLAGS)

This change facilitates the building with customized CXXFLAGS (rpmbuild
exports a customized CXXFLAGS with hardening flags).

More 2 links:
https://stackoverflow.com/questions/495598/difference-between-cppflags-and-cxxflags-in-gnu-make
https://stackoverflow.com/questions/5541946/cflags-ccflags-cxxflags-what-exactly-do-these-variables-control

Renamed all variables in Makefiles that have 'CPP' in the name to 'CXX':

"CFLAGS enables the addition of switches for the C compiler, while
CXXFLAGS is meant to be used when invoking a C++ compiler. Similarly, a
variable CPPFLAGS exists with switches to be passed to the C or C++
preprocessor." (https://en.wikipedia.org/wiki/CFLAGS)

This change facilitates the building with customized CXXFLAGS (rpmbuild
exports a customized CXXFLAGS with hardening flags).
@renecannao renecannao merged commit 7acb7dc into sysown:master Jun 16, 2017
@renecannao
Copy link
Contributor

Thank you Augusto. I created #1053 to remind myself to apply this to future 1.3.8 and 1.41 . current master branch is not updated.
Actually, I am going to change the default branch to 1.4.1

renecannao added a commit that referenced this pull request Jun 27, 2017
renecannao added a commit that referenced this pull request Jun 27, 2017
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

Successfully merging this pull request may close these issues.

None yet

2 participants