Ticket #6501 (closed: fixed)
OSX 10.8 crash while using browse
| Reported by: | Martyn Gigg | Owned by: | Owen Arnold |
|---|---|---|---|
| Priority: | blocker | Milestone: | Release 2.5 |
| Component: | Mantid | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Stuart Campbell |
Change History
comment:2 Changed 8 years ago by Nick Draper
- Status changed from new to assigned
- Owner set to Owen Arnold
comment:4 Changed 8 years ago by Owen Arnold
This is still an issue. This is not fixed on Mountain Lion by moving to the new version of Qt. I tried this in my development environment and it still crashes.
comment:5 Changed 8 years ago by Owen Arnold
- Owner changed from Owen Arnold to Nick Draper
- Status changed from accepted to assigned
I'm temporarily reassigning this to Nick, because I want to know if he'd like me to find a fix for this. My job was to initially see whether the bug still existed - it does.
comment:6 Changed 7 years ago by Owen Arnold
- Owner changed from Nick Draper to Owen Arnold
I've talked to martyn about this, and we're going to disable the use of the native file dialog. This can be done in one-shot in FileDialogHandler.h (MantidQt/API)
comment:8 Changed 7 years ago by Owen Arnold
refs #6501. Use non-native dialog always.
Changeset: dfa115ad8d3535c3d54bc5f0830d00a2f68fa5ac
comment:9 Changed 7 years ago by Owen Arnold
Tester:
Branch is bugfix/6501_browse.
The scenario in the ticket description should no longer cause a crash.
comment:10 Changed 7 years ago by Owen Arnold
- Description modified (diff)
- Summary changed from Investigate whether crash with Mac save dialogs is solved with Qt 4.8 to OSX 10.8 crash while using browse
comment:11 Changed 7 years ago by Owen Arnold
- Status changed from accepted to verify
- Resolution set to fixed
comment:12 Changed 7 years ago by Russell Taylor
- Status changed from verify to verifying
- Tester set to Russell Taylor
comment:13 Changed 7 years ago by Russell Taylor
- Status changed from verifying to verify
- Tester Russell Taylor deleted
I don't see this crash with either the latest installed nightly build or a local build (of the code before the patch).
comment:14 Changed 7 years ago by Owen Arnold
refs #6501. Use non-native dialog always.
Changeset: dfa115ad8d3535c3d54bc5f0830d00a2f68fa5ac
comment:15 Changed 7 years ago by Owen Arnold
refs #6501. Use non-native dialog always.
Changeset: dfa115ad8d3535c3d54bc5f0830d00a2f68fa5ac
comment:16 Changed 7 years ago by Owen Arnold
refs #6501. Use non-native dialog always.
Changeset: dfa115ad8d3535c3d54bc5f0830d00a2f68fa5ac
comment:17 Changed 7 years ago by Stuart Campbell
- Status changed from verify to verifying
- Tester set to Stuart Campbell
comment:18 Changed 7 years ago by Stuart Campbell
Indeed doesn't cause crash and always uses non-native dialogs.
comment:19 Changed 7 years ago by Owen Arnold
refs #6501. Use non-native dialog always.
Changeset: dfa115ad8d3535c3d54bc5f0830d00a2f68fa5ac
comment:21 Changed 7 years ago by Owen Arnold
refs #6501. Use non-native dialog always.
Changeset: dfa115ad8d3535c3d54bc5f0830d00a2f68fa5ac
comment:22 Changed 7 years ago by Owen Arnold
refs #6501. Use non-native dialog always.
Changeset: dfa115ad8d3535c3d54bc5f0830d00a2f68fa5ac
comment:23 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7347
The changes made ticket #6497 should no longer be necessary once the fix is sorted and should be reverted. Also the special FileDialogHandler struct can go too as it was seemingly never used.