Skip to content

/config/rootmanager.dat madness - investigate

EDIT 2022-04-07: I suggest writing small patch to overwrite hardcoded "cbmsim" to "mpdsim" in FairRoot codebase.
It will have zero effect on quality of FairRoot code, but it will allow us to get rid of rootmanager.dat file and whole "config" directory. Howto:

  1. apply patch from the main FairRoot directory before building https://git.jinr.ru/nica/nicadist/-/blob/master/patches/FairRoot_18_6_8_rootmanager_patch.sh
  2. remove line 92 from the commit !338 (diffs)

We had an issue here #57 (closed)

FYI, AliceO2 have the same thing in their codebase https://github.com/AliceO2Group/AliceO2/tree/dev/config

The only reason we cannot get rid of /config directory in main, is one-liner file rootmanager.dat

EDIT: here it is from FairRoot guys
https://github.com/FairRootGroup/FairRoot/pull/806/files#diff-d92558d3b99f5798596afc443dda7b8fe054edf3695b8bc7dd7cde4d7ecbc0fe
FairRootManager.cxx line 1091
static char* FairRootManager::GetTreeName()
static char* FairRootManager::GetFolderName()

There is a FairRoot recent issue related to it https://github.com/FairRootGroup/FairRoot/issues/1120
this is the response I got
https://github.com/FairRootGroup/FairRoot/issues/1111

SUMMARY: FairRoot group are not planning to address this issue in their codebase.
They suggest to do your own workaround. (Replace FairRootManager::GetTreeName() FairRootManager::GetFolderName() with your own implementations at the places where they are called)

I am closing this as unresolved (for now).

Edited by Slavomir Hnatic
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information