CalorimeterHitReco: fix for latest podio
In file included from JugReco/src/components/CalorimeterHitReco.cpp:29:
include/edm4eic/CalorimeterHitCollection.h:164:8:
note: candidate: 'void
edm4eic::CalorimeterHitCollection::push_back(edm4eic::MutableCalorimeterHit)'
164 | void push_back(MutableCalorimeterHit object);
| ^~~~~~~~~
include/edm4eic/CalorimeterHitCollection.h:166:8:
note: candidate: 'void
edm4eic::CalorimeterHitCollection::push_back(edm4eic::CalorimeterHit)'
166 | void push_back(CalorimeterHit object);
| ^~~~~~~~~
Merge request reports
Activity
Filter activity
added 62 commits
-
dc75c7cc...6bd2ce9f - 61 commits from branch
main
- 8f771ec2 - CalorimeterHitReco: fix for latest podio
-
dc75c7cc...6bd2ce9f - 61 commits from branch
requested review from @wdconinc
The issue here in the container build is that the API call to https://eicweb.phy.anl.gov/api/v4/projects/EIC%2Fjuggler/repository/commits/${JUGGLER_VERSION} requires a url-encoded juggler version, so the slash is preventing this from working. This is in a cache busting layer that aims to resolve that link to get the most recent commit on a branch (otherwise docker would just reuse the same cached layers for main).
We can likely get around this by passing a url-encoded juggler version.GitHub doesn't have this requirement on their API, so we could also replace the link with a GitHub link.Edited by Wouter Deconinckadded 2 commits
Please register or sign in to reply