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

Use python3 in cmsRun in DEVEL build #26401

Merged
merged 2 commits into from
Apr 9, 2019

Conversation

davidlange6
Copy link
Contributor

PR description:

swap python2 and python3 linking in FWCore/Py*ParameterSet to allow test in DEVEL build [most likely to be reverted after one night]

PR validation:

one runTheMatrix workflow works correctly (runs).

@cmsbuild
Copy link
Contributor

cmsbuild commented Apr 9, 2019

A new Pull Request was created by @davidlange6 (David Lange) for CMSSW_10_6_DEVEL_X.

It involves the following packages:

FWCore/Framework
FWCore/PyDevParameterSet
FWCore/PythonParameterSet

@cmsbuild, @smuzaffar, @Dr15Jones can you please review it and eventually sign? Thanks.
@makortel, @Martin-Grunewald, @wddgit this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

@Dr15Jones
Copy link
Contributor

please test

@Dr15Jones
Copy link
Contributor

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Apr 9, 2019

The tests are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

cmsbuild commented Apr 9, 2019

This pull request is fully signed and it will be integrated in one of the next CMSSW_10_6_DEVEL_X IBs after it passes the integration tests.

@cmsbuild
Copy link
Contributor

cmsbuild commented Apr 9, 2019

-1
Tested at: UNKNOWN
I was not able to find a release to test this PR. See the Jenkins logs for more details.

@davidlange6
Copy link
Contributor Author

davidlange6 commented Apr 9, 2019 via email

@cmsbuild
Copy link
Contributor

cmsbuild commented Apr 9, 2019

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

@davidlange6
Copy link
Contributor Author

the tests went well enough that a DEVEL build is of interest.

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