alidist icon indicating copy to clipboard operation
alidist copied to clipboard

[DO NOT MERGE] Bump FairRoot v18.8.0-beta

Open dennisklein opened this issue 1 year ago • 2 comments

NOT MEANT FOR MERGING, just for testing with your CI.

@ktf This should get rid of Boost, FairMQ, DDS, PROOF dependencies. However, there were many C++ changes/deprecations as well, let's see how Alice is affected. (18.8 is not released, but we would like to soon)

cc: @karabowi @ChristianTackeGSI

dennisklein avatar Nov 25 '22 12:11 dennisklein

Just collecting what shows up so far:

Common/Field/src/MagFieldFact.cxx:30:73: warning: 'FairFieldFactory::fCreator' is deprecated [-Wdeprecated-declarations]
   30 | MagFieldFact::MagFieldFact() : FairFieldFactory(), mFieldPar(nullptr) { fCreator = this; }
      |                                                                         ^~~~~~~~
In file included from /sw/SOURCES/O2/build_O2_alidist-slc7_x86-64/0/Common/Field/include/Field/MagFieldFact.h:19,
                 from /sw/SOURCES/O2/build_O2_alidist-slc7_x86-64/0/Common/Field/src/MagFieldFact.cxx:12:
/sw/slc7_x86-64/FairRoot/v18.8.0-beta-local1/include/FairFieldFactory.h:33:38: note: declared here
   33 |     [[deprecated]] FairFieldFactory* fCreator;
      |                                      ^~~~~~~~
/sw/SOURCES/O2/build_O2_alidist-slc7_x86-64/0/Detectors/ITSMFT/test/HitAnalysis/src/HitAnalysis.cxx: In member function 'virtual InitStatus o2::its::HitAnalysis::Init()':
/sw/SOURCES/O2/build_O2_alidist-slc7_x86-64/0/Detectors/ITSMFT/test/HitAnalysis/src/HitAnalysis.cxx:71:3: error: 'FairRootManager' was not declared in this scope
   71 |   FairRootManager* mgr = FairRootManager::Instance();
      |   ^~~~~~~~~~~~~~~
/sw/SOURCES/O2/build_O2_alidist-slc7_x86-64/0/Detectors/ITSMFT/test/HitAnalysis/src/HitAnalysis.cxx:71:20: error: 'mgr' was not declared in this scope
   71 |   FairRootManager* mgr = FairRootManager::Instance();
      |                    ^~~
/sw/SOURCES/O2/build_O2_alidist-slc7_x86-64/0/Detectors/ITSMFT/test/HitAnalysis/src/HitAnalysis.cxx:71:26: error: 'FairRootManager' is not a class, namespace, or enumeration
   71 |   FairRootManager* mgr = FairRootManager::Instance();
      |                          ^~~~~~~~~~~~~~~
/sw/SOURCES/O2/build_O2_alidist-slc7_x86-64/0/Detectors/ITSMFT/test/HitAnalysis/src/HitAnalysis.cxx:77:29: error: expected primary-expression before 'const'
   77 |   mHits = mgr->InitObjectAs<const std::vector<o2::itsmft::Hit>*>("ITSHit");
      |                             ^~~~~

dennisklein avatar Nov 25 '22 13:11 dennisklein

Just collecting what shows up so far:

Common/Field/src/MagFieldFact.cxx:30:73: warning: 'FairFieldFactory::fCreator' is deprecated [-Wdeprecated-declarations]
   30 | MagFieldFact::MagFieldFact() : FairFieldFactory(), mFieldPar(nullptr) { fCreator = this; }
      |                                                                         ^~~~~~~~

Just remove the fCreator = this;. That really should still work on 18.6 as well. (see: https://github.com/FairRootGroup/FairRoot/commit/7236364b552dc50004dc92d4c576bd975f8687f3 )

/sw/SOURCES/O2/build_O2_alidist-slc7_x86-64/0/Detectors/ITSMFT/test/HitAnalysis/src/HitAnalysis.cxx: In member function 'virtual InitStatus o2::its::HitAnalysis::Init()':
/sw/SOURCES/O2/build_O2_alidist-slc7_x86-64/0/Detectors/ITSMFT/test/HitAnalysis/src/HitAnalysis.cxx:71:3: error: 'FairRootManager' was not declared in this scope
   71 |   FairRootManager* mgr = FairRootManager::Instance();
      |   ^~~~~~~~~~~~~~~

I think, you should add a #include <FairRootManager.h> in an appropriate place. We have cleaned up our includes, so you need to be more specific about including what you really need.

JFI: If you fully switch to 18.8 (and don't need compatibility with 18.6), and you're using FairRun & friends, you might look into getting the FairRootManager from the FairRun instead of via the ::Instance method. See https://github.com/FairRootGroup/FairRoot/commit/b8bb90e69968975034597621580f002c9f766b78 for the change in FairRoot that exposes this new functionality.

ChristianTackeGSI avatar Nov 25 '22 19:11 ChristianTackeGSI