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

[Do not merge] For testing new build rules for edm class version checks #36975

Closed
wants to merge 1 commit into from

Conversation

smuzaffar
Copy link
Contributor

No description provided.

@smuzaffar
Copy link
Contributor Author

please test
test will pass here though the class version check fails. Once this is done then I will start PR tests with cms-sw/cmssw-config#89 whcih should fix this issue

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36975/28343

  • This PR adds an extra 8KB to repository

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for master.

It involves the following packages:

  • DataFormats/Common (core)

@makortel, @smuzaffar, @Dr15Jones can you please review it and eventually sign? Thanks.
@makortel, @wddgit, @rovere this is something you requested to watch as well.
@perrotta, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

@cmsbuild
Copy link
Contributor

-1

Failed Tests: Build
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19884b/22444/summary.html
COMMIT: 9808462
CMSSW: CMSSW_12_3_X_2022-02-14-2300/slc7_amd64_gcc10
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/36975/22444/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-19884b/22444/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19884b/22444/git-merge-result

Build

I found compilation error when building:

>> Compiling  /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_3_X_2022-02-14-2300/src/DataFormats/Common/src/setIsMergeable.cc
>> Compiling LCG dictionary: tmp/slc7_amd64_gcc10/src/DataFormats/Common/src/DataFormatsCommon/a/DataFormatsCommon_xr.cc
>> Building  shared library tmp/slc7_amd64_gcc10/src/DataFormats/Common/src/DataFormatsCommon/libDataFormatsCommon.so
Copying tmp/slc7_amd64_gcc10/src/DataFormats/Common/src/DataFormatsCommon/libDataFormatsCommon.so to productstore area:
>> Checking EDM Class Version for src/DataFormats/Common/src/classes_def.xml in libDataFormatsCommon.so
error: class 'edm::RefCore' has a different checksum for ClassVersion 11. Increment ClassVersion to 12 and assign it to checksum 3425324092
Suggestion: You can run 'scram build updateclassversion' to generate src/DataFormats/Common/src/classes_def.xml.generated with updated ClassVersion
gmake: *** [tmp/slc7_amd64_gcc10/src/DataFormats/Common/src/DataFormatsCommon/libDataFormatsCommon.so] Error 1
Leaving library rule at DataFormats/Common
>> Compiling  /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_3_X_2022-02-14-2300/src/DataFormats/Common/test/DataFrame_t.cpp
>> Building binary DataFrame_t


@smuzaffar
Copy link
Contributor Author

please test

@cmsbuild
Copy link
Contributor

-1

Failed Tests: Build
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19884b/22451/summary.html
COMMIT: 9808462
CMSSW: CMSSW_12_3_X_2022-02-15-2300/slc7_amd64_gcc10
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/36975/22451/install.sh to create a dev area with all the needed externals and cmssw changes.

Build

I found compilation error when building:

>> Building LCG reflex dict from header file src/DataFormats/Common/src/classes.h
>> Compiling LCG dictionary: tmp/slc7_amd64_gcc10/src/DataFormats/Common/src/DataFormatsCommon/a/DataFormatsCommon_xr.cc
>> Building  shared library tmp/slc7_amd64_gcc10/src/DataFormats/Common/src/DataFormatsCommon/libDataFormatsCommon.so
Copying tmp/slc7_amd64_gcc10/src/DataFormats/Common/src/DataFormatsCommon/libDataFormatsCommon.so to productstore area:
>> Checking EDM Class Version for src/DataFormats/Common/src/classes_def.xml in libDataFormatsCommon.so
error: class 'edm::RefCore' has a different checksum for ClassVersion 11. Increment ClassVersion to 12 and assign it to checksum 3425324092
Suggestion: You can run 'scram build updateclassversion' to generate src/DataFormats/Common/src/classes_def.xml.generated with updated ClassVersion
gmake: *** [tmp/slc7_amd64_gcc10/src/DataFormats/Common/src/DataFormatsCommon/libDataFormatsCommon.so] Error 1
Leaving library rule at DataFormats/Common
>> Compiling  /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_3_X_2022-02-15-2300/src/DataFormats/Common/test/DataFrame_t.cpp
>> Building binary DataFrame_t


@smuzaffar smuzaffar closed this Feb 16, 2022
@smuzaffar smuzaffar deleted the smuzaffar-patch-5 branch February 16, 2022 10:53
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.

2 participants