<table><tr><td style="">rjvbb updated this revision to Diff 46025.<br />rjvbb added a comment.

</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D4929">View Revision</a></tr></table><br /><div><div><p>Refactored for the standalone DrKonqi repo and disabled the integration testing on Mac.</p>

<p>Making DrKonqi standalone is a good step, I'd strongly suggest to move it to KF5 Applications at the first possible occasion. The utility doesn't even depend on a single Plasma library and provides a service that has nothing Plasma-desktop specific.</p>

<p>Instead, ask yourself if automatic crash reports are welcome only from Plasma desktop users or if as many users as possible should be able to submit crash reports (i.e. from any platform where DrKonqi is functional). Better, don't ask yourself, ask the entire family of KDE developers.</p>

<p>On a related note: DrKonqi's dependencies have been bumped along with the other Plasma dependencies. That's overkill: it has no business requiring Qt 5.11, 5.9LTS provides all required APIs. Similarly, it builds just fine against KF5 Frameworks 5.47.0, possibly even earlier versions.</p></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D4929?vs=12368&id=46025">https://phabricator.kde.org/D4929?vs=12368&id=46025</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4929">https://phabricator.kde.org/D4929</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/CMakeLists.txt<br />

src/aboutbugreportingdialog.cpp<br />

src/backtracegenerator.cpp<br />

src/backtracegenerator.h<br />

src/backtracewidget.cpp<br />

src/bugzillaintegration/reportassistantdialog.cpp<br />

src/data/AppleTerminal<br />

src/data/CMakeLists.txt<br />

src/data/debuggers/external.mac/gdbrc<br />

src/data/debuggers/external.mac/kdbgrc<br />

src/data/debuggers/external.mac/lldbrc<br />

src/data/debuggers/external/lldbrc<br />

src/data/debuggers/internal/lldbrc<br />

src/debugger.cpp<br />

src/debugger.h<br />

src/drkonqibackends.cpp<br />

src/drkonqidialog.cpp<br />

src/main.cpp<br />

src/parser/CMakeLists.txt<br />

src/parser/backtraceparser.cpp<br />

src/parser/backtraceparserlldb.cpp<br />

src/parser/backtraceparserlldb.h<br />

src/tests/CMakeLists.txt</div></div></div><br /><div><strong>To: </strong>rjvbb, Plasma: Workspaces, kfunk<br /><strong>Cc: </strong>patrickelectric, kfunk, mart, broulik, kde-mac, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>