search

villano-lab / nrCascadeSim

calculating the NR spectrum resulting from neutron-capture cascades.
MIT License
0 stars 1 forks source link

trying to fix the clang c++11/c++17 compiler issue #28

Closed villaa closed 2 years ago

villaa commented 2 years ago

This pull request is related to issue #27 and is an attempt to resolve that bug.

villaa commented 2 years ago

Posted this on SO and am waiting for the ferocious hell storm of vitriol that comes from their "inclusive" community.

villaa commented 2 years ago

Based on the SO question and a good comment I got I have found that the root puts in this:

(base) Anthonys-MacBook-Pro:nrCascadeSim villaa$ root-config --cflags --glibs
-stdlib=libc++ -pthread -std=c++17 -m64 -I/usr/local/Cellar/root/6.24.04/include/root -L/usr/local/Cellar/root/6.24.04/lib/root -lGui -lCore -lImt -lRIO -lNet -lHist -lGraf -lGraf3d -lGpad -lROOTVecOps -lTree -lTreePlayer -lRint -lPostscript -lMatrix -lPhysics -lMathCore -lThread -lMultiProc -lROOTDataFrame -stdlib=libc++ -lpthread -lm -ldl

That seems to explicitly override the other specification.

villaa commented 2 years ago

Fixed it by a comment on the SO post above. But then I got a new error about how clang treats -lW,rpath=. I solved it with the answer from this.