-
Notifications
You must be signed in to change notification settings - Fork 78
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
fix: workaround to crash on startup with Qt 5.15.11 #12721
Conversation
Jenkins BuildsClick to see older builds (8)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but please consider adding link to the bug as a comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Woah! Awesome fix.
1cbed92
to
9831058
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nasty thing to find
the most recent Qt upstream version introduced a regression that causes our app to crash right on startup when a grouped property is assigned to inside a `PropertyChanges` handler upstream issue: https://bugreports.qt.io/browse/QTBUG-107795 Upstream fix will be released with Qt 5.15.12, code being at https://codereview.qt-project.org/c/qt/qtdeclarative/+/424702
9831058
to
ee3a15c
Compare
the most recent Qt upstream version introduced a regression that causes our app to crash right on startup when a grouped property is assigned to inside a
PropertyChanges
handlerupstream issue: https://bugreports.qt.io/browse/QTBUG-107795
Upstream fix will be released with Qt 5.15.12, code being at https://codereview.qt-project.org/c/qt/qtdeclarative/+/424702
Culprit found thanks to
PagesValidator
fromstorybook
What does the PR do
Fixes a nasty crash with Qt 5.15.11
Affected areas
ALL (indirectly via SendModal)
Screenshot of functionality (including design for comparison)
Back in business ;)