Skip to content

Conversation

NancyLi1013
Copy link
Contributor

windows.h is only needed on Windows platform.
So I updated it only when WIN32, the related definitions could be added.
Other changes:

  • Remove include(vcpkg_common_functions)
  • Add unsupported messages
  • Update copyright
  • Add Supports: widows field to CONTROL file

Related issue #10570

Note: No features need to test.

@JackBoosY JackBoosY added the info:reviewed Pull Request changes follow basic guidelines label Mar 27, 2020
@dan-shaw dan-shaw added waiting for response and removed info:reviewed Pull Request changes follow basic guidelines labels Mar 28, 2020
@JackBoosY
Copy link
Contributor

Needs to wait to resolve the osg-qt regression in #10637.

@JackBoosY JackBoosY requested a review from dan-shaw April 1, 2020 05:57
@JackBoosY
Copy link
Contributor

/azp run

@JackBoosY JackBoosY added the info:reviewed Pull Request changes follow basic guidelines label Apr 16, 2020
@ras0219-msft ras0219-msft merged commit 56738de into microsoft:master Apr 24, 2020
@ras0219-msft
Copy link
Collaborator

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants