-
-
Notifications
You must be signed in to change notification settings - Fork 510
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
Update curl pin #157
Update curl pin #157
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
Would prefer to go with a more precise version. Just so we can have a common lower bound across the stack. Otherwise agree bumping the version is needed. |
I remember we've had this conversation before, but can you explain this again? Can't remember the reason. |
Is your suggestion to bump the version to 7.62 or 7.62.0? |
That looks right. 7.62 is probably sufficient. |
@jakirkham can you please have a look. I included libcurl as well. Not sure if curl is still needed if we have libcurl. |
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.
Just need a version bump
Needed for conda-forge/git-feedstock#55
@jakirkham @isuruf can we land this? |
Make older gcc builds depend on libgcc-ng <=9.3.0
Note that 7.59 is not rebuilt for gcc7 label, so this was pulling in curl from defaults. I was thinking of bumping to 7.62, but since the pin was just
x
, we can keep it at 7.