-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fix/customdumpsoftwareversions #51
Conversation
@@ -23,7 +23,7 @@ process GZIP_FILES { | |||
gzip -c ${file_in} > ${output_name} | |||
cat <<-END_VERSIONS > versions.yml | |||
"${task.process}": | |||
gzip: \$(echo \$(gzip --version 2>&1) | sed 's/^.*coreutils) //; s/ .*\$//') | |||
gzip: \$(echo \$(gzip --help 2>&1 | head -n 1 ) | sed -e 's/ (.*//') |
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.
This version of gzip
in the container running this process does not have a --version
flag. The closest is --help
, which gives:
BusyBox v1.22.1 (2014-05-23 01:24:27 UTC) multi-call binary.
Usage: gzip [-cfd] [FILE]...
Compress FILEs (or stdin)
-d Decompress
-c Write to stdout
-f Force
This PR extracts the top line and uses that for the version (specifically uses BusyBox v1.22.1
). I'm including BusyBox
in the version since I believe these are custom versions of software (like gzip
) that are smaller/scaled down: https://www.busybox.net/about.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.
Looks good to me.
This fixes an error when generating the software versions file due to an invalid command for extracting the version of gzip.