Summary: | Purpose: send to ReviewBoard doesn't work on OS X and (sometimes) causes a crash on exit | ||
---|---|---|---|
Product: | [Developer tools] kdevplatform | Reporter: | RJVB <rjvbertin> |
Component: | patchreview | Assignee: | kdevelop-bugs-null |
Status: | RESOLVED FIXED | ||
Severity: | grave | CC: | aleixpol, david.nolden.kde |
Priority: | NOR | ||
Version: | git master | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | macOS | ||
Latest Commit: | https://commits.kde.org/purpose/3b558384a367ad1a9005ae00a9b53f1a922520fc | Version Fixed In: | |
Sentry Crash Report: | |||
Attachments: | purposeprocess is a nongui exec |
Description
RJVB
2016-12-30 19:33:22 UTC
I think this was operator error; I forgot to apply a patch which explains why the executable wasn't found. Uploading works again, but the dialog is still sized wrong each time I open it (vertical height too small). Created attachment 103098 [details]
purposeprocess is a nongui exec
purposeprocess must not be built as an app bundle on OS X, but as a regular executable that will be installed on the path.
Can you please submit that patch? To be certain: submit for review or commit directly? Whichever you feel most comfortable with Git commit 3b558384a367ad1a9005ae00a9b53f1a922520fc by René J.V. Bertin. Committed on 31/12/2016 at 19:25. Pushed by rjvbb into branch 'master'. purposeprocess should be a nongui executable Mark purposeprocess as a nongui executable to prevent building it as an app bundle on Mac. M +1 -0 CMakeLists.txt M +1 -0 src/CMakeLists.txt https://commits.kde.org/purpose/3b558384a367ad1a9005ae00a9b53f1a922520fc |