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

Unvendor zlib #22

Merged
merged 2 commits into from
Apr 27, 2020
Merged

Unvendor zlib #22

merged 2 commits into from
Apr 27, 2020

Conversation

brianv0
Copy link
Contributor

@brianv0 brianv0 commented Apr 27, 2020

Checklist

  • Used a fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • [] Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

zlib is currently vendored in cfitsio. This PR unvendors zlib. I have been having some issues related to the vendoring.

openSUSE (https://build.opensuse.org/package/view_file/science/cfitsio/cfitsio-zlib.patch?expand=1) and ubuntu (https://launchpadlibrarian.net/439818280/cfitsio_3.450-3_3.470-3.diff.gz) have similar patches which are appropriate for their environments.

@brianv0 brianv0 requested review from pkgw and teake as code owners April 27, 2020 05:08
@conda-forge-linter
Copy link

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 (recipe) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipe:

  • License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

@brianv0
Copy link
Contributor Author

brianv0 commented Apr 27, 2020

@conda-forge-admin, please rerender

@beckermr beckermr closed this Apr 27, 2020
@beckermr beckermr reopened this Apr 27, 2020
@beckermr
Copy link
Member

@conda-forge-admin rerender

@beckermr beckermr merged commit a9401dd into conda-forge:master Apr 27, 2020
@teake
Copy link
Contributor

teake commented Apr 27, 2020

Thanks for the getting the pipelines to pass @beckermr, and thanks for the contribution @brianv0! Any chance of landing this patch upstream, @brianv0?

@beckermr
Copy link
Member

@brianv0 and I have been discussing the state of the upstream dev. Hopefully we can get something to happen, but the ball is in my court right now.

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.

4 participants