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

enable cxxmodules fwcore packages #5985

Merged
merged 1 commit into from Jun 24, 2020

Conversation

smuzaffar
Copy link
Contributor

No description provided.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for branch IB/CMSSW_11_2_X/rootmodule.

@cmsbuild, @smuzaffar, @mrodozov, @tulamor can you please review it and eventually sign? Thanks.
cms-bot commands are listed here

@smuzaffar
Copy link
Contributor Author

test parameters

  • full_cmssw = true

@smuzaffar
Copy link
Contributor Author

please test

@smuzaffar
Copy link
Contributor Author

@davidlange6 , @vgvassilev this shoudl allow to build CXXMOdules for FWCore/Reflection FWCore/Utilities. I did not add FWCore/Framework FWCore/ParameterSet for now as there are build errors for those

@cms-sw cms-sw deleted a comment from cmsbuild Jun 23, 2020
@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 23, 2020

The tests are being triggered in jenkins.
Test Parameters:

@cmsbuild
Copy link
Contributor

-1

Tested at: 023b9f6

CMSSW: CMSSW_11_2_CXXMODULE_X_2020-06-22-2300
SCRAM_ARCH: slc7_amd64_gcc820
You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e56f42/7299/summary.html

I found follow errors while testing this PR

Failed tests: Build

  • Build:

I found compilation error when building:

Error: Module 'AlignmentCocoaDaq_xr_rdict.pcm' failed to load.
Warning: 'CondFormatsAlignment_xr_rdict.pcm' value is deprecated. Please use []CondFormatsAlignment_xr.pcm
Error: Module 'CondFormatsAlignment_xr_rdict.pcm' failed to load.
gmake: *** [tmp/slc7_amd64_gcc820/src/Alignment/CocoaFit/src/AlignmentCocoaFit/a/AlignmentCocoaFit_xr.cc] Error 1
>> Compiling LCG dictionary: tmp/slc7_amd64_gcc820/src/Alignment/CocoaFit/src/AlignmentCocoaFit/a/AlignmentCocoaFit_xr.cc
c++: error: AlignmentCocoaFit/a/AlignmentCocoaFit_xr.cc: No such file or directory
c++: fatal error: no input files
compilation terminated.
gmake: *** [tmp/slc7_amd64_gcc820/src/Alignment/CocoaFit/src/AlignmentCocoaFit/a/AlignmentCocoaFit_xr.cc.o] Error 1
>> Building  shared library tmp/slc7_amd64_gcc820/src/Alignment/CocoaFit/src/AlignmentCocoaFit/libAlignmentCocoaFit.so
c++: error: tmp/slc7_amd64_gcc820/src/Alignment/CocoaFit/src/AlignmentCocoaFit/a/AlignmentCocoaFit_xr.cc.o: No such file or directory


@cmsbuild
Copy link
Contributor

Comparison not run due to Build errors (RelVals and Igprof tests were also skipped)

@vgvassilev
Copy link
Contributor

@smuzaffar, sounds good!

@davidlange6 davidlange6 merged commit 8c00a5b into IB/CMSSW_11_2_X/rootmodule Jun 24, 2020
@davidlange6
Copy link
Contributor

hi @smuzaffar - is there an easy way to reproduce the issues you saw with the other FWCore packages?

@smuzaffar
Copy link
Contributor Author

Try creating dev area and

  • checkout the FW packages
  • touch FW/Package/src/classes.h and FW/Package/src/classes_def.xml
  • enable CXXMODULES for these FW packages in config/SCRAM/GMAKE/CXXModules.mk file
  • scram build

@davidlange6
Copy link
Contributor

davidlange6 commented Jun 24, 2020 via email

@vgvassilev
Copy link
Contributor

They work for me:
ls lib/slc7_amd64_gcc820/*pcm
lib/slc7_amd64_gcc820/FWCoreFramework.pcm lib/slc7_amd64_gcc820/FWCoreReflection.pcm
lib/slc7_amd64_gcc820/FWCoreParameterSet.pcm lib/slc7_amd64_gcc820/FWCoreUtilities.pcm

@smuzaffar
Copy link
Contributor Author

looks good @vgvassilev . For me the full build failed due due to errors in FWCoreFramework and FWCoreParameterSet.
I will open a PR to enable these and run a full build test

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

4 participants