-
Notifications
You must be signed in to change notification settings - Fork 441
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
Fix containerized merqury #5945
Fix containerized merqury #5945
Conversation
Might be faster than bioconda/bioconda-recipes#47201
I'm +1 on not bumping the version since the previous version is broken with containers. Also just for reference the coreutils bump is necessary since 8.32 was from defaults, does not exist in conda-forge, and no longer exists in defaults. |
I take care of the bioconda one |
In this case we can drop the addition of awk and sed requirements but will still need the coreutils change to build a new mulled container? |
Yes. Will do ... do you need it today or can we wait for the bot until tm? |
I think we can wait for the bot, thanks! |
This makes it much harder to tell what version of a tool was run if users run into this on other servers. We should really always bump the version. |
@@ -1 +1 @@ | |||
assembly 0 1037953 | |||
assembly 0 1037953 +inf 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's normal that this changes, it just means that the bug is fixed. The problem is that the test data was wrong initially.
Might be faster than bioconda/bioconda-recipes#47201
FOR CONTRIBUTOR: