-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Utilities/General: Replace invalid XML characters in test/BuildFile.xml #26730
Utilities/General: Replace invalid XML characters in test/BuildFile.xml #26730
Conversation
In test/BuildFile.xml replace the invalid characters && with proper &&
The code-checks are being triggered in jenkins. |
please test |
Utilities/General/test/BuildFile.xml
Outdated
@@ -12,5 +12,5 @@ | |||
<iftool name="cuda-gcc-support"> | |||
<test name="CudaGCCSupport" command="echo Supported GCC version"/> | |||
<else/> | |||
<test name="CudaGCCSupport" command="echo ERROR: Unsupported GCC version && exit 1"/> | |||
<test name="CudaGCCSupport" command="echo ERROR: Unsupported GCC version && exit 1"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gartung , please please &&
with ;
This is a shell command and echo ERROR: Unsupported GCC version && exit 1
will fail
Utilities/General/test/BuildFile.xml
Outdated
@@ -12,5 +12,5 @@ | |||
<iftool name="cuda-gcc-support"> | |||
<test name="CudaGCCSupport" command="echo Supported GCC version"/> | |||
<else/> | |||
<test name="CudaGCCSupport" command="echo ERROR: Unsupported GCC version && exit 1"/> | |||
<test name="CudaGCCSupport" command="echo ERROR: Unsupported GCC version && exit 1"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<test name="CudaGCCSupport" command="echo ERROR: Unsupported GCC version && exit 1"/> | |
<test name="CudaGCCSupport" command="echo ERROR: Unsupported GCC version ; exit 1"/> |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26730/9719
|
The tests are being triggered in jenkins. |
A new Pull Request was created by @gartung (Patrick Gartung) for master. It involves the following packages: Utilities/General @cmsbuild, @smuzaffar, @Dr15Jones can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
I ran 'scram b test' in that directory but nothing happended possibly because I was using a FWLITE project area. |
@Dr15Jones please abort current test and restart test |
abort test |
@cmsbot please abort |
Jenkins tests are aborted. |
@cmsbot please test |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
Comparison is ready Comparison Summary:
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26730/9731
|
@smuzaffar why this sudden code-checks? Another problem with timestamps somewhere? |
Yes looked like timestamp issue or something like that. 19 hours ago bot reset the |
+1 |
In test/BuildFile.xml replace the invalid characters && with proper &&