Skip to content

[Windows] Add wxWidgets custom port with fixed release build#4787

Closed
AenBleidd wants to merge 1 commit intoBOINC:masterfrom
AenBleidd:vko_4786_fix_wxwidgets
Closed

[Windows] Add wxWidgets custom port with fixed release build#4787
AenBleidd wants to merge 1 commit intoBOINC:masterfrom
AenBleidd:vko_4786_fix_wxwidgets

Conversation

@AenBleidd
Copy link
Member

This fixes #4786.

Signed-off-by: Vitalii Koshura lestat.de.lionkur@gmail.com

This fixes BOINC#4786.

Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
@talregev
Copy link
Contributor

vcpkg PR:
microsoft/vcpkg#25240

Copy link
Contributor

@talregev talregev left a comment

Choose a reason for hiding this comment

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

vcpkg approve this change, it mean that it soon it will be in vcpkg master.
Also after talk with @AenBleidd, the change seems logical.
Also it compile and work on our CI.
I am prefer to wait vcpkg merge and not merge this PR to the maseter:

  • Next batch vcpkg will merge this change.
  • If this PR will merge, we will need to have a next PR that revert that changes.
  • We usually waits for vcpkg PRs, unless is an hot fix for now.

@AenBleidd
Copy link
Member Author

Closed in favor of microsoft/vcpkg#25240

@AenBleidd AenBleidd closed this Jun 17, 2022
@AenBleidd AenBleidd deleted the vko_4786_fix_wxwidgets branch June 17, 2022 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Windows] wxWidgets built with vcpkg have wxDEBUG_LEVEL defined

2 participants