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.

Sign up for GitHub

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

Jump to bottom

Link with ROOT::Hist since TH1F is being used #45

Merged

jmcarcell merged 1 commit into main from add-root-hist

Jan 17, 2025

Contributor

jmcarcell commented Jan 17, 2025 •

edited

Loading

This is needed for builds on MacOS, otherwise it will fail with the error

Undefined symbols for architecture arm64:
 "TH1F::TH1F(char const, char const, int, double, double)", referenced from:
      HepMCHistograms::initialize() in HepMCHistograms.cpp.o
      HepMCHistograms::initialize() in HepMCHistograms.cpp.o
      HepMCHistograms::initialize() in HepMCHistograms.cpp.o
      HepMCHistograms::initialize() in HepMCHistograms.cpp.o


          Link with ROOT::Hist since TH1F is being used

9392cc5

jmcarcell force-pushed the add-root-hist branch from 8cb4ba3 to 9392cc5 Compare

January 17, 2025 21:12

jmcarcell enabled auto-merge (squash)

January 17, 2025 21:12

Contributor Author

jmcarcell commented Jan 17, 2025

Merging and tagging to have a version that can be built on MacOS

jmcarcell merged commit 99dad51 into main

4 checks passed

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

Labels

None yet