I wanted to thank you all for your feedback. It sounds to me that, based on that feedback:
The change has been implemented in commit 4aa5a0f7f93703d22e2cc81442d64a1bb2d81a46
Sorry I didn't notice this as kdesrc-build development has moved to invent.kde.org
Add missing arch dependencies
git: Show warning at end of build if git-stash fails.
module: Add ability to track messages to show user after build ends.
git: Remove dead code from invent migration.
git: Show warning at end of build if git-stash fails.
git: Remove dead code from invent migration.
module: Add ability to track messages to show user after build ends.
Merge remote-tracking branch 'origin/master' into make_it_mojo
mojo: Try to make 'airport' display on module build more clear.
mojo: Simplify error handling and debug msg from backend to frontend.
doc: Remove anongit references.
Merge remote-tracking branch 'origin/master' into fix-ipc-persistent-options
git: Revert change to git-desired-protocol.
ipc: Fix handling of persistent option changes in a subprocess.
kde_projects: Locate KDE projects post-Gitlab migration.
kde_projects: Follow symlinks to ensure metadata repo is read.
ui: Apply the clickable logdir from master as we need it in make_it_mojo.
Merge branch 'master' into make_it_mojo
ui: Make the summary output logdir clickable in most TTYs.
Updated embedded Mojolicious to 8.42.
Fix check for number of params in import script.
Merge remote-tracking branch 'origin/master' into make_it_mojo
git: Have git handle 'autostash' directly.
Merge branch 'release/20.04'
playlist: Fix crasher with playlists using reserved columns.
Merge branch 'prepare-for-new-metadata' into 'master'
Merge branch 'prepare-for-invent-migration' into 'master'
git: Capture basic git-status info before error-prone ops.
Merge branch 'alpine-support' into 'master'
Merge branch 'readme-dependency' into 'master'
Merge branch 'bump-qt-version-by-default' into 'master'
dep-resolv: Clarify test so you do not need to understand a code comment to…
dep-resolv: Complete test suite for dep-graph dep ordering.
dep-resolv: Improve code clarity and tests coverage for rc-ordering.
dep-resolv: Try to sort in rc-file order if not otherwise prevented.
git: Fix detection of repos with local changes to stash.
Thanks! This is a much better way to implement this, even if I had found git-diff-index in my search when I was implementing this I'm sure I would have missed the and cmdline options that make this a simple check. Please let me know if you're not able to commit and I will do so instead.
Remove or port FIXMEs in playlist shared settings and track iterator.
ui: Fix runtime warning about duplicate SVG elements.
playlist: Fix compiler warning about shadowing a variable name.
viewmode: Remove or port remaining code that is marked for FIXME.
playlist: Simplify dynamic playlist tracking of other playlists.
playlist: Move M3U-based file loading to happen after ctor runs.
playlist: Clarify that static use of virtual methods in destructor is…
playlist: Don't rely on overridden column count during construction.
ui: Fix bug in scrolling from playlist to search line.
ui: Fix return key failing to start playback.
search: Fix crash when clicking search playlist after startup.
Merge branch 'searchplaylist-fix' into 'master'
Merge branch 'search-proxy' into 'master'
GIT_SILENT: Remove unneeded TODO, this was already done.
Merge branch 'remove-observer' into 'master'
Replace observer pattern of PlaylistInterface
Merge branch 'first-run' into 'master'
Add zsh support in ksb::FirstRun
mojo: Note compiler warnings if any are emitted.
mojo: Show elapsed time for each module again.
GIT_SILENT mojo: Update docs for StatusMonitor to reflect number of events.
GIT_SILENT mojo: Add reminder for future porting.
mojo: Properly reject promise when a phase fails.
mojo: Add info about which phases were performed to onComplete output.
mojo: Show a bit more context for built modules.
Fix ignore-modules breakage.
Merge branch 'FirstRun/Fedora' into 'master'
Add missing FirstRun package for Fedora
Merge branch 'drag-move-view' into 'master'
Thanks Harald. Patch looks good and I've checked that the file name pattern is what Dr. Konqi uses by default. One question though, @broulik you mentioned there's also text/x-apport, should that be changed here too or is this changeset just for *.kcrash.txt? If the apport mention is for something else then I'd say to go ahead and commit.
Merge remote-tracking branch 'origin/master' into make_it_mojo
Add a script to import Mojolicious from an installed version.
Update embedded Mojolicious to 8.30.
Merge branch 'support-cmake-toolchain-files' into 'master'
Use https as default git protocol.
Reduce number of jobs in default config when building qtwebengine.
Merge branch 'cmake-with-ninja' into 'master'
Merge remote-tracking branch 'origin/release/19.12'
Merge branch 'unify-volume' into 'release/19.12'
Merge branch 'cblack-master-patch-71400' into 'master'
Merge branch 'close-saveconfig' into 'release/19.12'
Continued search widget cleanup; remove needless KToolBar usage.
Merge branch 'search-hig' into 'master'
Refactor: Use Qt class for the SearchWidget
Merge branch 'fix-all-artists-item' into 'master'
fix(cover-manager): change <All Artists> to <All Artists>
Looks good to me as well. Thanks for the work on this!
Hush the trivial dep warning. GIT_SILENT
module_list: Fix ignoring module-sets.
GIT_SILENT Yearly version bump.
Merge branch 'missingFirstRunPackages' into 'master'
mojo: Check for failed build before applying option updates.
Merge branch 'disablePhabricator' into 'master'