Skip to content

MinGW: Fix vcpkg common definitions#9807

Merged
vicroms merged 1 commit intomicrosoft:masterfrom
cristianadam:mingw-vcpkg-common-definitions
Jan 23, 2020
Merged

MinGW: Fix vcpkg common definitions#9807
vicroms merged 1 commit intomicrosoft:masterfrom
cristianadam:mingw-vcpkg-common-definitions

Conversation

@cristianadam
Copy link
Contributor

Since MinGW has both VCPKG_TARGET_IS_MINGW and VCPKG_TARGET_IS_WINDOWS,
make sure to test the first before.

Triggered by #9137 (review)

Since MinGW has both VCPKG_TARGET_IS_MINGW and VCPKG_TARGET_IS_WINDOWS,
make sure to test the first before.
@cristianadam cristianadam requested a review from vicroms January 23, 2020 18:57
@vicroms vicroms added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Jan 23, 2020
@vicroms
Copy link
Member

vicroms commented Jan 23, 2020

Thanks for fixing this so quickly!

@vicroms vicroms merged commit 8c77506 into microsoft:master Jan 23, 2020
@cristianadam cristianadam deleted the mingw-vcpkg-common-definitions branch January 23, 2020 20:35
Jimmy-Hu added a commit to Jimmy-Hu/vcpkg that referenced this pull request Jan 24, 2020
MinGW: Fix vcpkg common definitions (microsoft#9807)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants