-
Notifications
You must be signed in to change notification settings - Fork 54
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
ncm-download: Fixed false failure bug, added noaction support #262
Conversation
Automatic reply from Jenkins: Can I test this? |
# http://eu-datagrid.web.cern.ch/eu-datagrid/license.html. | ||
# | ||
# THE FOLLOWING DISCLAIMER APPLIES TO ALL SOFTWARE CODE AND OTHER MATERIALS | ||
# http://eu-datagrid.web.cern.ch/eu-datagrid/license.html. |
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.
replace block with # ${license-info}
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.
Fixed in all pan files
ok to test |
Fixed the problem where occasionally curl reported a failure (inspite of using the option to prevent this) which caused ncm-download to incorrectly report an error. Modernised to include --noaction support. Component already had unit tests in spite of using LC::Check::status. It turns out that LC::Check::status *does* support noaction so is test safe.
Looks good for 14.8 @stdweird merge when you are happy! |
@stdweird, I leave it to you to decide when this should be merged. |
ncm-download: Fixed false failure bug, added noaction support
Fixed the problem where occasionally curl reported a failure (inspite of
using the option to prevent this) which caused ncm-download to
incorrectly report an error.
Modernised to include --noaction support. Component already had unit
tests in spite of using LC::Check::status. It turns out that
LC::Check::status does support noaction so is test safe.