Skip to content

use Nextflow.file() instead of new File() for samples/accessions to support cloud storage paths - #1

Merged
rpetit3 merged 1 commit into
bactopia:mainfrom
rwilliams:fix/samples-cloud-storage-path
Oct 9, 2026
Merged

rpetit3 merged 1 commit into
bactopia:mainfrom
rwilliams:fix/samples-cloud-storage-path

Conversation

@rwilliams

Copy link
Copy Markdown
Contributor

new File() doesn't understand s3:// (or gs://) URIs and mangles them (s3:// -> s3:/) when Java normalizes the path, causing "No such file or directory" errors on cloud-hosted samplesheets/accession files.

See bactopia/bactopia#679

… support cloud storage paths

new File() doesn't understand s3:// (or gs://) URIs and mangles them
(s3:// -> s3:/) when Java normalizes the path, causing "No such file
or directory" errors on cloud-hosted samplesheets/accession files.
@rwilliams rwilliams changed the title use Nextflow.file() instead of java.io.File for samples/accessions to support cloud storage paths use Nextflow.file() instead of new File() for samples/accessions to support cloud storage paths Sep 30, 2026
@rpetit3

rpetit3 commented Oct 9, 2026

Copy link
Copy Markdown
Member

yo! Meant to get this merged a while ago, haha but figured you probably edited things to work on your system!

I'll get this merged, thank you very much for taking the time to get this figured out!

@rpetit3
rpetit3 merged commit b8acb28 into bactopia:main Oct 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants