config.mk: allow overriding CXX - Githubissues

eic / epic-analysis

General (SI)DIS analysis framework for the EIC

GNU Lesser General Public License v3.0

3 stars 9 forks source link

config.mk: allow overriding CXX #243

Closed veprbl closed 1 year ago

veprbl commented 1 year ago

Briefly, what does this PR introduce?

Allows overriding CXX via make CXX=foo

What kind of change does this PR introduce?

[ ] Bug fix (issue #__)
[ ] New feature (issue #__)
[ ] Documentation update
[ ] Other: __

Please check if this PR fulfills the following:

[ ] Tests for the changes have been added
[ ] Documentation has been added / updated
[ ] Changes have been communicated to collaborators

Does this PR introduce breaking changes? What changes might users need to make to their code?

Does this PR change default behavior?

c-dilks commented 1 year ago

Thanks. I propagated this change to https://github.com/eic/epic-analysis/pull/238, which merges config.mk and all the Makefiles into one. An edm4eic issue is (possibly) causing CI job failures, but we can go ahead and merge this.