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.

Sign up for GitHub

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

Jump to bottom

wxwidgets: Update to v3.2.6 #4486

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

wxwidgets: Update to v3.2.6 #4486

wants to merge 11 commits into from

Conversation

androidnisse
Copy link
Contributor

Summary

  • Release notes can be found here.

Test Plan

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged (Write an appropriate message in the Summary section)

@androidnisse androidnisse changed the title Update wxwidgets wxwidgets: Update to v3.2.6 Nov 29, 2024
EbonJaeger
EbonJaeger requested changes Dec 14, 2024
View reviewed changes
Copy link
Member

@EbonJaeger EbonJaeger left a comment

Choose a reason for hiding this comment

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

While you're here, could you add monitoring.yml files to both packages?

EbonJaeger
EbonJaeger previously approved these changes Dec 16, 2024
View reviewed changes
Copy link
Member

@EbonJaeger EbonJaeger left a comment

Choose a reason for hiding this comment

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

LGTM, if someone wants to merge it. I don't know that I'll get to it today.

@davidjharder
Copy link
Member

Sadly it seems rebuilds might be a good idea, I can't get codelite to build with new wxwidgets

@EbonJaeger EbonJaeger dismissed their stale review December 16, 2024 23:24

Reported revdep failure to build

@davidjharder
Copy link
Member

You have a conflict with cemu because it was updated in the time since this PR was started. You will have a similar issue with audacity. You will need to rebase

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EbonJaeger EbonJaeger EbonJaeger left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
Solus
Status: Triage
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Please update wxwidgets to 3.2.5 or higher to resolve crash with veracrypt
3 participants
@androidnisse @davidjharder @EbonJaeger