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

Updated libtiff to 4.5.1 #7233

Merged
merged 2 commits into from
Jun 29, 2023
Merged

Updated libtiff to 4.5.1 #7233

merged 2 commits into from
Jun 29, 2023

Conversation

radarhere
Copy link
Member

@radarhere radarhere commented Jun 27, 2023

libtiff 4.5.1 has been released - http://www.simplesystems.org/libtiff/

In https://download.osgeo.org/libtiff/tiff-4.5.1.tar.gz, test/CMakeLists.txt refers to test_write_read_tags.c, but that file is missing. It can be seen at https://gitlab.com/libtiff/libtiff/-/tree/master/test, so I presume this was a packaging error.

Since it is only a test, I've patched test/CMakeLists.txt in winbuild/build_prepare.py to remove the references to it.

@hugovk
Copy link
Member

hugovk commented Jun 27, 2023

Please can you report the packaging error upstream?

@radarhere
Copy link
Member Author

I've created https://gitlab.com/libtiff/libtiff/-/issues/585

Copy link
Contributor

@nulano nulano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW, the following would disable including test\CMakeLists.txt altogether:

         "build": [
             *cmds_cmake(
                 "tiff",
                 "-DBUILD_SHARED_LIBS:BOOL=OFF",
                 '-DCMAKE_C_FLAGS="-nologo -DLZMA_API_STATIC"',
+                "-Dtiff-tests:BOOL=OFF",
             )
         ],

However, I slightly prefer your patch since it makes it easier to test the libtiff build locally, although I don't think I've ever actually done that.

winbuild/build_prepare.py Outdated Show resolved Hide resolved
Co-authored-by: Ondrej Baranovič <[email protected]>
@hugovk hugovk merged commit b2387f9 into python-pillow:main Jun 29, 2023
53 checks passed
@radarhere radarhere deleted the libtiff branch June 29, 2023 11:00
@radarhere
Copy link
Member Author

https://gitlab.com/libtiff/libtiff/-/issues/585 has been resolved by https://gitlab.com/libtiff/libtiff/-/merge_requests/519, and so should be part of the next libtiff release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants