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

Geant4p03 tf fix 75 x #1419

Merged
2 commits merged into from Feb 25, 2015
Merged

Geant4p03 tf fix 75 x #1419

2 commits merged into from Feb 25, 2015

Conversation

ghost
Copy link

@ghost ghost commented Feb 25, 2015

+1
Tested compilation.
Tested compilation of SimG4Core/ against it.
Tested matrix tests against it.

@cmsbuild cmsbuild added this to the Next CMSSW_7_5_X milestone Feb 25, 2015
@ghost ghost mentioned this pull request Feb 25, 2015
@civanch
Copy link
Contributor

civanch commented Feb 25, 2015

@Degano , Great! do not forget the Geant4e patch on top of 10.0p03. For any case I put a link for any case: http://vnivanch.web.cern.ch/vnivanch/verification/verification/data/geant4-10.0p03-cms.patch

@ghost
Copy link
Author

ghost commented Feb 25, 2015

@civanch sorry, I don't understand: so https://github.com/cms-externals/geant4/tree/cms/4.10.00.p03 never had that patch?
But it does have other cms patches?

@ghost
Copy link
Author

ghost commented Feb 25, 2015

@civanch Here you can see what we already have on the externals on top of 4.10.00.p03:
cms-externals/geant4@4.10.00.p03...cms/4.10.00.p03 it seems that patch is already merged in.
Can you confirm please?

@civanch
Copy link
Contributor

civanch commented Feb 25, 2015

@Degano , Thanks, Geant4e is there. This is what we need.

ghost pushed a commit that referenced this pull request Feb 25, 2015
@ghost ghost merged commit c8fb568 into cms-sw:IB/CMSSW_7_5_X/stable Feb 25, 2015
@ghost ghost deleted the geant4p03_tf_fix_75X branch February 25, 2015 15:33
This pull request was closed.
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

3 participants