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

[opencv4] Add GTK support for opencv4 portfile #10886

Merged
merged 1 commit into from
May 1, 2020

Conversation

shelper
Copy link
Contributor

@shelper shelper commented Apr 17, 2020

Describe the pull request
Add GTK support for opencv4 portfile

  • What does your PR fix? Fixes issue #
    with out this diff, the opencv4 built by vcpkg causes below error when calling GUI based functions like cv::imshow():
    Rebuild the library with Windows, GTK+ 2.x or Carbon support. If you are on Ubuntu or Debian, install libgtk2.0-dev and pkg-config, then re-run cmake or configure script in function cvimshow

  • Which triplets are supported/not supported? Have you updated the CI baseline?
    triplets tested: x64-linux (should also affect x86-linux)
    no update to the CI baseline

  • Does your PR follow the maintainer guide?
    It seems to comply with the guide

@shelper shelper changed the title Add GTK support for opencv4 portfile [opencv4] Add GTK support for opencv4 portfile Apr 17, 2020
@LilyWangL LilyWangL self-assigned this Apr 20, 2020
@LilyWangL
Copy link
Contributor

LilyWangL commented Apr 20, 2020

Thanks for your PR! Can you please correct the Version field in the CONTROL file? And you should add Feature:gtk to CONTROL. You can get more information from maintainer-guide.md.

@msftclas
Copy link

msftclas commented Apr 21, 2020

CLA assistant check
All CLA requirements met.

@shelper
Copy link
Contributor Author

shelper commented Apr 21, 2020

Updated.

@LilyWangL LilyWangL added needs-feature-review info:reviewed Pull Request changes follow basic guidelines and removed waiting for response labels Apr 21, 2020
@strega-nil
Copy link
Contributor

strega-nil commented May 1, 2020

LGTM, thanks @shelper :) I'm just gonna run it on the new PR CI pipeline.

@strega-nil
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@strega-nil strega-nil merged commit 3a1e8e0 into microsoft:master May 1, 2020
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