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.

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

thisk4geo.sh redefines K4GEO environment variable #313

Open
BrieucF opened this issue Dec 14, 2023 · 3 comments
Open

thisk4geo.sh redefines K4GEO environment variable #313

BrieucF opened this issue Dec 14, 2023 · 3 comments
Labels
enhancement

Comments

@BrieucF
Copy link
Contributor

BrieucF commented Dec 14, 2023

Is it possible to have the script thisk4geo.sh redefining the $K4GEO environment variable?
@giovannimarchiori
Copy link
Contributor

Hi @BrieucF
I see that thisk4geo.sh does not (yet) set K4GEO ... and this is done by k4_local_repo inside k4geo instead. Is this request still relevant or somehow superseded by k4_local_repo?

@BrieucF
Copy link
Contributor Author

BrieucF commented Nov 5, 2024

Yes, with k4_local_repo this became less needed but I think it still makes sense to have thisk4geo.sh setting K4GEO (and LCGEO for consistency)

@jmcarcell
Copy link
Member

jmcarcell commented Dec 15, 2024 •
edited
Loading

In this case this is coming from DD4hep: https://github.com/key4hep/k4geo/blob/main/CMakeLists.txt#L152 so it's not completely trivial to do, while maintaining the generic script that DD4hep creates and installs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@BrieucF @jmcarcell @giovannimarchiori