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
Update Configuration-Generator tag - 7_1 #4762
Update Configuration-Generator tag - 7_1 #4762
Conversation
Regarding this request #4761 (comment)
A new Pull Request was created by @mrodozov (Mircho Rodozov) for branch IB/CMSSW_7_1_X/stable. @cmsbuild, @smuzaffar, @gudrutis, @mrodozov can you please review it and eventually sign? Thanks. |
Is any test required for this request? |
+1 |
The tests are being triggered in jenkins. |
@mrodozov I would like to integrate the master version and this backport, as they are just addition of data files |
-1 Tested at: 0ee93b9
I found compilation error when building: + case "${DOWNLOAD_TOOL}" in + touch V01-00-02.tar.gz + curl -L -O -k https://github.com/cms-data/Configuration-Generator/archive/V01-00-02.tar.gz curl: (35) SSL connect error error: Bad exit status from /build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.ZyOytZ (%build) RPM build errors: Bad exit status from /build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.ZyOytZ (%build) You can see the results of the tests here: |
does the compilation error message give any hint to you? Is there any action should I take? |
sorry to bother you again on this, but do you think this error can be fixed? Please let me know if I should take any action. |
please test to verify whether it was just a temporary issue in the build procedure |
The tests are being triggered in jenkins. |
I am looking in to it. |
please test |
The tests are being triggered in jenkins. |
Pull request #4762 was updated. |
Comparison is ready Comparison Summary:
|
+1 |
+externals |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_7_1_X/stable IBs (tests are also fine). This pull request will be automatically merged. |
Regarding this request #4761 (comment)