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

[many ports] Don't depend on default features of tiff #38049

Merged
merged 85 commits into from
May 2, 2024

Conversation

Thomas1664
Copy link
Contributor

@Thomas1664 Thomas1664 commented Apr 8, 2024

This removes the transitive dependency of liblzma via tiff

  • Changes comply with the maintainer guide.
  • SHA512s are updated for each updated download.
  • The "supports" clause reflects platforms that may be fixed by this new version.
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

@Thomas1664 Thomas1664 closed this Apr 8, 2024
@Thomas1664 Thomas1664 reopened this Apr 8, 2024
@Osyotr
Copy link
Contributor

Osyotr commented Apr 8, 2024

Duplicate #38047?

@Thomas1664 Thomas1664 changed the title [freeimage] Don't depend on default features of tiff [many ports] Don't depend on default features of tiff Apr 8, 2024
@Thomas1664 Thomas1664 marked this pull request as draft April 8, 2024 23:10
@Thomas1664
Copy link
Contributor Author

Duplicate #38047?

Probably should have checked beforehand. But why stopping there?

@WangWeiLin-MV WangWeiLin-MV self-assigned this Apr 9, 2024
@WangWeiLin-MV WangWeiLin-MV added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Apr 9, 2024
@dg0yt
Copy link
Contributor

dg0yt commented Apr 9, 2024

But why stopping there?

Because #38047 revises all dependencies in one port, instead of one dep in many ports. This reduces overlap with other (people's) PRs.

@Thomas1664
Copy link
Contributor Author

Because #38047 revises all dependencies in one port, instead of one dep in many ports. This reduces overlap with other (people's) PRs.

I will remove the changes to freeimage. However, I don't think it makes sense to open one PR per port because this would take like 30+ PRs.

ports/colmap/portfile.cmake Outdated Show resolved Hide resolved
@Thomas1664 Thomas1664 marked this pull request as ready for review April 22, 2024 21:30
@Thomas1664
Copy link
Contributor Author

@WangWeiLin-MV this PR is ready for review.

@WangWeiLin-MV
Copy link
Contributor

Please resolve conflicts.

WangWeiLin-MV
WangWeiLin-MV previously approved these changes Apr 26, 2024
@WangWeiLin-MV WangWeiLin-MV added the info:reviewed Pull Request changes follow basic guidelines label Apr 28, 2024
@data-queue data-queue merged commit 3a882b3 into microsoft:master May 2, 2024
17 checks passed
yurybura pushed a commit to yurybura/vcpkg that referenced this pull request May 8, 2024
This removes the transitive dependency of liblzma via tiff

- [x] Changes comply with the [maintainer
guide](https://github.com/microsoft/vcpkg-docs/blob/main/vcpkg/contributing/maintainer-guide.md).
- [x] SHA512s are updated for each updated download.
- [x] The "supports" clause reflects platforms that may be fixed by this
new version.
- [x] Any fixed [CI
baseline](https://github.com/microsoft/vcpkg/blob/master/scripts/ci.baseline.txt)
entries are removed from that file.
- [x] Any patches that are no longer applied are deleted from the port's
directory.
- [x] The version database is fixed by rerunning `./vcpkg x-add-version
--all` and committing the result.
- [x] Only one version is added to each modified port's versions file.

---------

Co-authored-by: Alexander Neumann <30894796+Neumann-A@users.noreply.github.com>
Co-authored-by: Alexander Neumann <you@example.com>
Co-authored-by: Billy Robert O'Neal III <bion@microsoft.com>
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 info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants