New convention for Geant4 tags with new cmake builds to better match internal Geant4 versioning
In the project cmakefile the version is set to major.minor.patch
in the Geant4 native convention (e.g. 10.6.2
) and then LHCbsetting adds to it the .lhcbpatch (.6)
called tweak
.
Marco said
I could try to tweak the build of Geant4 to accept
106.2.6
as well as10.6.2.6
, but he thinks it is better if we use native Geant4 version. The tag can stayv106r2p6 or become
v10r6p2t6` (t comes from CMake version fragments names major, minor, patch, tweak).
@kreps, @dpopov and @gcorti agreed with that we should use the native Geant4 version, i.e. 10.6.2.tN for releases from now on
For the tag there are pro&cons in keeping the existing convention or moving to a new one.
In any case we will have to force the exact version in Gauss...
As far as my understanding goes it does not matter which one we choose for older versions using the old cmake because they will pick v106r2p4
(v96r4p
2 for Sim09
using cmt); for versions with the new cmake they will look into the cmake configuration and pick the one with the correct version... so it is more a matter of human understanding.
-
v106r2p6
is consistent with what we have done so far, whilev10r6p2t6
makes it clearer what is Geant4 and what LHCb -
v106r2p6
is consistent with what we are using for generators data packages where the version of the generator is in synch, where we have adopted the same for major, minor, patch, tweak : if we change we may want to be consistent with the data packages as well and review their versioning with the new cmake - if we use
v10r6p2t6
we restart the version numbering and we will havev10rX
thenv11rX
... : I don't think we will ever re-use an already used version, the older one I see on gitlab/svn isv71r0p16
used in DC06 production. So no worry on that point of view. - if we use
v10r6p2t6
we are more consistent with what in the cmake files10.6.2.tN
At the end we all agreed that we should move to v10r6p2tN for the tags: adding a tweak suffix to the standard G4 format is more clear and only few people use