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

Compatibility of DaVinci and Analysis in LHCb software development lesson and update of DaVinci version #297

Open
vlukashenko opened this issue Mar 14, 2023 · 1 comment · May be fixed by #308
Open

Compatibility of DaVinci and Analysis in LHCb software development lesson and update of DaVinci version #297

vlukashenko opened this issue Mar 14, 2023 · 1 comment · May be fixed by #308

Comments

@vlukashenko
Copy link
Contributor

here it looks like DaVinci/v45r8 would not be anymore compatible with whatever the changes happened in Analysis.

lb-dev DaVinci/v45r8
lb-use Analysis
git lb-checkout Analysis/run2-patches Phys/DecayTreeTuple

will result in

CMake Error at Phys/DecayTreeTuple/CMakeLists.txt:94 (gaudi_install):
  Unknown CMake command "gaudi_install".

It seems that the DaVinci/v46r8 works though - but then need to change the DaVinci version everywhere and check that commands are still working.

Thanks to Emily Kaiyin Jiang for spotting this.
@jpwgnr
Copy link

jpwgnr commented Sep 27, 2024

Hi,
I would prefer here to switch to

git lb-checkout Analysis/v21r8 Phys/DecayTreeTuple

as Analysis/v21r8 corresponds to DaVinci/v45r8.

I think this fix is easier than changing DaVinci in the whole tutorial.

This was referenced Sep 27, 2024
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@vlukashenko @jpwgnr