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

Fix using wrong JPEG header by disabling OSX Frameworks #37

Merged
merged 5 commits into from
Sep 18, 2019

Conversation

blowekamp
Copy link
Contributor

@blowekamp blowekamp commented Sep 16, 2019

Miss matching of the jpeg library is causing the following runtime
error on OSX:
Wrong JPEG library version: library is 90, caller expects 80

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.

@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.

@blowekamp
Copy link
Contributor Author

The lastest PR used the third party/vendored JPEG library. Now the test on OSX is also producing:

libpng warning: Application built with libpng-1.4.12 but running with 1.6.37
Error during rescale and writing of ITK-JPEG.jpg.test.png

@blowekamp blowekamp changed the title Do no use conda jpeg link to avoid runtime error Fix using wrong JPEG header by disabling OSX Frameworks Sep 17, 2019
On OSX header files for JPEG, PNG, EXPAT, TIFF were being found in the
Mono framework (/Library/Frameworks/Mono.framework/Headers). CMake
searching in frameworks is disabled.

Use CMAKE_FIND_ROOT_PATH to isolate searching to conda PREFIX.
@blowekamp
Copy link
Contributor Author

@mingwandroid I had to use some different CMake variables then what you prose. And the PROGRAM path needs to search the BUILD_PREFIX directory structure.

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