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

minor issue: container used in modules/local/kofamscan/download.nf does not include gnu wget so recording the version fails #249

Closed
wresch opened this issue Jan 25, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@wresch
Copy link

wresch commented Jan 25, 2024

Description of the bug

the download module for kofamscan uses the following in its script to record the version of wget:

wget --version | grep 'GNU Wget' | sed 's/GNU Wget ([0-9.]+) .*/\1/'

see
https://github.com/nf-core/metatdenovo/blob/39b9a9fc171ec99e0fad4ea0f60866395a4cc869/modules/local/kofamscan/download.nf#L28C5-L31C17

However the container https://depot.galaxyproject.org/singularity/curl:7.80.0 is based on busybox, not gnu wget so

$ singularity shell depot.galaxyproject.org-singularity-curl-7.80.0.img
(singularity)$ wget --version | grep 'GNU Wget' | sed 's/GNU Wget \([0-9.]\+\) .*/\1/'
wget: unrecognized option '--version'
BusyBox v1.32.1 (2021-04-13 11:15:36 UTC) multi-call binary.

Usage: wget [-c|--continue] [--spider] [-q|--quiet] [-O|--output-document FILE]
        [-o|--output-file FILE] [--header 'header: value'] [-Y|--proxy on/off]
        [--no-check-certificate]
        [-P DIR] [-S|--server-response] [-U|--user-agent AGENT] [-T SEC] URL...

Retrieve files via HTTP or FTP

        --spider        Only check URL existence: $? is 0 if exists
        --no-check-certificate  Don't validate the server's certificate
        -c              Continue retrieval of aborted transfer
        -q              Quiet
        -P DIR          Save to DIR (default .)
        -S              Show server response
        -T SEC          Network read timeout is SEC seconds
        -O FILE         Save to FILE ('-' for stdout)
        -o FILE         Log messages to FILE
        -U STR          Use STR for User-Agent header
        -Y on/off       Use proxy

Command used and terminal output

No response

Relevant files

No response

System information

No response

@wresch wresch added the bug Something isn't working label Jan 25, 2024
@erikrikarddaniel
Copy link
Member

Thanks for reporting. I think we should base this on a different container.

@erikrikarddaniel
Copy link
Member

Closing this as it should be fixed now.

@wresch
Copy link
Author

wresch commented Jan 26, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants