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.

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

Bump of CLHEP::Hep3Vector ClassVersion needed for (GEANT4 10.7.1) #33231

Closed
wants to merge 317 commits into from

Conversation

silviodonato
Copy link
Contributor

PR description:

Bump of CLHEP::Hep3Vector ClassVersion in DataFormats/CLHEP/src/classes_def.xml
needed for the update of GEANT4 ( cms-sw/cmsdist#6742 )

PR validation:

CMSSW_11_3_0_pre3_G4VECGEOM fully validated by PdmV

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2)

@cmsbuild
Copy link
Contributor

-1

Failed Tests: Build
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d3b18d/13639/summary.html
COMMIT: 543c2d9
CMSSW: CMSSW_11_3_X_2021-03-19-1100/slc7_amd64_gcc900
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/33231/13639/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d3b18d/13639/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d3b18d/13639/git-merge-result

Build

I found compilation error when building:

>> Building LCG reflex dict from header file src/DataFormats/CLHEP/src/classes.h
>> Compiling LCG dictionary: tmp/slc7_amd64_gcc900/src/DataFormats/CLHEP/src/DataFormatsCLHEP/a/DataFormatsCLHEP_xr.cc
>> Building  shared library tmp/slc7_amd64_gcc900/src/DataFormats/CLHEP/src/DataFormatsCLHEP/libDataFormatsCLHEP.so
Copying tmp/slc7_amd64_gcc900/src/DataFormats/CLHEP/src/DataFormatsCLHEP/libDataFormatsCLHEP.so to productstore area:
>> Checking EDM Class Version for src/DataFormats/CLHEP/src/classes_def.xml in libDataFormatsCLHEP.so
error: class 'CLHEP::Hep3Vector' has a different checksum for ClassVersion 11. Increment ClassVersion to 12 and assign it to checksum 3040806103
Suggestion: You can run 'scram build updateclassversion' to generate src/DataFormats/CLHEP/src/classes_def.xml.generated with updated ClassVersion
gmake: *** [tmp/slc7_amd64_gcc900/src/DataFormats/CLHEP/src/DataFormatsCLHEP/libDataFormatsCLHEP.so] Error 1
Leaving library rule at DataFormats/CLHEP
>> Leaving Package DataFormats/CLHEP
>> Package DataFormats/CLHEP built


@makortel
Copy link
Contributor

error: class 'CLHEP::Hep3Vector' has a different checksum for ClassVersion 11. Increment ClassVersion to 12 and assign it to checksum 3040806103

Something didn't work? The current version 10 has the same checksum 3040806103.

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33231/21679

  • This PR adds an extra 8KB to repository

@cmsbuild
Copy link
Contributor

Pull request #33231 was updated. @makortel, @smuzaffar, @cmsbuild, @Dr15Jones can you please check and sign again.

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33231/21682

  • This PR adds an extra 8KB to repository

@cmsbuild
Copy link
Contributor

Pull request #33231 was updated. @makortel, @smuzaffar, @cmsbuild, @Dr15Jones can you please check and sign again.

@silviodonato
Copy link
Contributor Author

moved to #33234

@silviodonato
Copy link
Contributor Author

I closed this PR to avoid the "noise" of the commits "Merge CMSSW_11_3_X into CMSSW_11_3_GEANT4_X. "

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants