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

Compiliation with -DGeant4VMC_USE_G4Root=no fails #68

Open
olifre opened this issue Nov 17, 2024 · 5 comments
Open

Compiliation with -DGeant4VMC_USE_G4Root=no fails #68

olifre opened this issue Nov 17, 2024 · 5 comments

Comments

@olifre
Copy link
Contributor

olifre commented Nov 17, 2024

Starting with commit:
ecd7c86
compilation fails as follows:

$ mkdir build
$ cd build
$ cmake -DGeant4VMC_USE_G4Root=no ..
$ make -j10
...
[  4%] Building CXX object source/CMakeFiles/geant4vmc.dir/geometry/src/TG4GeometryManager.cxx.o
/home/olifre/gits/geant4_vmc/source/geometry/src/TG4GeometryManager.cxx:32:10: fatal error: TG4RootDetectorConstruction.h: No such file or directory
   32 | #include "TG4RootDetectorConstruction.h"
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[2]: *** [source/CMakeFiles/geant4vmc.dir/build.make:455: source/CMakeFiles/geant4vmc.dir/geometry/src/TG4GeometryManager.cxx.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:515: source/CMakeFiles/geant4vmc.dir/all] Error 2
@olifre olifre changed the title Compiliation iwht -DGeant4VMC_USE_G4Root= Compiliation with -DGeant4VMC_USE_G4Root=no fails Nov 17, 2024
@ihrivnac
Copy link
Contributor

Thank you, @olifre , for reporting the problem. I was on travel in the past weeks; I will take a look this week.

@ihrivnac
Copy link
Contributor

Hi @olifre ,
The problem should be fixed with 528092d.
Can you try ?

Best regards,

@olifre
Copy link
Contributor Author

olifre commented Nov 28, 2024

Hi @ihrivnac ,

many thanks, I can confirm that this works like a charm. Thanks for the quick fix! 👍

Cheers and all the best, Olvier

@ihrivnac
Copy link
Contributor

Thank you, @olifre , for checking.

Do you need also a tag with this fix, or can it wait for the next tag based on Geant4 11.3 ?

@olifre
Copy link
Contributor Author

olifre commented Nov 29, 2024

Dear @ihrivnac ,
no, I do not need a tag, but many thanks for the offer! 😄
This issue showed up when packaging 6.6_p1 for Gentoo Linux, which basically tests all combinations of flags, and I will just include the patch for this case.
Thanks again!

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

No branches or pull requests

2 participants