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

[vcpkg_acquire_msys.cmake] Add gettext tool #13297

Merged
merged 2 commits into from
Sep 4, 2020
Merged

[vcpkg_acquire_msys.cmake] Add gettext tool #13297

merged 2 commits into from
Sep 4, 2020

Conversation

StarGate-One
Copy link
Contributor

Describe the pull request
[vcpkg_acquire_msys.cmake] Add gettext tool needed by libpq

@NancyLi1013 NancyLi1013 self-assigned this Sep 2, 2020
@NancyLi1013 NancyLi1013 added the category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly label Sep 2, 2020
scripts/cmake/vcpkg_acquire_msys.cmake Outdated Show resolved Hide resolved
@StarGate-One StarGate-One deleted the vcpkg_acquire_msys branch September 2, 2020 20:21
@StarGate-One StarGate-One restored the vcpkg_acquire_msys branch September 3, 2020 00:39
@StarGate-One StarGate-One reopened this Sep 3, 2020
 1. Added dependencies to gettext-0.19.8.1-1-x86_64.pkg.tar.xz
    of libintl libgettextpo libasprintf
 2. Added dependency download for libgettextpo-0.19.8.1-1-x86_64.pkg.tar.xz
 3. Added dependency download for libasprintf-0.19.8.1-1-x86_64.pkg.tar.xz
 4. Put downloads in reverse dependcy order - seems it is called
    topological order now
Copy link
Contributor

@ras0219 ras0219 left a comment

Choose a reason for hiding this comment

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

Thanks for making the fixes and sorry for the confusing term! This is exactly what I meant :)

The reason we want to split up the msys2 directories is to avoid one port having a "hidden" dependency on another -- for example, when A installs gettext we don't want it to silently also be provided to B.

@NancyLi1013 NancyLi1013 added the info:reviewed Pull Request changes follow basic guidelines label Sep 4, 2020
@BillyONeal BillyONeal merged commit a3142da into microsoft:master Sep 4, 2020
@BillyONeal
Copy link
Member

Thanks for your contribution!

@StarGate-One
Copy link
Contributor Author

Thanks for your contribution!

You are most welcome @BillyONeal. Have a wonderful weekend... Thank you for all the hard work. 👍🏻

@StarGate-One StarGate-One deleted the vcpkg_acquire_msys branch September 4, 2020 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[vcpkg_acquire_msys.cmake] missing download of gettext
4 participants