-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
bump zlib requirement to be in sync with conda-forge #7385
Conversation
@bgruening thanks! If we need to be careful about using minutes on CCI, then what is our mechanism for doing bulk updates? Should we run these on travis? |
@daler its not yet clear. I think no one can estimate of this is a real problem for us. Lets try to use the new setup in a normal way and see how it goes. I will try to restore the travis setup next week so we can always go back for a few days. |
Should we include an update of the |
Sure, I'm not sure I will find time in the next days, but will do my best. |
Do we want to get bioconda/bioconda-utils#264 in before this? That would enable us to remove some unnecessary pinnings. I'll review this PR shortly. |
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.
Wow, that's a big one! Nice work
recipes/abricate/0.5/meta.yaml
Outdated
@@ -1,35 +1,35 @@ | |||
package: | |||
name: abricate |
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.
4-space indentation
@@ -5,7 +5,7 @@ package: | |||
version: {{ version }} | |||
|
|||
build: | |||
number: 0 | |||
number: 1 |
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.
htslib
is not pinned
recipes/alfred/0.1.2/meta.yaml
Outdated
@@ -11,7 +11,7 @@ source: | |||
- Makefile.patch | |||
|
|||
build: | |||
number: 1 | |||
number: 3 |
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.
2
recipes/alfred/meta.yaml
Outdated
@@ -11,7 +11,7 @@ source: | |||
- Makefile.patch | |||
|
|||
build: | |||
number: 1 | |||
number: 3 |
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.
2
@@ -16,7 +16,7 @@ source: | |||
sha256: 4b6ac3831061bc8f8c8adb011fe58c8741402a9c21c95af395aa98154c9ab775 # [linux] | |||
|
|||
build: | |||
number: 0 | |||
number: 1 |
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.
pinning on run-only dep can be removed (needs bioconda/bioconda-utils#264 to make linter happy)
@@ -13,7 +13,7 @@ build: | |||
|
|||
requirements: | |||
build: | |||
- perl-threaded | |||
- perl |
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.
needs build bump
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.
I decided against this, it would not change anything, but with the next version bump people will have it easier.
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.
fine by me
run: | ||
- libgcc # [linux] | ||
- zlib | ||
build: |
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.
4-space indentation?
@@ -7,24 +7,24 @@ source: | |||
url: https://github.com/Ensembl/ensembl-tools/archive/release/87.zip | |||
|
|||
build: | |||
number: 0 | |||
number: 2 |
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.
1
- xz | ||
- bzip2 | ||
- zlib | ||
- xz {{CONDA_XZ}}* |
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.
needs build bump
recipes/wub/meta.yaml
Outdated
@@ -6,7 +6,7 @@ package: | |||
version: {{ version }} | |||
|
|||
build: | |||
number: 1 | |||
number: 2 |
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.
htslib
in not pinned
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.
maybe this is needed, I will leave this to the maintainer
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.
We just don't need to bump the build number here.
@mbargull I need to leave now. I tried to fix the most urgent onces. |
I just tagged |
@mbargull in bulk linting should be disabled, so I think we can proceed here as soon as you are ready :) |
Argh, right, straight to |
To me, open questions are what version restrictions (if any) should be used for the run-only requirements in |
Thanks @mbargull for this great review - as usual! Lets see how things go over the night. |
~600 recipes (of which ~240, 40 %, are UCSC ones) -- that should take a while 😆. Nice work on your part as well! |
The Linux job still has some more
|
Most of the once I have seen are due to llvm errors, I assume GCC will help in a few/most cases. |
This PR bumps the zlib requirements to be in sync with conda-forge. I have also cleanup up a large chunk of recipes and included CONDA_ZLIB everywhere.
@daler I will cancel the CI as this is targeting bulk and I don't want to waste CI minutes on CCI.