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

[10_6_X-gcc820] Added cepgen 1.2.1patch1 to standard CMSSW tools #9045

Conversation

forthommel
Copy link
Contributor

This attempts to build CepGen 1.2.1patch1 using gcc820, as #9042 shows an important refactoring is needed to be able to build with gcc700.

Main features:

  • cloned from CLHEP spec-file
  • with interfacing libraries for HepMC2/3 (event output), LHAPDF (partonic photon PDF), and Pythia 6 ("legacy" proton remnant dissociation)
  • requires GSL, OpenBLAS, bzlib2
  • collateral damages: CMake bumped to version 3.25.2, Ninja bumped to version 1.10.0

Backport of #8023, #8089, #8319, #8329, #9030, #9033
FYI: @bbilin

- cloned from CLHEP spec-file
- with interfacing libraries for HepMC2/3 (event output), LHAPDF (partonic photon PDF), and Pythia 6 ("legacy" proton remnant dissociation)
- requires GSL, OpenBLAS, bzlib2
- collateral damages: CMake bumped to version 3.25.2, Ninja bumped to
  version 1.10.0
@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 1, 2024

A new Pull Request was created by @forthommel for branch IB/CMSSW_10_6_X/gcc820.

@iarspider, @cmsbuild, @smuzaffar, @aandvalenzuela can you please review it and eventually sign? Thanks.
@antoniovilela, @sextonkennedy, @rappoccio you are the release manager for this.
cms-bot commands are listed here

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 1, 2024

cms-bot internal usage

@forthommel forthommel changed the title Added cepgen to standard CMSSW tools [10_6_X-gcc820] Added cepgen 1.2.1patch1 to standard CMSSW tools Mar 1, 2024
@iarspider
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 1, 2024

-1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b2fd61/37852/summary.html
COMMIT: 6b65f08
CMSSW: CMSSW_10_6_X_2024-02-25-0000/slc7_amd64_gcc700
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmsdist/9045/37852/install.sh to create a dev area with all the needed externals and cmssw changes.

External Build

I found compilation error when building:

Generated cache: external+davix+0.6.7-c45ef47d4dc654aafd6f03c571059a81
FATAL: malformed spec found while quering it. Command: 
source /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/slc7_amd64_gcc700/rpm-env.sh ;  rpm -q --specfile /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/tmpspec-xrootd --info --define "cmsdist_directory /data/cmsbld/jenkins/workspace/ib-run-pr-tests/cmsdist" --define "compilerv 700" --define "cmscompilerv 700" --define "cmsos slc7_amd64" --define "package_vectorization %{nil}" --define "cmsswdata_version_link 1" --define 'buildroot /foo'
Resulted in:

error: line 334: Unknown tag: <<<<<<< HEAD
error: query of specfile /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/tmpspec-xrootd failed, can't parse
Traceback (most recent call last):
  File "./pkgtools/cmsBuild", line 4603, in 
    build(opts, args[1:], PKGFactory)
  File "./pkgtools/cmsBuild", line 3877, in build


@forthommel
Copy link
Contributor Author

Interesting! there does not seem to be any merge conflicts leftovers in here...

@iarspider
Copy link
Contributor

iarspider commented Mar 1, 2024

please test for CMSSW_10_6_X/slc7_amd64_gcc820

@iarspider
Copy link
Contributor

please test

@iarspider
Copy link
Contributor

please abort

@iarspider
Copy link
Contributor

please test for CMSSW_10_6_X/slc7_amd64_gcc820

@iarspider
Copy link
Contributor

There weren't any recent builds of 10_6 with gcc820, only with gcc700, so bot can't find an IB to test against.

@forthommel
Copy link
Contributor Author

Oh I see! Thanks for the explanation, @iarspider. It therefore looks like these adaptations to gcc700 are indeed needed on the CepGen side to properly integrate with 10_6_X, then...

@forthommel
Copy link
Contributor Author

Closing it for now, and switching (back) to gcc700 branch (see #9042).

@forthommel forthommel closed this Mar 4, 2024
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.

3 participants