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

[tiff] Update to 4.3.0, improvements #18187

Merged
merged 18 commits into from
Jun 9, 2021
Merged

Conversation

dg0yt
Copy link
Contributor

@dg0yt dg0yt commented May 29, 2021

  • What does your PR fix?

  • Important change in libtiff 4.3.0

    Use of "int8", "uint8", "int16", "uint16", "int32", "uint32", "int64", "uint64" typedefs is now deprecated. libtiff code and headers no longer use them, and use their C99 standard equivalents (with "_t" suffix). Those typedefs are still available, with deprecation warnings, but external code is strongly encouraged to use the corresponding C99 stdint.h types. The deprecated types might be removed in a future release.

    (From https://libtiff.gitlab.io/libtiff/v4.3.0.html#libtiff)
    This affects at least port selene which treated these warnings as errors.

  • Which triplets are supported/not supported? Have you updated the CI baseline?

    all, no

  • Does your PR follow the maintainer guide?

    yes

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

    I am still working on this PR.

@dg0yt dg0yt force-pushed the tiff branch 3 times, most recently from 82d33ca to e0f0ac7 Compare May 30, 2021 08:02
@dg0yt
Copy link
Contributor Author

dg0yt commented May 30, 2021

Manually tested, covering the cmake wrapper:
tiff[cxx,jpeg,lzma,tools,webp,zip,zstd]:x64-linux + libgeotiff:x64-linux

@dg0yt dg0yt marked this pull request as ready for review May 30, 2021 13:03
@dg0yt dg0yt marked this pull request as draft May 30, 2021 13:15
@dg0yt dg0yt marked this pull request as ready for review May 30, 2021 13:20
@JonLiu1993 JonLiu1993 self-assigned this May 31, 2021
@JonLiu1993 JonLiu1993 added the category:port-update The issue is with a library, which is requesting update new revision label May 31, 2021
ports/tiff/vcpkg.json Outdated Show resolved Hide resolved
versions/t-/tiff.json Outdated Show resolved Hide resolved
ports/selene/portfile.cmake Outdated Show resolved Hide resolved
@dg0yt dg0yt requested a review from JonLiu1993 May 31, 2021 19:19
ports/selene/tiff-deprecated-typedefs.patch Outdated Show resolved Hide resolved
@JackBoosY JackBoosY added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Jun 1, 2021
ports/tiff/portfile.cmake Outdated Show resolved Hide resolved
@JonLiu1993 JonLiu1993 added requires:all-feature-testing vcpkg install port[all features supported by that port] needs to be demonstrated to function and removed requires:author-response labels Jun 1, 2021
@JonLiu1993
Copy link
Member

JonLiu1993 commented Jun 2, 2021

All features are tested successfully in the following triplets:

  • x86-windows
  • x64-windows
  • x64-windows-static
  • x64-linux

@JonLiu1993 JonLiu1993 added info:reviewed Pull Request changes follow basic guidelines and removed requires:all-feature-testing vcpkg install port[all features supported by that port] needs to be demonstrated to function labels Jun 2, 2021
Copy link
Contributor

@ras0219-msft ras0219-msft left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the update!

@strega-nil-ms
Copy link
Contributor

LGTM, thanks @dg0yt :)

@AbdulAbCellera
Copy link

AbdulAbCellera commented Nov 27, 2022

Hi, I have a question about CXX feature support on arm.

I want to compile with the CXX feature for Apple M1 (arm64). So I did the following:

  • I removed "supports" : !arm
  • I added the cxx feature to vcpkg.json
  • I linked my targets to TIFF::CXX
  • I compiled

These changes do not seem to have any issues.

The question is what is the best approach to get this support natively?
Do I have to link to another target?
or post a PR for arm64 support of CXX feature?

@dg0yt
Copy link
Contributor Author

dg0yt commented Nov 27, 2022

I guess this particular arm restriction is obsolete. The next PR can remove the supports line from this feature. In the mean time you may also install tiff[cxx] with --allow-unsupported.

@AbdulAbCellera
Copy link

Is there a way to set --allow-unsupported in manifest mode?

@dg0yt
Copy link
Contributor Author

dg0yt commented Nov 27, 2022

Is there a way to set --allow-unsupported in manifest mode?

Yes, just pass is through with CMAKE_INSTALL_OPTIONSVCPKG_INSTALL_OPTIONS :
https://github.com/microsoft/vcpkg/blob/master/docs/users/buildsystems/cmake-integration.md#vcpkg_install_options

@AbdulAbCellera
Copy link

AbdulAbCellera commented Dec 20, 2022

I think you meant VCPKG_INSTALL_OPTIONS instead of CMAKE_INSTALL_OPTIONS :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist category:port-update The issue is with a library, which is requesting update new revision info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[tiff] update to 4.3.0 [tiff] Installed pkg-config (.pc) debug files specify non-existent includedir
7 participants