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

Feature Request: parse shell quotes in an arguments file #181

Open
GATKSupportTeam opened this issue Jan 3, 2022 · 4 comments
Open

Feature Request: parse shell quotes in an arguments file #181

GATKSupportTeam opened this issue Jan 3, 2022 · 4 comments
Assignees

Comments

@GATKSupportTeam
Copy link

A user is having an issue when their file paths have spaces in an arguments file. This issue does not exist when there is no arguments file. It would be helpful for GATK to be able to parse shell quotes correctly in the arguments file.

From the user:

I'm using gatk CombineGVCFs, and am supplying the paths of the GVCFs to combine using an arguments file, but I'm getting an error when there are spaces in the file path in the arguments file.
Here's an example of the content of the arguments file:
--variant /mnt/gpfs/live/rd01__/ritd-ag-project-rd018o-mdflo13/data/Illumina_amplicon_sequencing/2021.10.13 Emma MSH3 KO Illumina amplicon sequencing/2021.12.14 results/sample/results/variants/FAN1-1B5_g.vcf.gz
--variant /mnt/gpfs/live/rd01__/ritd-ag-project-rd018o-mdflo13/refs/1000G/cram/ROI/1000G_sample_ROI.g.vcf
I've tried putting the file path in inverted commas (e.g. `--variant "/mnt/gpfs/live/rd01__/ritd-ag-project-rd018o-mdflo13/data/Illumina_amplicon_sequencing/2021.10.13 Emma MSH3 KO Illumina amplicon sequencing/2021.12.14 results/sample/results/variants/FAN1-1B5_g.vcf.gz"`), and also tried escaping the spaces (e.g. `/mnt/gpfs/live/rd01__/ritd-ag-project-rd018o-mdflo13/data/Illumina_amplicon_sequencing/2021.10.13\ Emma\ MSH3\ KO\ Illumina\ amplicon\ sequencing/2021.12.14\ results/sample/results/variants/FAN1-1B5_g.vcf.gz`), but can't get it to work.
And here's the gatk call:
gatk CombineGVCFs \
--java-options '-DGATK_STACKTRACE_ON_USER_EXCEPTION=true' \
-R "${GKREF}/Homo_sapiens_assembly38.fasta" \
--arguments_file "${OUT}/results/variants/input.list" \
-O "${OUT}/results/variants/cohort.g.vcf"
And here's the error:

***********************************************************************

A USER ERROR has occurred: Illegal argument value: Positional arguments were provided ',Emma\{MSH3\{KO\{Illumina\{amplicon\{sequencing/2021.12.14\{results/sample/results/variants/FAN1-1B5_g.vcf.gz}' but no positional argument is defined for this tool.

***********************************************************************

(created from Zendesk ticket #244432)
gz#244432

@droazen
Copy link
Contributor

droazen commented Feb 7, 2022

@cmnbroad We discussed this one, and it seems like it should be a barclay issue rather than a GATK issue. Mind moving it over and closing this one when you get a chance?

@droazen droazen assigned ghost and cmnbroad Feb 7, 2022
@cmnbroad cmnbroad transferred this issue from broadinstitute/gatk Feb 7, 2022
@cmnbroad
Copy link
Collaborator

cmnbroad commented Feb 7, 2022

Moving to Barclay and closing. #181

@cmnbroad cmnbroad closed this as completed Feb 7, 2022
@jason-cerrato
Copy link

Hi @cmnbroad , it looks like this was migrated to barclay but then subsequently closed while in the barclay repository. Was this intentional?

@cmnbroad cmnbroad reopened this Feb 7, 2022
@cmnbroad
Copy link
Collaborator

cmnbroad commented Feb 7, 2022

Ha No @jason-cerrato. thanks for pointing that out.

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

No branches or pull requests

4 participants