Skip to content

[qt5-webengine] Fix issues#16770

Merged
vicroms merged 8 commits intomicrosoft:masterfrom
Neumann-A:fix_qt5-webengine_issues
Mar 30, 2021
Merged

[qt5-webengine] Fix issues#16770
vicroms merged 8 commits intomicrosoft:masterfrom
Neumann-A:fix_qt5-webengine_issues

Conversation

@Neumann-A
Copy link
Contributor

@Neumann-A Neumann-A commented Mar 18, 2021

changes from #16583

  • add feature "proprietary-codecs"

  • fix build error due to internal changes how options are passed.

  • try to fail early on too long build paths instead of late

  • wait for CI

  • Run vcpkg x-add-version --all

 - add feature "proprietary-codecs"
 - fix build error due to internal changes how options are passed.
 - try to fail early on too long build paths instead of late
and only build one configuration to not overstress the ci system
@JackBoosY JackBoosY self-assigned this Mar 18, 2021
@JackBoosY JackBoosY added the category:port-bug The issue is with a library, which is something the port should already support label Mar 18, 2021
@JackBoosY
Copy link
Contributor

Seems CI passed.

@Neumann-A Neumann-A marked this pull request as ready for review March 18, 2021 19:56
@strega-nil
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@ras0219-msft ras0219-msft 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 the PR! LGTM once we use the official helper.

If you would like it to be a fatal error, we should rely on the official helper to provide the message and then emit the fatal error afterwards.

Neumann-A and others added 2 commits March 26, 2021 10:12
Co-authored-by: Robert Schumacher <roschuma@microsoft.com>
@JackBoosY
Copy link
Contributor

Depends on #16904

@JackBoosY JackBoosY added the depends:different-pr This PR or Issue depends on a PR which has been filed label Mar 26, 2021
@JackBoosY JackBoosY removed the depends:different-pr This PR or Issue depends on a PR which has been filed label Mar 29, 2021
@JackBoosY JackBoosY requested a review from ras0219-msft March 29, 2021 17:12
@JackBoosY JackBoosY added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Mar 29, 2021
@vicroms vicroms merged commit ed54efb into microsoft:master Mar 30, 2021
@Neumann-A Neumann-A deleted the fix_qt5-webengine_issues branch March 30, 2021 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support 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.

5 participants