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

nextflow environment fix #4

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

nextflow environment fix #4

wants to merge 3 commits into from

Conversation

meono
Copy link

@meono meono commented Jul 10, 2024

Adding nextflow to environment and fixing typo on readme - not tested.
updating the sra-tools container to avoid config bug - tested.

Previously, there were instructions to install nextflow. They are removed but nextflow was not added to environment file.
The docker container had caused issues for us due to sra-tools configuration. It was configured as far as i can tell in the image. I think the requirement was removed on V3.0.1. This image uses the latest version of sra-tools and has worked without issues so far. It's built from this:

```FROM ayyuan/fasterq-dump:1.0

# Download, extract sratoolkit, and clean up.
RUN wget -q http://ftp-trace.ncbi.nlm.nih.gov/sra/sdk/current/sratoolkit.current-ubuntu64.tar.gz -O /tmp/sratoolkit.tar.gz \
    && tar -xzf /tmp/sratoolkit.tar.gz -C /opt \
    && rm -r /tmp/sratoolkit.tar.gz /opt/sratoolkit.3.0.0-ubuntu64

# Set the PATH environment variable.
ENV PATH="/opt/sratoolkit.3.1.1-ubuntu64/bin/:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"

# Set the default command to run
CMD ["bash"]```
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.

1 participant