<div dir="auto">Hi Mitch,<div dir="auto"><br></div><div dir="auto">Check also¬†<a href="https://bugreports.qt.io/browse/QTBUG-71811">https://bugreports.qt.io/browse/QTBUG-71811</a>. It might be related too.<br><br><div data-smartmail="gmail_signature" dir="auto">--<br>J-P Nurmi</div></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 14, 2018, 11:28 Mitch Curtis <<a href="mailto:mitch.curtis@qt.io">mitch.curtis@qt.io</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi.<br>
<br>
My fixes for <a href="https://bugreports.qt.io/browse/QTBUG-67559" rel="noreferrer noreferrer" target="_blank">https://bugreports.qt.io/browse/QTBUG-67559</a> have recently introduced a crash in KDE:<br>
<br>
<a href="https://bugreports.qt.io/browse/QTBUG-71770" rel="noreferrer noreferrer" target="_blank">https://bugreports.qt.io/browse/QTBUG-71770</a><br>
<br>
Quoting the commit message from my patch that reverts¬† <a href="https://codereview.qt-project.org/#/c/245480/" rel="noreferrer noreferrer" target="_blank">https://codereview.qt-project.org/#/c/245480/</a>:<br>
<br>
"Revert all Menu delegate patches<br>
<br>
This reverts the following commits:<br>
d5cb26bc56a3b6f6e99c88654d4f7a65f43551ac - Menu: ensure the correct delegates are used when created via Component<br>
d923dd467c1aeb3e195a09949b04862084002f88 - MenuBar: ensure the correct delegates are used when created via Component<br>
d56c193eb4ceb640611d66f22e1f26aae91cd7d1 - QQuickPopupPositioner: avoid adding duplicate item change listeners<br>
567a2de8cd493aabe0055d6dbc367b39447e70dd - Stabilize tst_qquickmenubar<br>
953fbac6131823e4fce0eb4707a854469c4c04ff - Fix Instantiator-created MenuItems disappearing<br>
936d31179d44220571ded15840bedeccb581c83b - tst_qquickmenu: add a test for MenuItems before and after a Repeater<br>
fc1832810f6c09505d9413685ed0b2d6295bea4a - QQuickMenuBar: fix menu not opening<br>
<br>
The fix for QTBUG-67559 has caused lots of issues, with the latest being a<br>
crash right before the 5.12 release. The bug that they fix is a P2,<br>
so it's not worth the hassle. The patches might be able to be resubmitted<br>
to dev after the crash is fixed.<br>
<br>
Change-Id: Ic192c7a302176bcdb2503b636b3462b10898a2ba<br>
Fixes: QTBUG-71770"<br>
<br>
I believe that the patches are correct in principle, and the crash is probably just an oversight and not a fundamental problem, but I don't want to hold up the release for a P2 that has already required lots of follow-up fixes.<br>
<br>
Jani said:<br>
<br>
"Ok, but then we should agree if QTBUG-71770 is a blocker or can we release Qt 5.12.0 with it as known issue. It is a crash but if you can't reproduce the issue it sounds like a corner case for me and so on we should be able to keep it as known issue to minimize new risks."<br>
<br>
So I thought I'd check here since I haven't gotten any response elsewhere when asking about what to do.<br>
<br>
Cheers.<br>
<br>
_______________________________________________<br>
Development mailing list<br>
<a href="mailto:Development@qt-project.org" target="_blank" rel="noreferrer">Development@qt-project.org</a><br>
<a href="http://lists.qt-project.org/mailman/listinfo/development" rel="noreferrer noreferrer" target="_blank">http://lists.qt-project.org/mailman/listinfo/development</a><br>
</blockquote></div>