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

Change pgzip container to pigz, pgzip conda package appears to be broken #2807

Merged
merged 5 commits into from
Oct 24, 2023

Conversation

lrauschning
Copy link
Contributor

No description provided.

@bgruening
Copy link
Member

@lrauschning why is the package broken? Is that reported upstream?

@lrauschning
Copy link
Contributor Author

Sorry, should have been a bit more descriptive in the PR and not just rephrased my commit message.
I noticed yesterday while updating a PR to NF-Core I had made that I copy-pasted the wrong package from anaconda.org in #2792 – concretely, the Python package pgzip instead of the commandline tool pigz. That broke tests on my local machine, the package itself is fine as far as I can tell.

@lrauschning
Copy link
Contributor Author

As there was previously no pgzip container in hash.tsv, I propose changing that to pigz (which is the one I need containerized for the NF-Core module).

@bgruening
Copy link
Member

Ok, thanks for the explanation. Can you please add your new container to the file without removing the old line. Thanks.

@lrauschning
Copy link
Contributor Author

Sure, one moment.

@bgruening bgruening enabled auto-merge October 24, 2023 20:08
@bgruening bgruening merged commit d89695e into BioContainers:master Oct 24, 2023
1 check passed
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