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

Move IB test to pdf set inside cmssw external (rather than on cvmfs) #4473

Merged
merged 1 commit into from Jul 1, 2014

Conversation

bendavid
Copy link
Contributor

@bendavid bendavid commented Jul 1, 2014

... no cvmfs access

@bendavid
Copy link
Contributor Author

bendavid commented Jul 1, 2014

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 1, 2014

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes or unless it breaks tests.

@ktf
Copy link
Contributor

ktf commented Jul 1, 2014

bump! :D

ktf added a commit that referenced this pull request Jul 1, 2014
Move IB test to pdf set inside cmssw external (rather than on cvmfs)
@ktf ktf merged commit eec415a into cms-sw:CMSSW_7_2_X Jul 1, 2014
@davidlange6
Copy link
Contributor

@bendavid - seems that this fix did not work in 72x. Can you have a look at the IB errors - e.g.

https://cmssdt.cern.ch/SDT/cgi-bin//showMatrixTestLogs.py/slc5_amd64_gcc481/www/sun/7.2-sun-14/CMSSW_7_2_X_2014-07-06-1400/pyRelValMatrixLogs/run/

On Jul 1, 2014, at 3:40 PM, Giulio Eulisse notifications@github.com wrote:

Merged #4473.


Reply to this email directly or view it on GitHub.

@bendavid
Copy link
Contributor Author

bendavid commented Jul 7, 2014

Hi David,
This is unrelated. These failures are due to the integration of lhapdf6 in 72x. Some older generators which do not support lhapdf6 are evidently unintentionally picking it up instead of LHAPDF 5. @mkirsano can you take a look?

ktf added a commit to ktf/cmssw that referenced this pull request Jul 9, 2014
@ktf ktf mentioned this pull request Jul 9, 2014
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