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

Avoid feelnc writing to input directory #3980

Merged
merged 2 commits into from
Sep 23, 2021

Conversation

pavanvidem
Copy link
Member

@pavanvidem pavanvidem commented Sep 22, 2021

FOR CONTRIBUTOR:

  • - I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • - License permits unrestricted use (educational + commercial)
  • - This PR adds a new tool or tool collection
  • - This PR updates an existing tool or tool collection
  • - This PR does something else (explain below)

@bgruening
Copy link
Member

maybe add a comment, why this linking is needed :)

ping @natefoo

@pavanvidem
Copy link
Member Author

The tool needs a .index file for the genome FASTA file. The .index file is created in the input directory where the FASTA file is located. Symlink the genome FASTA to the working directory to avoid writing to inputdir.

@bgruening
Copy link
Member

The tool needs a .index file for the genome FASTA file. The .index file is created in the input directory where the FASTA file is located. Symlink the genome FASTA to the working directory to avoid writing to inputdir.

Sorry, I meant in the XML. Not that someone else removes it.

@bgruening bgruening merged commit 8913264 into galaxyproject:master Sep 23, 2021
@pavanvidem pavanvidem deleted the feelnc-input-fix branch July 9, 2024 14:47
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