search

nextflow-io / nextflow

A DSL for data-driven computational pipelines
http://nextflow.io
Apache License 2.0
2.61k stars 605 forks source link

Fix for issue #4757 #5068

Closed tzuni closed 1 week ago

tzuni commented 2 weeks ago

This is a fix for #4757

When nxf_s3_download attempts to check whether an s3 URI represents a file or directory it fails because the matching string expects a single space where the output of s5cmd has two spaces. This causes it to always treat the URI as a file and not work.

I just added the extra space and verified it fixes the issue.

netlify[bot] commented 2 weeks ago

Deploy Preview for nextflow-docs-staging canceled.

Name Link
Latest commit 1f0aa02c6677b67538677d959d5015dab5c0fe2f
Latest deploy log https://app.netlify.com/sites/nextflow-docs-staging/deploys/666c97cf3058ab0008fb3592
pditommaso commented 1 week ago

Thanks for reporting this. Closing in favour of #5069