[kdesrc-build] [Bug 109501] [Feature request] Quiet mode
https://bugs.kde.org/show_bug.cgi?id=109501 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/-/commit/61688 ||25682c81d3e6984b698d8cfa822 ||bac1797d --- Comment #3 from Michael Pyne --- Git commit 6168825682c81d3e6984b698d8cfa822bac1797d by Michael Pyne. Committed on 25/07/2005 at 02:45. Pushed by ashark into branch 'docbook_historied_per_file'. Add documentation for the recently added --quiet option. Also, make --dry-run an alias for --pretend since --dry-run seems to be a common name for that type of option. svn path=/trunk/KDE/kdesdk/doc/scripts/kdesvn-build/; revision=438341 Original commit: 660d9d66 https://invent.kde.org/sdk/kdesrc-build/-/commit/660d9d660e3e8047fb1379e1d656472dcf6d4202 M +23 -1doc/cmdline/index.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/6168825682c81d3e6984b698d8cfa822bac1797d -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 194635] Add support for ionice
https://bugs.kde.org/show_bug.cgi?id=194635 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/-/commit/3324e ||f4ff7d46714bd18e91abb5508ad ||5b1b2c8c --- Comment #3 from Michael Pyne --- Git commit 3324ef4ff7d46714bd18e91abb5508ad5b1b2c8c by Michael Pyne. Committed on 31/01/2010 at 02:04. Pushed by ashark into branch 'docbook_historied_per_file'. Add support for automatically using the ionice tool supported in Linux since 2.6.13 (or so) using CFQ, in kdesvn-build. Also update the documentation at the same time for once. To enable, set use-idle-io-priority to true in your .kdesvn-buildrc (it is disabled by default for now). svn path=/trunk/KDE/kdesdk/doc/scripts/kdesvn-build/; revision=1082647 Original commit: 76bbcd80 https://invent.kde.org/sdk/kdesrc-build/-/commit/76bbcd80c6aed139da835aad9ad462f8d74cf7a4 M +2-2doc/index.docbook M +11 -0doc/kdesvn-buildrc/conf-options-table.docbook M +29 -14 doc/using-kdesvn-build/basic-features.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/3324ef4ff7d46714bd18e91abb5508ad5b1b2c8c -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 248826] Buggy feature: skip build when no source update occurs
https://bugs.kde.org/show_bug.cgi?id=248826 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/-/commit/9aee3 ||f12a3aaded1d016426bad966bfd ||41712683 --- Comment #4 from Michael Pyne --- Git commit 9aee3f12a3aaded1d016426bad966bfd41712683 by Michael Pyne. Committed on 28/08/2010 at 02:28. Pushed by ashark into branch 'docbook_historied_per_file'. Some fixes to make the "skipping build" feature hopefully more useful: * It is now controlled by an option 'build-when-unchanged' which defaults to true, so the optimization must now be explicitly enabled. * The build is only skipped if the module was successfully installed (as determined by the persistent data recorded last run). * A command-line option, --force-build is added, along with --no-build-when-unchanged to be symmetric with the rc-file option. The new command line option and rc-file option are also documented and added to the Kate/Kwrite syntax highlighting file. FIXED-IN:1.12.1 svn path=/trunk/KDE/kdesdk/doc/scripts/kdesrc-build/; revision=1168981 Original commit: 142c7911 https://invent.kde.org/sdk/kdesrc-build/-/commit/142c79caa8b2473fd93cfe2af2a905b7fbf2 M +15 -0doc/cmdline/supported-cmdline-params.docbook M +26 -0doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/9aee3f12a3aaded1d016426bad966bfd41712683 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 109044] Add "--remove-builddir" option to kdesvn-build
https://bugs.kde.org/show_bug.cgi?id=109044 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/-/commit/cbe91 ||666f39690f6f3febc5994ac7495 ||599f1e24 --- Comment #3 from Michael Pyne --- Git commit cbe91666f39690f6f3febc5994ac7495599f1e24 by Michael Pyne. Committed on 15/07/2005 at 01:12. Pushed by ashark into branch 'docbook_historied_per_file'. Add remove-after-install option to add wish 109044. This can be used to remove the builddir and/or source dir after a successful module install, for people with limited disk space. Supports the following values: none - Don't delete anything. Obviously this is the default. builddir - Delete the build directory after install. This isn't a command line option though, it needs to be set in your ~/.kdesvn-buildrc (it is possible to set it via the command line though by doing --remove-after-install=builddir). svn path=/trunk/KDE/kdesdk/doc/scripts/kdesvn-build/; revision=434684 Original commit: b89f6bea https://invent.kde.org/sdk/kdesrc-build/-/commit/b89f6bea3a3edf596b5a17bc1df48c0a576ffddb M +22 -0doc/kdesvn-buildrc/index.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/cbe91666f39690f6f3febc5994ac7495599f1e24 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 337446] malformed JSON string
https://bugs.kde.org/show_bug.cgi?id=337446 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/6911da5c8713c3e33e |kdesrc-build/-/commit/fab78 |8e43c6707a2c871bf79375 |b389048faff2db425e3e942da01 ||403c74c0 --- Comment #13 from Michael Pyne --- Git commit fab78b389048faff2db425e3e942da01403c74c0 by Michael Pyne. Committed on 16/07/2014 at 05:00. Pushed by ashark into branch 'docbook_historied_per_file'. Improve first-run behavior with build metadata. This commit makes the kde-build-metadata module a requirement instead of an option, since the vast majority of runs will require this module now. Additionally the --metadata-only option is added and documented to allow for downloading the kde-build-metadata module alone (and make the --pretend option work afterwards), and kdesrc-build recommends using it if you run with --pretend and without metadata. This should hopefully make the first-run use case easier for users. FIXED-IN:1.16 Original commit: 6911da5c https://invent.kde.org/sdk/kdesrc-build/-/commit/6911da5c8713c3e33e8e43c6707a2c871bf79375 M +24 -3doc/cmdline/supported-cmdline-params.docbook M +5-1doc/features/features-overview.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/fab78b389048faff2db425e3e942da01403c74c0 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 331654] cmake-git does not compile with ninja as build command
https://bugs.kde.org/show_bug.cgi?id=331654 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/9fb3fb04d170478c92 |kdesrc-build/-/commit/bca4e |2c5d85bb996f8f4b7ed82c |917547a7b1a720b18bd4598a4ac ||63d5eba4 --- Comment #4 from Michael Pyne --- Git commit bca4e917547a7b1a720b18bd4598a4ac63d5eba4 by Michael Pyne. Committed on 30/03/2014 at 03:39. Pushed by ashark into branch 'docbook_historied_per_file'. build-systems: Mask default options for non-default build systems. This is a first-step hack to have non-default build systems ignore global options which are related to the build system in use (e.g. compile or make flags, install/uninstall options, etc.). This should fix bug 331654 (now that I understand it better) and improve the situation for bug 332789 (which would be the long-term fix when implemented). Related: bug 332789 FIXED-IN:1.16 Original commit: 9fb3fb04 https://invent.kde.org/sdk/kdesrc-build/-/commit/9fb3fb04d170478c922c5d85bb996f8f4b7ed82c M +11 -8doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/bca4e917547a7b1a720b18bd4598a4ac63d5eba4 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 365813] options doesn't work for module-sets
https://bugs.kde.org/show_bug.cgi?id=365813 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/c3081844077eb53adb |kdesrc-build/-/commit/dc70f |8a9d58c8a3e3dc66921cee |55c7dc6e494fbc8a16b94e48111 ||a59b5740 --- Comment #4 from Michael Pyne --- Git commit dc70f55c7dc6e494fbc8a16b94e48111a59b5740 by Michael Pyne. Committed on 24/07/2016 at 17:01. Pushed by ashark into branch 'docbook_historied_per_file'. Allow 'options' groups to apply to named module-sets. This makes it more convenient to work with baseline configurations (that are 'include'd) without changing the upstream configuration. Still to do: add a warning when modules and module-sets end up sharing the same name somehow. FIXED-IN:16.08 Original commit: c3081844 https://invent.kde.org/sdk/kdesrc-build/-/commit/c3081844077eb53adb8a9d58c8a3e3dc66921cee M +1-1doc/index.docbook M +8-0doc/kdesrc-buildrc/kdesrc-buildrc-overview.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/dc70f55c7dc6e494fbc8a16b94e48111a59b5740 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 473859] Add short options for --include-dependencies and --no-include-dependencies
https://bugs.kde.org/show_bug.cgi?id=473859 Michael Pyne changed: What|Removed |Added Latest Commit|https://invent.kde.org/sdk/ |https://invent.kde.org/sdk/ |kdesrc-build/-/commit/5b155 |kdesrc-build/-/commit/5c875 |c5e29dd2d377fd1f84fb42d0e62 |8a40dd9c47b73c22f77be71f09a |9a9f368b|2051d3bf --- Comment #3 from Michael Pyne --- Git commit 5c8758a40dd9c47b73c22f77be71f09a2051d3bf by Michael Pyne. Committed on 07/09/2023 at 00:34. Pushed by ashark into branch 'docbook_historied_per_file'. Add short options for --[no-]include-dependencies. This adds and documents -d as a short alias for --include-dependencies, and a corresponding -D as its negation. While I'm at it, and at user request, this also adds short options for: * --help * --no-src * --src-only * --refresh-build * --ignore-modules * --resume-from and --resume-after Original commit: 5b155c5e https://invent.kde.org/sdk/kdesrc-build/-/commit/5b155c5e29dd2d377fd1f84fb42d0e629a9f368b M +11 -11 doc/cmdline/supported-cmdline-params.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/5c8758a40dd9c47b73c22f77be71f09a2051d3bf -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 352478] Support Identity
https://bugs.kde.org/show_bug.cgi?id=352478 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/21b5eb9371fbb0f5f4 |kdesrc-build/-/commit/7c266 |3f7cba7606c4bdb32fedd4 |5eb90aafece2786217021586042 ||c13ac24e --- Comment #3 from Michael Pyne --- Git commit 7c2665eb90aafece2786217021586042c13ac24e by Michael Pyne. Committed on 12/09/2015 at 20:57. Pushed by ashark into branch 'docbook_historied_per_file'. Add git-user option. This option can be used to setup the user.name and user.email git-config options within each new git repository cloned or installed from snapshot, including kde-projects repositories. This can be useful for KDE developers with multiple Git identities that they use for development. Module settings override global settings so you could e.g. have a global git-user option for KDE modules and then override specific modules or module-sets with a different git-user setting. This setting is only applied during the initial git-clone process though... if there's interest in having kdesrc-build re-apply settings later that's possible too. FIXED-IN:15.09 Original commit: 21b5eb93 https://invent.kde.org/sdk/kdesrc-build/-/commit/21b5eb9371fbb0f5f43f7cba7606c4bdb32fedd4 M +2-2doc/index.docbook M +31 -0doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/7c2665eb90aafece2786217021586042c13ac24e -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 406268] telepathy-accounts-signon does not seem to have a build system to use
https://bugs.kde.org/show_bug.cgi?id=406268 --- Comment #4 from Michael Pyne --- Git commit 6b226e2ce10c359f7b803a160020dcec7ec5b8ac by Michael Pyne. Committed on 04/05/2019 at 20:15. Pushed by ashark into branch 'docbook_historied_per_file'. Add support for Meson build system. New/updated config file options: * 'configure-flags', reused as the way to pass cmdline options to the meson setup command. * 'ninja-options', a new option to pass cmdline options to the `ninja` command. Note that ninja is mandated by Meson as the underlying build tool. Tested with https://github.com/plibither8/2048.cpp Fixes #27, reviewed in !8. Test suite passes and I continue to be able to build 2048.cpp. I've also validated that ninja-options is passed to ninja when building 2048.cpp, though this was a manual verification. Original commit: 6385f5e4 https://invent.kde.org/sdk/kdesrc-build/-/commit/6385f5e429dd11393b48690a33d67a66edeacfd2 M +43 -2doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/6b226e2ce10c359f7b803a160020dcec7ec5b8ac -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 398858] kdesrc-build should use HTTPS instead of SSH for anonymous git clones
https://bugs.kde.org/show_bug.cgi?id=398858 Michael Pyne changed: What|Removed |Added Latest Commit|2c09ca0d8bc469d9860fc293b3e |https://invent.kde.org/sdk/ |1eae2814dd4cb |kdesrc-build/-/commit/96093 ||c4fead6ea56dd037335786fd31e ||6459d255 --- Comment #4 from Michael Pyne --- Git commit 96093c4fead6ea56dd037335786fd31e6459d255 by Michael Pyne, on behalf of Johan Ouwerkerk. Committed on 23/09/2018 at 00:21. Pushed by ashark into branch 'docbook_historied_per_file'. Use HTTPS instead of Git protocol as default git transport. With this change, users of kdesrc-build have better protection from man in the middle attacks on source code transfers by default without having to switch to SSH-tunneled Git protocol. FIXED-IN:18.09 Original commit: 2c09ca0d https://invent.kde.org/sdk/kdesrc-build/-/commit/2c09ca0d8bc469d9860fc293b3e1eae2814dd4cb M +1-1doc/getting-started/before-building.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/96093c4fead6ea56dd037335786fd31e6459d255 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 342416] Please add qmake-options to be able to pass options to qmake
https://bugs.kde.org/show_bug.cgi?id=342416 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/432efae0a9841be849 |kdesrc-build/-/commit/70e5f |c78f9ef24ea48296f3c19c |241e3c2e35507f4446f653e4c0c ||48c6c8a5 --- Comment #2 from Michael Pyne --- Git commit 70e5f241e3c2e35507f4446f653e4c0c48c6c8a5 by Michael Pyne. Committed on 07/02/2015 at 22:31. Pushed by ashark into branch 'docbook_historied_per_file'. qmake: Add qmake-options to pass options to qmake. Obviously this only applies to the few modules that utilize the qmake build system. FIXED-IN:1.16 Original commit: 432efae0 https://invent.kde.org/sdk/kdesrc-build/-/commit/432efae0a9841be849c78f9ef24ea48296f3c19c M +1-1doc/index.docbook M +14 -0doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/70e5f241e3c2e35507f4446f653e4c0c48c6c8a5 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 332789] kdesrc-build should support giving options for each build system
https://bugs.kde.org/show_bug.cgi?id=332789 --- Comment #7 from Michael Pyne --- Git commit bca4e917547a7b1a720b18bd4598a4ac63d5eba4 by Michael Pyne. Committed on 30/03/2014 at 03:39. Pushed by ashark into branch 'docbook_historied_per_file'. build-systems: Mask default options for non-default build systems. This is a first-step hack to have non-default build systems ignore global options which are related to the build system in use (e.g. compile or make flags, install/uninstall options, etc.). This should fix bug 331654 (now that I understand it better) and improve the situation for bug 332789 (which would be the long-term fix when implemented). Related: bug 331654 FIXED-IN:1.16 Original commit: 9fb3fb04 https://invent.kde.org/sdk/kdesrc-build/-/commit/9fb3fb04d170478c922c5d85bb996f8f4b7ed82c M +11 -8doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/bca4e917547a7b1a720b18bd4598a4ac63d5eba4 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 308493] "tag" option doesn't work
https://bugs.kde.org/show_bug.cgi?id=308493 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/c22b7b0653ae10fdb0 |kdesrc-build/-/commit/80364 |4b9991720ac5501c0b79c9 |4c42e94aebbd2c87b075f61e8be ||10f12f4a --- Comment #5 from Michael Pyne --- Git commit 803644c42e94aebbd2c87b075f61e8be10f12f4a by Michael Pyne. Committed on 22/10/2012 at 03:44. Pushed by ashark into branch 'docbook_historied_per_file'. Implement tag and revision options for git modules. This makes the 'tag' option allow for checking out a given git tag for a module (which is enforced by using refs/tags/$foo when passing to git). Additionally, the 'revision' option has been adopted for git modules as well, to allow for checking out any other git "tree-ish" that git supports (i.e. anything supported by 'git checkout $foo'). Both options should leave the source directory in a detached HEAD state, although kdesrc-build will adapt as necessary with each source update. The auto-stashing is still attempted for both types, and a failure to apply an auto-stashed change will cause the module to fail to build. Ensure you're in a clean working directory if you want to avoid issues due to this. The test suite continues to pass (not that it's very extensive). FIXED-IN:1.16 Original commit: c22b7b06 https://invent.kde.org/sdk/kdesrc-build/-/commit/c22b7b0653ae10fdb04b9991720ac5501c0b79c9 M +9-3doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/803644c42e94aebbd2c87b075f61e8be10f12f4a -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 330387] set-module-option-value Mising from Help Message
https://bugs.kde.org/show_bug.cgi?id=330387 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/2f52754d9547aeb283 |kdesrc-build/-/commit/8c4c2 |811fb0021497c972814622 |c96c734a41185716c440ff9634b ||cdf7a5eb --- Comment #2 from Michael Pyne --- Git commit 8c4c2c96c734a41185716c440ff9634bcdf7a5eb by Michael Pyne. Committed on 26/01/2014 at 05:11. Pushed by ashark into branch 'docbook_historied_per_file'. doc: Correct docs for changes to arbitrary global cmdline flags. This was changed in the recent refactor, now the docs should be up-to-date. Original commit: 2f52754d https://invent.kde.org/sdk/kdesrc-build/-/commit/2f52754d9547aeb283811fb0021497c972814622 M +13 -3doc/cmdline/supported-cmdline-params.docbook M +3-2doc/index.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/8c4c2c96c734a41185716c440ff9634bcdf7a5eb -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 303692] kdesrc-build should support custom make command
https://bugs.kde.org/show_bug.cgi?id=303692 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/9d0f34cefd1f7233cc |kdesrc-build/-/commit/382af |90cf3f17b8566a280dd3f8 |a11df63fd8924ea891165540d53 ||27e6d74b --- Comment #5 from Michael Pyne --- Git commit 382afa11df63fd8924ea891165540d5327e6d74b by Michael Pyne. Committed on 28/07/2012 at 23:12. Pushed by ashark into branch 'docbook_historied_per_file'. Add custom build command option. Unfortunately I was not able to split out moving safe_make and run_make_command with the actual implementation of this option after-the-fact. Whoops. This should implement support for a custom build command (and custom install command, assuming the install can be performed by adding 'install' to the command line). Hopefully that will be sufficient for 'ninja' or other buildsystems. Note that this only changes the *build* command, the other parts of the build system still needs to be supported in kdesrc-build (for now, custom build systems probably wouldn't be too hard to implement if necessary). Assuming your proposed command is available in PATH and can be called like make, doing something like this should suffice: module (or module-set) foo custom-build-command ninja end module (or module-set) Original commit: 9d0f34ce https://invent.kde.org/sdk/kdesrc-build/-/commit/9d0f34cefd1f7233cc90cf3f17b8566a280dd3f8 M +18 -0doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/382afa11df63fd8924ea891165540d5327e6d74b -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 315251] Add --stop-before and --stop-after options
https://bugs.kde.org/show_bug.cgi?id=315251 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/f3d30fb5a056f29718 |kdesrc-build/-/commit/b109e |1aa63d66af5f92f45b2635 |05d3eb833c384114cb07856a510 ||99f29416 --- Comment #2 from Michael Pyne --- Git commit b109e05d3eb833c384114cb07856a51099f29416 by Michael Pyne. Committed on 20/02/2013 at 04:23. Pushed by ashark into branch 'docbook_historied_per_file'. Add --stop-{before,after} command line options. Should hopefully be pretty much what it says on the tin. Stop either right before or right after the given module. (GUI flag since this involves documentation changes to the Handbook and man page). FIXED-IN:1.16 GUI: Original commit: f3d30fb5 https://invent.kde.org/sdk/kdesrc-build/-/commit/f3d30fb5a056f297181aa63d66af5f92f45b2635 M +30 -0doc/cmdline/supported-cmdline-params.docbook M +4-3doc/index.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/b109e05d3eb833c384114cb07856a51099f29416 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 309713] “At the lowest level comes Nokia's &Qt; library,”
https://bugs.kde.org/show_bug.cgi?id=309713 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/8c038f9bcb66e3e8ad |kdesrc-build/-/commit/aed03 |cc11bc587f6258bc66d2b6 |3ed56a4347b65a8c66f18742cc1 ||7abadd4e --- Comment #2 from Michael Pyne --- Git commit aed033ed56a4347b65a8c66f18742cc17abadd4e by Michael Pyne. Committed on 10/11/2012 at 03:04. Pushed by ashark into branch 'docbook_historied_per_file'. doc: Digia now maintains Qt, not Nokia. FIXED-IN:1.16 Original commit: 8c038f9b https://invent.kde.org/sdk/kdesrc-build/-/commit/8c038f9bcb66e3e8adcc11bc587f6258bc66d2b6 M +0-26 doc/cmdline/supported-cmdline-params.docbook M +1-1doc/getting-started/kde-modules-and-selection.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/aed033ed56a4347b65a8c66f18742cc17abadd4e -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 331941] Re-add --resume
https://bugs.kde.org/show_bug.cgi?id=331941 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/7222909901afcf1a2f |kdesrc-build/-/commit/09b14 |1f395714e6ed0ac46020f6 |a79b4c3223731b3b7f040da6179 ||5b5ef9bf --- Comment #2 from Michael Pyne --- Git commit 09b14a79b4c3223731b3b7f040da61795b5ef9bf by Michael Pyne. Committed on 16/03/2014 at 03:15. Pushed by ashark into branch 'docbook_historied_per_file'. Add --resume (and --stop-on-failure) command line options. Now you don't have to do --stop-on-failure=1. --resume is documented in the man page and docbook, but basically lets you resume the last build list quickly without source or metadata updates, starting from the module that failed. If you want to skip the failed module, look into --resume-after and --stop-before. FIXED-IN:1.16 Original commit: 72229099 https://invent.kde.org/sdk/kdesrc-build/-/commit/7222909901afcf1a2f1f395714e6ed0ac46020f6 M +40 -0doc/cmdline/supported-cmdline-params.docbook M +8-0doc/using-kdesrc-build/advanced-features.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/09b14a79b4c3223731b3b7f040da61795b5ef9bf -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 331939] Add --no-metadata option (or similar)
https://bugs.kde.org/show_bug.cgi?id=331939 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/900a08a7d8421452be |kdesrc-build/-/commit/f8204 |08d0e0967d46975541cb21 |c946cd23fd2a13afb5c09926af3 ||56f0d17f --- Comment #2 from Michael Pyne --- Git commit f8204c946cd23fd2a13afb5c09926af356f0d17f by Michael Pyne. Committed on 15/03/2014 at 22:51. Pushed by ashark into branch 'docbook_historied_per_file'. kde-projects: Add --no-metadata command line option. FIXED-IN:1.16 Original commit: 900a08a7 https://invent.kde.org/sdk/kdesrc-build/-/commit/900a08a7d8421452be08d0e0967d46975541cb21 M +12 -0doc/cmdline/supported-cmdline-params.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/f8204c946cd23fd2a13afb5c09926af356f0d17f -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 321667] Flaw in direct-repo-child filter prevents other valid repos from being recursively added
https://bugs.kde.org/show_bug.cgi?id=321667 Michael Pyne changed: What|Removed |Added Latest Commit|https://invent.kde.org/sdk/ |https://invent.kde.org/sdk/ |kdesrc-build/-/commit/26461 |kdesrc-build/-/commit/a825e |b97d3f53c09024bc602ddaca3c9 |c00f754dae8edc158f90405fa9b |04a3b043|61ad611e --- Comment #28 from Michael Pyne --- Git commit a825ec00f754dae8edc158f90405fa9b61ad611e by Michael Pyne. Committed on 11/01/2014 at 21:06. Pushed by ashark into branch 'docbook_historied_per_file'. rc-file: Add specific "option override" entry type. Add a specific config file grouping (which acts just like a module declaration), to allow for specifying options to override a previously-declared module. The use case here is for a module-set: You can specify options which apply to an entire module-set when declaring the module-set, and then override those options with any changes in a later "options" declaration. These declarations can stack too, so this can also be useful for multi-level file includes (but this is less useful since an "options" declaration requires a specific module, it doesn't work on module-sets; in that case you'd want to have the different module-sets in your most-specific included config files instead of in a base file). Tested on my personal test case for bug 321883, and on a --pretend run, and with a bug 321883 test case modified to not pre-declare the overridden module first. Example: module-set kde-mm repository kde-projects use-modules kde/kdemultimedia end module-set options kmix # Not mentioned before this line branch KDE/4.11 end options In this case kmix would use KDE/4.11 branch while juk (and the rest of kde-mm) would use whatever the global branch or branch-group was (probably 'master'). Original commit: 38ea9975 https://invent.kde.org/sdk/kdesrc-build/-/commit/38ea997502e18313e6fe0f7120dee9ed8544bd82 M +67 -1doc/kdesrc-buildrc/kdesrc-buildrc-overview.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/a825ec00f754dae8edc158f90405fa9b61ad611e -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 321667] Flaw in direct-repo-child filter prevents other valid repos from being recursively added
https://bugs.kde.org/show_bug.cgi?id=321667 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/38ea997502e18313e6 |kdesrc-build/-/commit/26461 |fe0f7120dee9ed8544bd82 |b97d3f53c09024bc602ddaca3c9 ||04a3b043 --- Comment #27 from Michael Pyne --- Git commit 26461b97d3f53c09024bc602ddaca3c904a3b043 by Michael Pyne. Committed on 12/07/2013 at 03:41. Pushed by ashark into branch 'docbook_historied_per_file'. kde-projects: Don't auto-filter git-repo's direct children. Kind of a mouthful, but remove a filter that was added to avoid automatically including an actual git repository's children that were also git repositories. E.g. think of something like a SuperBuild git module that also had its component git modules as logical children within projects.kde.org. This filter is unnecessary now that kdesrc-build supports both kde.org metadata (kde-build-metadata/build-script-ignore) and user-configurable module filtering (ignore-modules option). More importantly this filter precludes many other desirable types of group syntax (e.g. including all of kdebindings just by asking for kdebindings). So we remove the filter. The trouble I've seen so far is that it is now rather difficult to build *only* kdelibs, since kde/kdelibs is a logical parent of nepomuk-core, nepomuk-widgets, and kactivities. This can be worked-around by using a normal single module declaration. E.g. module kdelibs repository kde:kdelibs end module On the other hand, the recently-added config file option "ignore-modules" should now remove any kde-projects modules that contain a path component(s) matching the ignored module. You may have to be specific with a given ignore atom for this reason (e.g. ignore playground/libs, not 'libs' otherwise you'll also ignore kdegraphics/libs at this point... though that was also true before this commit). Original commit: a0338664 https://invent.kde.org/sdk/kdesrc-build/-/commit/a033866429b62c29fd93aea843c2f21427634ec8 M +12 -0doc/getting-started/kde-modules-and-selection.docbook M +12 -0doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/26461b97d3f53c09024bc602ddaca3c904a3b043 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 308493] "tag" option doesn't work
https://bugs.kde.org/show_bug.cgi?id=308493 --- Comment #4 from Michael Pyne --- Git commit 85d31758599fef91532a3a84f50abafcbb67fe1d by Michael Pyne. Committed on 21/10/2012 at 16:40. Pushed by ashark into branch 'docbook_historied_per_file'. Use 'tag' option when cloning git modules. This partially implements bug 308493. To fully implement it will be required to handle the case of an already-checked-out module (though prepatory work is already complete). Original commit: e3c41daf https://invent.kde.org/sdk/kdesrc-build/-/commit/e3c41dafab0796cbf441c5dd52306db8e614dbf1 M +3-0doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/85d31758599fef91532a3a84f50abafcbb67fe1d -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 305409] wish: add option to default to http for kde-project repositories
https://bugs.kde.org/show_bug.cgi?id=305409 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/08372a530b584e86ef |kdesrc-build/-/commit/61d2e |5757398e255a646741b37e |601a8a2d587af5ae9fa8a5a326c ||55e7ea26 --- Comment #2 from Michael Pyne --- Git commit 61d2e601a8a2d587af5ae9fa8a5a326c55e7ea26 by Michael Pyne. Committed on 19/08/2012 at 23:19. Pushed by ashark into branch 'docbook_historied_per_file'. Allow using HTTP for kde-projects modules. This adds an option git-desired-protocol to allow for using HTTP instead of git protocol when updating git modules. For non kde-projects modules, it should already be possible to use an http:// URL as a repository instead of git:// (though I haven't tested this yet). This should implement wish 305409. FIXED-IN:1.16 Original commit: 08372a53 https://invent.kde.org/sdk/kdesrc-build/-/commit/08372a530b584e86ef5757398e255a646741b37e M +28 -0doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/61d2e601a8a2d587af5ae9fa8a5a326c55e7ea26 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 305411] wish: Use http_proxy environment variable.
https://bugs.kde.org/show_bug.cgi?id=305411 Michael Pyne changed: What|Removed |Added Latest Commit|http://commits.kde.org/kdes |https://invent.kde.org/sdk/ |rc-build/da098bebfd5da903f4 |kdesrc-build/-/commit/808c8 |9ead490a5e1178df5f6954 |36b4a3d051c42415f9c2c3d6382 ||59fc0a21 --- Comment #2 from Michael Pyne --- Git commit 808c836b4a3d051c42415f9c2c3d638259fc0a21 by Michael Pyne. Committed on 19/08/2012 at 21:56. Pushed by ashark into branch 'docbook_historied_per_file'. Allow setting http-proxy settings. Right now this will be utilized for anything needing libwww-perl (kde_projects.xml download, tarball snapshots), and will be propagated to subprocesses via http_proxy environment variable, unless that variable is already set. Tested by using a simple HTTP proxy utilizing HTTP::Proxy from CPAN. Implements wish 305411. FIXED-IN:1.16 Original commit: da098beb https://invent.kde.org/sdk/kdesrc-build/-/commit/da098bebfd5da903f49ead490a5e1178df5f6954 M +18 -0doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/808c836b4a3d051c42415f9c2c3d638259fc0a21 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 297847] kdesrc-build changes the order of modules without reason, breaking compilation
https://bugs.kde.org/show_bug.cgi?id=297847 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/-/commit/79685 ||7861a789a84876489a7ae6b1ee1 ||44f8fa9b --- Comment #3 from Michael Pyne --- Git commit 796857861a789a84876489a7ae6b1ee144f8fa9b by Michael Pyne. Committed on 14/04/2012 at 01:17. Pushed by ashark into branch 'docbook_historied_per_file'. Revert "Bracket modules into sorted tiers." This reverts commit 089edc6a73646bce540d39c784fd2991cdefd1d2. Although auto-grouping is a nice easy way to get approximately right dependency ordering, it doesn't work in all situations. The "Best" thing would be to only sort within kde-project modules (as expanded) but for now just back out this commit so I don't break customized rc-files. Original commit: e1a2c285 https://invent.kde.org/sdk/kdesrc-build/-/commit/e1a2c2850336285b90bc0e1cebaa5fe2b1ce0770 M +0-21 doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/796857861a789a84876489a7ae6b1ee144f8fa9b -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 297357] Review "use-stable-kde" option to build current stable release
https://bugs.kde.org/show_bug.cgi?id=297357 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/-/commit/01fad ||9c26c729699862701cbd7d06b58 ||05af8e19 --- Comment #2 from Michael Pyne --- Git commit 01fad9c26c729699862701cbd7d06b5805af8e19 by Michael Pyne, on behalf of Valery Yundin. Committed on 05/04/2012 at 02:19. Pushed by ashark into branch 'docbook_historied_per_file'. Support defaulting to stable branches from proj.k.o. Valery Yundin has kindly submitted a patch (including documentation!) resurrecting the old "use-stable-kde" option to allow for defaulting kde-projects modules to the stable branch (if defined for that module). This should track with changes to the stable branch as defined, so it would be a good way to track the "not quite bleeding-edge" of development. A hearty thanks to Valery for making up a patch and then adapting it to make use of the old use-stable-kde option. FIXED-IN:1.15 Original commit: 84447b1d https://invent.kde.org/sdk/kdesrc-build/-/commit/84447b1deffea6b3c4740d987b064b75e3b5d185 M +13 -0doc/getting-started/configure-data.docbook M +18 -5doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/01fad9c26c729699862701cbd7d06b5805af8e19 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 223334] Add an option to configure the path of '.kdesvn-build-data'
https://bugs.kde.org/show_bug.cgi?id=223334 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/-/commit/20359 ||d040c92549c6c143272340bc53f ||f0bf0b87 --- Comment #2 from Michael Pyne --- Git commit 20359d040c92549c6c143272340bc53ff0bf0b87 by Michael Pyne. Committed on 05/03/2012 at 03:34. Pushed by ashark into branch 'docbook_historied_per_file'. Allow persistent data location to be user-defined. For all my users with multiple rc files in the same directory. ;) FEATURE: FIXED-IN:1.15 Original commit: 814ad429 https://invent.kde.org/sdk/kdesrc-build/-/commit/814ad429e4b1a2d6c9b174fcff2586a1e5a50931 M +1-1doc/index.docbook M +14 -0doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/20359d040c92549c6c143272340bc53ff0bf0b87 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 284071] [Patch] Add option to set ssh identity file
https://bugs.kde.org/show_bug.cgi?id=284071 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/-/commit/af67d ||864606f1a8920c49a6a42daf19c ||c816cbd5 --- Comment #4 from Michael Pyne --- Git commit af67d864606f1a8920c49a6a42daf19cc816cbd5 by Michael Pyne. Committed on 20/10/2011 at 00:11. Pushed by ashark into branch 'docbook_historied_per_file'. Allow SSH identity to be specified. Now you can specify the private key to be passed to your SSH agent if you forgot to run that before running kdesrc-build. Proof-of-concept patch graciously provided by Ralf Jung. FIXED-IN:1.14.2 Original commit: 2430b0a3 https://invent.kde.org/sdk/kdesrc-build/-/commit/2430b0a308bc63c5d008ba602f0bcf345d35166f M +13 -0doc/kdesrc-buildrc/conf-options-table.docbook https://invent.kde.org/sdk/kdesrc-build/-/commit/af67d864606f1a8920c49a6a42daf19cc816cbd5 -- You are receiving this mail because: You are watching all bug changes.
[KDE Config Driver Manager] [Bug 402901] kdesrc-build --initial-setup reports "* No installer for linux!" when executing the command on Debian 9 stable
https://bugs.kde.org/show_bug.cgi?id=402901 --- Comment #8 from Michael Pyne --- Git commit a5d286145e9b9374b5e9c6ec6045de1e58f812e9 by Michael Pyne. Committed on 06/01/2019 at 02:39. Pushed by ashark into branch 'no-ff-recreate'. first-run: Add basic installer and packages for Debian distros. This adds the installer to allow Debian-based distros to run and a basic set of packages needed to at least allow kdesrc-build to successfully run --initial-setup, as tested under a Docker image of debian 9. This will still miss the dependencies required to fully make it through any kind of reasonable KDE software build. In particular even if I included the list of Qt modules that Debian 9 stable has available, they are too old for current git-based applications. But other modules are probably needed and still missing (suggestions accepted! :). So there's still more to do here, which is why the bug remains open. See issue #15 Original commit: 845e9da4 https://invent.kde.org/sdk/kdesrc-build/-/commit/845e9da4ac262e14161bb777449d85972cfe6103 M +3-1distro-dependencies/debian.ini https://invent.kde.org/sysadmin/repo-metadata/-/commit/a5d286145e9b9374b5e9c6ec6045de1e58f812e9 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 402484] kdesrc-build's initial setup needs to install perl-io-socket-ssl on Arch et al.
https://bugs.kde.org/show_bug.cgi?id=402484 Michael Pyne changed: What|Removed |Added Latest Commit|e84022a7acd6347b34cefd58a5c |https://invent.kde.org/sysa |002e4bf03f440 |dmin/repo-metadata/-/commit ||/0ff608a2cf7b58dfa7fef9e247 ||4bbb8d873065fa --- Comment #3 from Michael Pyne --- Git commit 0ff608a2cf7b58dfa7fef9e2474bbb8d873065fa by Michael Pyne, on behalf of Nate Graham. Committed on 26/12/2018 at 01:28. Pushed by ashark into branch 'no-ff-recreate'. Add Arch package for IO::Socket::SSL as a kdesrc-build dep. Fixes #3 Differential Revision: https://phabricator.kde.org/D17768 FIXED-IN:19.01 Original commit: e84022a7 https://invent.kde.org/sdk/kdesrc-build/-/commit/e84022a7acd6347b34cefd58a5c002e4bf03f440 M +1-1distro-dependencies/arch.ini https://invent.kde.org/sysadmin/repo-metadata/-/commit/0ff608a2cf7b58dfa7fef9e2474bbb8d873065fa -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 473859] Add short options for --include-dependencies and --no-include-dependencies
https://bugs.kde.org/show_bug.cgi?id=473859 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/-/commit/5b155 ||c5e29dd2d377fd1f84fb42d0e62 ||9a9f368b Resolution|--- |FIXED Status|CONFIRMED |RESOLVED --- Comment #2 from Michael Pyne --- Git commit 5b155c5e29dd2d377fd1f84fb42d0e629a9f368b by Michael Pyne. Committed on 07/09/2023 at 02:34. Pushed by mpyne into branch 'master'. Add short options for --[no-]include-dependencies. This adds and documents -d as a short alias for --include-dependencies, and a corresponding -D as its negation. While I'm at it, and at user request, this also adds short options for: * --help * --no-src * --src-only * --refresh-build * --ignore-modules * --resume-from and --resume-after M +11 -11 doc/index.docbook M +28 -10 doc/man-kdesrc-build.1.docbook M +37 -21 modules/ksb/Cmdline.pm https://invent.kde.org/sdk/kdesrc-build/-/commit/5b155c5e29dd2d377fd1f84fb42d0e629a9f368b -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 456593] plasma-wayland-protocol builds, but doesn't install properly
https://bugs.kde.org/show_bug.cgi?id=456593 --- Comment #7 from Michael Pyne --- Git commit 7e1a75e66d5b6ac4fc0db3e828d13728cd04f83c by Michael Pyne. Committed on 14/07/2022 at 22:47. Pushed by mpyne into branch 'master'. buildsystem: Be more restrictive on when to skip install. This fixes a bug where the documentation of how to force kdesrc-build to proceed with the install "--refresh-build this module to force install" did not match kdesrc-build's behavior. But in addition this adds a check that the module was ever successfully installed by checking for a matching persistent option. I suspect this might be part of the problem experienced by a user in bug 456593. M +6-3modules/ksb/Module.pm https://invent.kde.org/sdk/kdesrc-build/commit/7e1a75e66d5b6ac4fc0db3e828d13728cd04f83c -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kjobwidgets] [Bug 450325] Ark crashes in KUiServerV2JobTracker::registerJob() when turning on second monitor
https://bugs.kde.org/show_bug.cgi?id=450325 Michael Pyne changed: What|Removed |Added Latest Commit|https://invent.kde.org/fram |https://invent.kde.org/fram |eworks/kjobwidgets/commit/b |eworks/kjobwidgets/commit/5 |8752085d2a480dfc93d2d422c46 |aeba3f01ef8cdf723813cacdd29 |3a59d46af5ee|945328288663 Status|ASSIGNED|RESOLVED Resolution|--- |FIXED --- Comment #27 from Michael Pyne --- Git commit 5aeba3f01ef8cdf723813cacdd29945328288663 by Michael Pyne. Committed on 09/07/2022 at 18:42. Pushed by mpyne into branch 'master'. ui-server: Fix crash by only re-registering live KJobs. This addresses a frequently-reported crash in the job tracker for KUiServerV2 that occurs when attempting to re-register new job views for active KJobs after a new UI server comes online. Although I have not been able to reproduce the crash myself, (by attempting to use both long-lived and short-lived file transfers from Dolphin and restarting plasmashell), inspection of the code shows that it is possible for there to be deleted KJobs pointing to JobView objects during some portions of the job tracker's lifetime. The current code deals with this in situations including DBus calls to create a U/I view for a KJob (the KJob may terminate before the DBus reply is received) and even a short delay that can be optionally introduced (the KJob may terminate before the delay elapses). A QPointer is used as a guard in these situations, but there is no similar guard for the re-registration code. In this case we cannot use QPointer to guard the job's lifetime because the KJob must be alive when the QPointer is created, and this crash occurs when the KJob is terminated. However the KJob's destruction should lead to the unregisterJob() function being called, which handles removing the terminated KJob from the map of job views with only one exception, where instead the job view for the KJob has its "terminated" pending status set. So the fix here checks for the "terminated" state in the same way as performed in requestView(), and if the KJob is terminated, handles requesting the job view to terminate the U/I and finally removing the terminated KJob from the map of job views. By doing this, we avoid passing a deleted KJob to the registerJob() function, which will attempt to dereference it and crash the application. See also merge request !22 M +16 -4src/kuiserverv2jobtracker.cpp https://invent.kde.org/frameworks/kjobwidgets/commit/5aeba3f01ef8cdf723813cacdd29945328288663 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 436739] kdesrc-build fails to update packages that don't have a "master" branch
https://bugs.kde.org/show_bug.cgi?id=436739 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/commit/f1d429d ||147f8ec0240e6ac0f6ab2eec797 ||a6c4dd Status|CONFIRMED |RESOLVED Resolution|--- |FIXED --- Comment #1 from Michael Pyne --- Git commit f1d429d147f8ec0240e6ac0f6ab2eec797a6c4dd by Michael Pyne. Committed on 08/07/2022 at 00:52. Pushed by mpyne into branch 'master'. git: Don't fail to clone due to branch name if none required. There is a wider variety of potential default branch names from a git repository that we might want to clone. Many use now 'main' but not all of them. We also have modules that use something else entirely, like 'dev' (as reported in bug 436739). Most of the time the user doesn't actually care about the branch, they just want whatever the mainline trunk is. To handle this we had previously defaulted to assuming the branch was 'master' so we could optimize the amount of data transferred during initial clone. Now this is likely to lead to an error. Instead, if the user hasn't specified otherwise, just perform the git clone as normal. To do this right there's a wee bit of other work elsewhere to detect which branch head we actually got from the remote git repository but that's not difficult to do either. M +21 -6modules/ksb/Updater/Git.pm https://invent.kde.org/sdk/kdesrc-build/commit/f1d429d147f8ec0240e6ac0f6ab2eec797a6c4dd -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kjobwidgets] [Bug 450325] Ark crashes in KUiServerV2JobTracker::registerJob() when turning on second monitor
https://bugs.kde.org/show_bug.cgi?id=450325 Michael Pyne changed: What|Removed |Added Ever confirmed|0 |1 Resolution|FIXED |--- Status|RESOLVED|REOPENED --- Comment #23 from Michael Pyne --- I thought that a MR needed to be merged to master to have its BUG: close a bug, reopening until a fix is merged. -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kjobwidgets] [Bug 450325] Ark crashes in KUiServerV2JobTracker::registerJob() when turning on second monitor
https://bugs.kde.org/show_bug.cgi?id=450325 Michael Pyne changed: What|Removed |Added Resolution|--- |FIXED Latest Commit||https://invent.kde.org/fram ||eworks/kjobwidgets/commit/b ||8752085d2a480dfc93d2d422c46 ||3a59d46af5ee Status|REPORTED|RESOLVED --- Comment #22 from Michael Pyne --- Git commit b8752085d2a480dfc93d2d422c463a59d46af5ee by Michael Pyne. Committed on 02/07/2022 at 22:13. Pushed by mpyne into branch 'work-bug-450325-fix-crash'. ui-server: Fix crash by only re-registering live KJobs. This addresses a frequently-reported crash in the job tracker for KUiServerV2 that occurs when attempting to re-register new job views for active KJobs after a new UI server comes online. Although I have not been able to reproduce the crash myself, (by attempting to use both long-lived and short-lived file transfers from Dolphin and restarting plasmashell), inspection of the code shows that it is possible for there to be deleted KJobs pointing to JobView objects during some portions of the job tracker's lifetime. The current code deals with this in situations including DBus calls to create a U/I view for a KJob (the KJob may terminate before the DBus reply is received) and even a short delay that can be optionally introduced (the KJob may terminate before the delay elapses). A QPointer is used as a guard in these situations, but there is no similar guard for the re-registration code. In this case we cannot use QPointer to guard the job's lifetime because the KJob must be alive when the QPointer is created, and this crash occurs when the KJob is terminated. However the KJob's destruction should lead to the unregisterJob() function being called, which handles removing the terminated KJob from the map of job views with only one exception, where instead the job view for the KJob has its "terminated" pending status set. So the fix here checks for the "terminated" state in the same way as performed in requestView(), and if the KJob is terminated, handles requesting the job view to terminate the U/I and finally removing the terminated KJob from the map of job views. By doing this, we avoid passing a deleted KJob to the registerJob() function, which will attempt to dereference it and crash the application. M +16 -4src/kuiserverv2jobtracker.cpp https://invent.kde.org/frameworks/kjobwidgets/commit/b8752085d2a480dfc93d2d422c463a59d46af5ee -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kjobwidgets] [Bug 450325] Ark crashes in KUiServerV2JobTracker::registerJob() when turning on second monitor
https://bugs.kde.org/show_bug.cgi?id=450325 --- Comment #21 from Michael Pyne --- (In reply to Christoph Cullmann from comment #20) > Sounds like a reasonable explanation. well, maybe. I did find the code meant to update the jobView structure when the KJob is finished, but it is a) called indirectly (from KUiServerV2JobTracker::unregisterJob), and b) has a weird special case to handle terminated KJobs that don't have an existing view (presumably because the signal that would create it being later in the event loop): // Remember that the job finished in the meantime and // terminate the JobView once it arrives d->scheduleUpdate(job, QStringLiteral("terminated"), true); if (job->error()) { d->scheduleUpdate(job, QStringLiteral("errorCode"), static_cast(job->error())); d->scheduleUpdate(job, QStringLiteral("errorMessage"), job->errorText()); } But scheduleUpdate() simply recreates the jobView mapping to the soon-to-be dead KJob and if the re-registration happens before the "terminate the JobView once it arrives" occurs we're in for a bad time, as the re-registration code assumes all entries in the map of jobs to jobViews have valid KJobs. I'll take a look at how best to handle the weird lifetimes here and try to open the MR tonight. -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kjobwidgets] [Bug 450325] Ark crashes in KUiServerV2JobTracker::registerJob() when turning on second monitor
https://bugs.kde.org/show_bug.cgi?id=450325 Michael Pyne changed: What|Removed |Added CC||mp...@kde.org --- Comment #19 from Michael Pyne --- I suspect the issue is in using a deleted KJob when re-registering a job view (kuiserverv2jobtracker.cpp:197) // Watch the server registering/unregistering and re-register the jobs as needed if (!d->serverRegisteredConnection) { d->serverRegisteredConnection = connect(serverProxy(), &KSharedUiServerV2Proxy::serverRegistered, this, [this]() { const auto staleViews = d->jobViews; // elided for (auto it = staleViews.begin(), end = staleViews.end(); it != end; ++it) { KJob *job = it.key(); const JobView &view = it.value(); // elided registerJob(job); In the section of code right after this in registerJob, the code is careful to ensure that a QPointer is used in order to verify that the KJob being registered won't have been deleted in the functor to be added to the event loop by QTimer (kuiserverv2jobtracker.cpp:248): QPointer jobGuard = job; QTimer *delayTimer = new QTimer(); delayTimer->setSingleShot(true); connect(delayTimer, &QTimer::timeout, this, [this, job, jobGuard, desktopEntry] { // elided }); This part of the code also references the d->jobViews the re-registration code uses earlier, so if it were possible to assume that a KJob* held in d->jobViews was enough to guarantee the KJob were still valid, the later delay timer could have done the same thing. It would take more code to do this so that's not airtight evidence, but I can't find anywhere else that guarantees a KJob ending also removes its JobView (the re-registration code as much as calls it a "stale" job view...) so I suspect this is the cause of this crash. I'll put together a patch that uses the same QPointer jobGuard trick and open an MR. -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 289754] kdesrc-build should flag module as unbuilt if last build for that module was interrupted
https://bugs.kde.org/show_bug.cgi?id=289754 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/commit/8f80fdf ||17cb5c49b9993f5a0bf395402da ||ab5d8e Resolution|--- |FIXED Status|CONFIRMED |RESOLVED --- Comment #2 from Michael Pyne --- Git commit 8f80fdf17cb5c49b9993f5a0bf395402daab5d8e by Michael Pyne. Committed on 24/06/2022 at 02:40. Pushed by mpyne into branch 'master'. build: Ensure a rebuild following an interrupted build is attempted. Previously, interrupting a module build during the build was not treated as a build failure, so the module's persistent options were not changed. In the normal scenario this means the module's last build was remembered as a success. Then by default, when you'd go to rebuild the module right after, the build would be skipped because the source didn't change. M +10 -3modules/ksb/Application.pm https://invent.kde.org/sdk/kdesrc-build/commit/8f80fdf17cb5c49b9993f5a0bf395402daab5d8e -- You are receiving this mail because: You are watching all bug changes.
[kmail2] [Bug 431921] Black square appears sometimes in KMail
https://bugs.kde.org/show_bug.cgi?id=431921 --- Comment #7 from Michael Pyne --- If it occurs in multiple programs it is likely a style bug (presumably breeze). If you're up to it, you could try running kmail as "kmail -style fusion" to try a different theme (Qt has a "Fusion" theme built-in) and see if the black square still comes up at random intervals. Note that KMail will look a bit different if you run it this way since the style is different. -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 317666] juk startup is slow if using tree mode
https://bugs.kde.org/show_bug.cgi?id=317666 --- Comment #8 from Michael Pyne --- Git commit a65a4e8a037bae5d2731267b60ed61bf09526413 by Michael Pyne. Committed on 26/03/2021 at 01:52. Pushed by mpyne into branch 'release/21.04'. tag_scan: Fix painful rescan of music metadata on startup. For the longest time, JuK has suffered from a problem where its intended behavior to load music metadata from a cached database, instead of re-scanning every individual track on startup, was not working right. There has been debugging lines in JuK since all the way back to 2013 trying to trace what area of startup sequence was taking up all the time, but to no avail in helping me fix the bug. The Problem === Recently I took a different approach, of adding a debug-only crash whenever we would load a music track tag the "slow" way, and long story short there were two bugs that each would cause slowdown: 1. Playlists aside from the CollectionList would cause every music track in that playlist to be re-scanned. What this means is that every though the music in the CollectionList would be loaded quickly, if you had that same music track in a separate Playlist, that music track would reload the same tags from disk rather than copying from the existing CollectionList item. This was especially bad for users of the old "tree mode" view, since every individual artist *and* album were rendered as individual playlists, which would therefore each re-scan the music over and over again. 2. JuK supports a "folder scan" feature, and in fact really wants the user to have at least one folder assigned. Any music identified in this folder is added to the CollectionList automatically on startup and, you guessed it, causes the music track information to be loaded from disk, even if the music was already in the CollectionList! :( The net effect is that most music would be re-scanned on startup unless you were a user who used CollectionList exclusively, and had most of your music not visible to the folder scanner. The Solution Due to how painful this problem has been, I had ended up adding a threaded solution for the folder scan process. This didn't help make things any faster but at least the GUI wasn't frozen. But now that the threading code is present I judged it would be easier and safer to make the central object holding track metadata (CollectionList's m_itemsDict) available in thread-safe fashion. This then permitted me to check for whether a track has already been loaded when performing folder scan, and to check whether a track has already been loaded when creating a new (non-CollectionList) Playlist. In either event if the track already exists, then we copy the FileHandle rather than create a new one. The combination speeds up loading significantly, taking anywhere from 60% to 70% off of the total time to load on my system, with mostly a CollectionList under folder scan and few additional playlists. In this configuration I go from about 5.4 seconds to 1.5 seconds with cold caches. The difference should be even more stark on systems where disk I/O is expensive, or where there are a great number of tracks in playlists outside of the CollectionList. I consider this a bugfix (and there are even multiple bug reports) so I will backport shortly. CHANGELOG:Reduce startup time by 60-70% or more. Related: bug 428772 FIXED-IN:21.04 (cherry picked from commit d6b28a9b4c8e21a0b9ccd5bb7585091e501d94ab) M +58 -25 collectionlist.cpp M +5-3collectionlist.h M +11 -3directoryloader.cpp M +6-1filehandle.cpp M +10 -1playlist.cpp https://invent.kde.org/multimedia/juk/commit/a65a4e8a037bae5d2731267b60ed61bf09526413 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 428772] JuK painful folder scanning at the EVERY (Re)start
https://bugs.kde.org/show_bug.cgi?id=428772 --- Comment #2 from Michael Pyne --- Git commit a65a4e8a037bae5d2731267b60ed61bf09526413 by Michael Pyne. Committed on 26/03/2021 at 01:52. Pushed by mpyne into branch 'release/21.04'. tag_scan: Fix painful rescan of music metadata on startup. For the longest time, JuK has suffered from a problem where its intended behavior to load music metadata from a cached database, instead of re-scanning every individual track on startup, was not working right. There has been debugging lines in JuK since all the way back to 2013 trying to trace what area of startup sequence was taking up all the time, but to no avail in helping me fix the bug. The Problem === Recently I took a different approach, of adding a debug-only crash whenever we would load a music track tag the "slow" way, and long story short there were two bugs that each would cause slowdown: 1. Playlists aside from the CollectionList would cause every music track in that playlist to be re-scanned. What this means is that every though the music in the CollectionList would be loaded quickly, if you had that same music track in a separate Playlist, that music track would reload the same tags from disk rather than copying from the existing CollectionList item. This was especially bad for users of the old "tree mode" view, since every individual artist *and* album were rendered as individual playlists, which would therefore each re-scan the music over and over again. 2. JuK supports a "folder scan" feature, and in fact really wants the user to have at least one folder assigned. Any music identified in this folder is added to the CollectionList automatically on startup and, you guessed it, causes the music track information to be loaded from disk, even if the music was already in the CollectionList! :( The net effect is that most music would be re-scanned on startup unless you were a user who used CollectionList exclusively, and had most of your music not visible to the folder scanner. The Solution Due to how painful this problem has been, I had ended up adding a threaded solution for the folder scan process. This didn't help make things any faster but at least the GUI wasn't frozen. But now that the threading code is present I judged it would be easier and safer to make the central object holding track metadata (CollectionList's m_itemsDict) available in thread-safe fashion. This then permitted me to check for whether a track has already been loaded when performing folder scan, and to check whether a track has already been loaded when creating a new (non-CollectionList) Playlist. In either event if the track already exists, then we copy the FileHandle rather than create a new one. The combination speeds up loading significantly, taking anywhere from 60% to 70% off of the total time to load on my system, with mostly a CollectionList under folder scan and few additional playlists. In this configuration I go from about 5.4 seconds to 1.5 seconds with cold caches. The difference should be even more stark on systems where disk I/O is expensive, or where there are a great number of tracks in playlists outside of the CollectionList. I consider this a bugfix (and there are even multiple bug reports) so I will backport shortly. CHANGELOG:Reduce startup time by 60-70% or more. Related: bug 317666 FIXED-IN:21.04 (cherry picked from commit d6b28a9b4c8e21a0b9ccd5bb7585091e501d94ab) M +58 -25 collectionlist.cpp M +5-3collectionlist.h M +11 -3directoryloader.cpp M +6-1filehandle.cpp M +10 -1playlist.cpp https://invent.kde.org/multimedia/juk/commit/a65a4e8a037bae5d2731267b60ed61bf09526413 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 428772] JuK painful folder scanning at the EVERY (Re)start
https://bugs.kde.org/show_bug.cgi?id=428772 Michael Pyne changed: What|Removed |Added Resolution|--- |FIXED Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/d6b28a9b4 ||c8e21a0b9ccd5bb7585091e501d ||94ab Version Fixed In||21.04 Status|REPORTED|RESOLVED --- Comment #1 from Michael Pyne --- Git commit d6b28a9b4c8e21a0b9ccd5bb7585091e501d94ab by Michael Pyne. Committed on 26/03/2021 at 01:48. Pushed by mpyne into branch 'master'. tag_scan: Fix painful rescan of music metadata on startup. For the longest time, JuK has suffered from a problem where its intended behavior to load music metadata from a cached database, instead of re-scanning every individual track on startup, was not working right. There has been debugging lines in JuK since all the way back to 2013 trying to trace what area of startup sequence was taking up all the time, but to no avail in helping me fix the bug. The Problem === Recently I took a different approach, of adding a debug-only crash whenever we would load a music track tag the "slow" way, and long story short there were two bugs that each would cause slowdown: 1. Playlists aside from the CollectionList would cause every music track in that playlist to be re-scanned. What this means is that every though the music in the CollectionList would be loaded quickly, if you had that same music track in a separate Playlist, that music track would reload the same tags from disk rather than copying from the existing CollectionList item. This was especially bad for users of the old "tree mode" view, since every individual artist *and* album were rendered as individual playlists, which would therefore each re-scan the music over and over again. 2. JuK supports a "folder scan" feature, and in fact really wants the user to have at least one folder assigned. Any music identified in this folder is added to the CollectionList automatically on startup and, you guessed it, causes the music track information to be loaded from disk, even if the music was already in the CollectionList! :( The net effect is that most music would be re-scanned on startup unless you were a user who used CollectionList exclusively, and had most of your music not visible to the folder scanner. The Solution Due to how painful this problem has been, I had ended up adding a threaded solution for the folder scan process. This didn't help make things any faster but at least the GUI wasn't frozen. But now that the threading code is present I judged it would be easier and safer to make the central object holding track metadata (CollectionList's m_itemsDict) available in thread-safe fashion. This then permitted me to check for whether a track has already been loaded when performing folder scan, and to check whether a track has already been loaded when creating a new (non-CollectionList) Playlist. In either event if the track already exists, then we copy the FileHandle rather than create a new one. The combination speeds up loading significantly, taking anywhere from 60% to 70% off of the total time to load on my system, with mostly a CollectionList under folder scan and few additional playlists. In this configuration I go from about 5.4 seconds to 1.5 seconds with cold caches. The difference should be even more stark on systems where disk I/O is expensive, or where there are a great number of tracks in playlists outside of the CollectionList. I consider this a bugfix (and there are even multiple bug reports) so I will backport shortly. CHANGELOG:Reduce startup time by 60-70% or more. Related: bug 317666 FIXED-IN:21.04 M +58 -25 collectionlist.cpp M +5-3collectionlist.h M +11 -3directoryloader.cpp M +6-1filehandle.cpp M +10 -1playlist.cpp https://invent.kde.org/multimedia/juk/commit/d6b28a9b4c8e21a0b9ccd5bb7585091e501d94ab -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 317666] juk startup is slow if using tree mode
https://bugs.kde.org/show_bug.cgi?id=317666 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/d6b28a9b4 ||c8e21a0b9ccd5bb7585091e501d ||94ab Version Fixed In||21.04 Status|REPORTED|RESOLVED Resolution|--- |FIXED --- Comment #7 from Michael Pyne --- Git commit d6b28a9b4c8e21a0b9ccd5bb7585091e501d94ab by Michael Pyne. Committed on 26/03/2021 at 01:48. Pushed by mpyne into branch 'master'. tag_scan: Fix painful rescan of music metadata on startup. For the longest time, JuK has suffered from a problem where its intended behavior to load music metadata from a cached database, instead of re-scanning every individual track on startup, was not working right. There has been debugging lines in JuK since all the way back to 2013 trying to trace what area of startup sequence was taking up all the time, but to no avail in helping me fix the bug. The Problem === Recently I took a different approach, of adding a debug-only crash whenever we would load a music track tag the "slow" way, and long story short there were two bugs that each would cause slowdown: 1. Playlists aside from the CollectionList would cause every music track in that playlist to be re-scanned. What this means is that every though the music in the CollectionList would be loaded quickly, if you had that same music track in a separate Playlist, that music track would reload the same tags from disk rather than copying from the existing CollectionList item. This was especially bad for users of the old "tree mode" view, since every individual artist *and* album were rendered as individual playlists, which would therefore each re-scan the music over and over again. 2. JuK supports a "folder scan" feature, and in fact really wants the user to have at least one folder assigned. Any music identified in this folder is added to the CollectionList automatically on startup and, you guessed it, causes the music track information to be loaded from disk, even if the music was already in the CollectionList! :( The net effect is that most music would be re-scanned on startup unless you were a user who used CollectionList exclusively, and had most of your music not visible to the folder scanner. The Solution Due to how painful this problem has been, I had ended up adding a threaded solution for the folder scan process. This didn't help make things any faster but at least the GUI wasn't frozen. But now that the threading code is present I judged it would be easier and safer to make the central object holding track metadata (CollectionList's m_itemsDict) available in thread-safe fashion. This then permitted me to check for whether a track has already been loaded when performing folder scan, and to check whether a track has already been loaded when creating a new (non-CollectionList) Playlist. In either event if the track already exists, then we copy the FileHandle rather than create a new one. The combination speeds up loading significantly, taking anywhere from 60% to 70% off of the total time to load on my system, with mostly a CollectionList under folder scan and few additional playlists. In this configuration I go from about 5.4 seconds to 1.5 seconds with cold caches. The difference should be even more stark on systems where disk I/O is expensive, or where there are a great number of tracks in playlists outside of the CollectionList. I consider this a bugfix (and there are even multiple bug reports) so I will backport shortly. CHANGELOG:Reduce startup time by 60-70% or more. Related: bug 428772 FIXED-IN:21.04 M +58 -25 collectionlist.cpp M +5-3collectionlist.h M +11 -3directoryloader.cpp M +6-1filehandle.cpp M +10 -1playlist.cpp https://invent.kde.org/multimedia/juk/commit/d6b28a9b4c8e21a0b9ccd5bb7585091e501d94ab -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 303901] Restart album when using random album
https://bugs.kde.org/show_bug.cgi?id=303901 Michael Pyne changed: What|Removed |Added Latest Commit|https://invent.kde.org/mult |https://invent.kde.org/mult |imedia/juk/commit/6aef3be86 |imedia/juk/commit/b46844f6a |82d829f07ab86901ceff8755fee |ad1359f971ad2c876d7c0728cec |8610|77d5 --- Comment #5 from Michael Pyne --- Git commit b46844f6aad1359f971ad2c876d7c0728cec77d5 by Michael Pyne. Committed on 25/03/2021 at 01:38. Pushed by mpyne into branch 'release/21.04'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 302250, bug 336637, bug 353259, bug 404157, bug 417551 FIXED-IN:21.04 (cherry picked from commit 6aef3be8682d829f07ab86901ceff8755fee8610) M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +1-0juk.cpp M +19 -29 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +21 -22 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/b46844f6aad1359f971ad2c876d7c0728cec77d5 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 404157] Track length is not updated when advancing in the playlist
https://bugs.kde.org/show_bug.cgi?id=404157 Michael Pyne changed: What|Removed |Added Latest Commit|https://invent.kde.org/mult |https://invent.kde.org/mult |imedia/juk/commit/6aef3be86 |imedia/juk/commit/b46844f6a |82d829f07ab86901ceff8755fee |ad1359f971ad2c876d7c0728cec |8610|77d5 --- Comment #3 from Michael Pyne --- Git commit b46844f6aad1359f971ad2c876d7c0728cec77d5 by Michael Pyne. Committed on 25/03/2021 at 01:38. Pushed by mpyne into branch 'release/21.04'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 302250, bug 303901, bug 336637, bug 353259, bug 417551 FIXED-IN:21.04 (cherry picked from commit 6aef3be8682d829f07ab86901ceff8755fee8610) M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +1-0juk.cpp M +19 -29 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +21 -22 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/b46844f6aad1359f971ad2c876d7c0728cec77d5 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 353259] Juk player will not play tracks added to play queue
https://bugs.kde.org/show_bug.cgi?id=353259 Michael Pyne changed: What|Removed |Added Latest Commit|https://invent.kde.org/mult |https://invent.kde.org/mult |imedia/juk/commit/6aef3be86 |imedia/juk/commit/b46844f6a |82d829f07ab86901ceff8755fee |ad1359f971ad2c876d7c0728cec |8610|77d5 --- Comment #3 from Michael Pyne --- Git commit b46844f6aad1359f971ad2c876d7c0728cec77d5 by Michael Pyne. Committed on 25/03/2021 at 01:38. Pushed by mpyne into branch 'release/21.04'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 302250, bug 303901, bug 336637, bug 404157, bug 417551 FIXED-IN:21.04 (cherry picked from commit 6aef3be8682d829f07ab86901ceff8755fee8610) M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +1-0juk.cpp M +19 -29 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +21 -22 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/b46844f6aad1359f971ad2c876d7c0728cec77d5 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 336637] After playing queued items, juk returns to "collection" not previous playlist
https://bugs.kde.org/show_bug.cgi?id=336637 Michael Pyne changed: What|Removed |Added Latest Commit|https://invent.kde.org/mult |https://invent.kde.org/mult |imedia/juk/commit/6aef3be86 |imedia/juk/commit/b46844f6a |82d829f07ab86901ceff8755fee |ad1359f971ad2c876d7c0728cec |8610|77d5 --- Comment #4 from Michael Pyne --- Git commit b46844f6aad1359f971ad2c876d7c0728cec77d5 by Michael Pyne. Committed on 25/03/2021 at 01:38. Pushed by mpyne into branch 'release/21.04'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 302250, bug 303901, bug 353259, bug 404157, bug 417551 FIXED-IN:21.04 (cherry picked from commit 6aef3be8682d829f07ab86901ceff8755fee8610) M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +1-0juk.cpp M +19 -29 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +21 -22 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/b46844f6aad1359f971ad2c876d7c0728cec77d5 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 165899] juk take to much CPU / Power when IDLE
https://bugs.kde.org/show_bug.cgi?id=165899 --- Comment #5 from Michael Pyne --- Git commit b64b37616f0490f60526defaf5335c4731dd41b4 by Michael Pyne. Committed on 25/03/2021 at 01:29. Pushed by mpyne into branch 'release/21.04'. systemtray: Cleanups and modernization. Also a timer bugfix. The bug: When using the track popup announcement, a timer kicks off to tell the popup to fade out. A second timer is used to incrementally fade the popup a bit every few milliseconds until the popup can finally be hidden. However the first timer was never stopped, and would kick off the fadeout sequence again, over and over, only hidden to view since the popup is no longer visible. This might be related to the old bug 165899 (which I tried to fix a different way). I believe this happened in the Qt3/4 transition since the code seems to assume the first timer was a "single shot" timer, but it could have been wrong forever. (cherry picked from commit 62561ad031a2c1c03f9abc266c621d83e48f6b8b) M +80 -84 systemtray.cpp M +12 -14 systemtray.h https://invent.kde.org/multimedia/juk/commit/b64b37616f0490f60526defaf5335c4731dd41b4 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 417551] Juk ignores the queue if a file gets dragged there
https://bugs.kde.org/show_bug.cgi?id=417551 Michael Pyne changed: What|Removed |Added Latest Commit|https://invent.kde.org/mult |https://invent.kde.org/mult |imedia/juk/commit/6aef3be86 |imedia/juk/commit/b46844f6a |82d829f07ab86901ceff8755fee |ad1359f971ad2c876d7c0728cec |8610|77d5 --- Comment #2 from Michael Pyne --- Git commit b46844f6aad1359f971ad2c876d7c0728cec77d5 by Michael Pyne. Committed on 25/03/2021 at 01:38. Pushed by mpyne into branch 'release/21.04'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 302250, bug 303901, bug 336637, bug 353259, bug 404157 FIXED-IN:21.04 (cherry picked from commit 6aef3be8682d829f07ab86901ceff8755fee8610) M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +1-0juk.cpp M +19 -29 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +21 -22 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/b46844f6aad1359f971ad2c876d7c0728cec77d5 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 166711] random playing is not so random
https://bugs.kde.org/show_bug.cgi?id=166711 Michael Pyne changed: What|Removed |Added Latest Commit|https://invent.kde.org/mult |https://invent.kde.org/mult |imedia/juk/commit/6aef3be86 |imedia/juk/commit/b46844f6a |82d829f07ab86901ceff8755fee |ad1359f971ad2c876d7c0728cec |8610|77d5 --- Comment #7 from Michael Pyne --- Git commit b46844f6aad1359f971ad2c876d7c0728cec77d5 by Michael Pyne. Committed on 25/03/2021 at 01:38. Pushed by mpyne into branch 'release/21.04'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 302250, bug 303901, bug 336637, bug 353259, bug 404157, bug 417551 FIXED-IN:21.04 (cherry picked from commit 6aef3be8682d829f07ab86901ceff8755fee8610) M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +1-0juk.cpp M +19 -29 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +21 -22 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/b46844f6aad1359f971ad2c876d7c0728cec77d5 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 100356] double-click to play item only works, till play-queue was used
https://bugs.kde.org/show_bug.cgi?id=100356 Michael Pyne changed: What|Removed |Added Latest Commit|https://invent.kde.org/mult |https://invent.kde.org/mult |imedia/juk/commit/6aef3be86 |imedia/juk/commit/b46844f6a |82d829f07ab86901ceff8755fee |ad1359f971ad2c876d7c0728cec |8610|77d5 --- Comment #7 from Michael Pyne --- Git commit b46844f6aad1359f971ad2c876d7c0728cec77d5 by Michael Pyne. Committed on 25/03/2021 at 01:38. Pushed by mpyne into branch 'release/21.04'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 166711, bug 302250, bug 303901, bug 336637, bug 353259, bug 404157, bug 417551 FIXED-IN:21.04 (cherry picked from commit 6aef3be8682d829f07ab86901ceff8755fee8610) M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +1-0juk.cpp M +19 -29 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +21 -22 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/b46844f6aad1359f971ad2c876d7c0728cec77d5 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 302250] juk album random play does not play album
https://bugs.kde.org/show_bug.cgi?id=302250 Michael Pyne changed: What|Removed |Added Latest Commit|https://invent.kde.org/mult |https://invent.kde.org/mult |imedia/juk/commit/6aef3be86 |imedia/juk/commit/b46844f6a |82d829f07ab86901ceff8755fee |ad1359f971ad2c876d7c0728cec |8610|77d5 --- Comment #5 from Michael Pyne --- Git commit b46844f6aad1359f971ad2c876d7c0728cec77d5 by Michael Pyne. Committed on 25/03/2021 at 01:38. Pushed by mpyne into branch 'release/21.04'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 303901, bug 336637, bug 353259, bug 404157, bug 417551 FIXED-IN:21.04 (cherry picked from commit 6aef3be8682d829f07ab86901ceff8755fee8610) M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +1-0juk.cpp M +19 -29 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +21 -22 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/b46844f6aad1359f971ad2c876d7c0728cec77d5 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 356224] Always move the caret to the currently playing item
https://bugs.kde.org/show_bug.cgi?id=356224 Michael Pyne changed: What|Removed |Added Status|REPORTED|RESOLVED Version Fixed In||21.04 Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/cbfe92c37 ||14ba1edf1403a46ae79d905db28 ||1aaf Resolution|--- |FIXED --- Comment #2 from Michael Pyne --- Git commit cbfe92c3714ba1edf1403a46ae79d905db281aaf by Michael Pyne. Committed on 24/03/2021 at 03:08. Pushed by mpyne into branch 'master'. playlist: Ensure playing track is visible when it changes. I tried to ensure this only causes the playlist to jump when the user isn't directly interacting with the track (e.g. double click to play). FIXED-IN:21.04 M +1-1playermanager.cpp M +10 -2playlist.cpp M +1-1playlist.h M +6-0playlistcollection.cpp https://invent.kde.org/multimedia/juk/commit/cbfe92c3714ba1edf1403a46ae79d905db281aaf -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 210384] Juk starts(?) but no window ever comes up
https://bugs.kde.org/show_bug.cgi?id=210384 Michael Pyne changed: What|Removed |Added Resolution|--- |WORKSFORME Status|REPORTED|RESOLVED --- Comment #11 from Michael Pyne --- I remember this bug and fixed it a few years back. Though I don't seem to have made it easy to find the commit that did it. -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 100571] Wrong playlist after last song in Play Queue finished
https://bugs.kde.org/show_bug.cgi?id=100571 Michael Pyne changed: What|Removed |Added Status|REPORTED|RESOLVED Latest Commit||6aef3be8682d829f07ab86901ce ||ff8755fee8610 Version Fixed In||21.04 Resolution|--- |FIXED CC||mp...@kde.org --- Comment #3 from Michael Pyne --- I believe this is fixed with a recent commit to JuK. In some of my testing I was able to reproduce ways for the Play Queue to end up switching to the CollectionList instead of the last item's playlist but I can't reproduce that myself now... this mostly seems to work for me. -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 222919] Juk: clicking on Album name of current played song produces list with false entries
https://bugs.kde.org/show_bug.cgi?id=222919 Michael Pyne changed: What|Removed |Added Status|CONFIRMED |RESOLVED Resolution|--- |WORKSFORME --- Comment #4 from Michael Pyne --- I haven't been able to reproduce this in years, though I have refactored this code a bit to fix some other bugs which should hopefully address this as well. -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 373056] The player works, but no music
https://bugs.kde.org/show_bug.cgi?id=373056 Michael Pyne changed: What|Removed |Added Resolution|--- |FIXED Status|REPORTED|RESOLVED Latest Commit||6aef3be8682d829f07ab86901ce ||ff8755fee8610 Version Fixed In||21.04 --- Comment #2 from Michael Pyne --- I believe this is fixed with a recent commit that should be included in the upcoming 21.04 release. -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 336250] Hide action using a wrong icon
https://bugs.kde.org/show_bug.cgi?id=336250 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/83cb227ee ||4e7b2b713f3c0799960adc04c1f ||fd5e Status|REOPENED|RESOLVED Resolution|--- |FIXED Version Fixed In||21.04 --- Comment #3 from Michael Pyne --- Git commit 83cb227ee4e7b2b713f3c0799960adc04c1ffd5e by Michael Pyne. Committed on 23/03/2021 at 03:50. Pushed by mpyne into branch 'master'. playqueue: Use an appropriate icon for rebadged "remove playlist" action. A JuK user pointed out that the Play Queue context menu shows "Hide" instead of "Remove Playlist" on the context menu, but that the icon still looks like a trash can, which is not appropriate for "Hide". I wasn't sure of a great icon to use either, but "list-remove" at least does not imply that the Play Queue is being tossed in the rubbish. FIXED-IN:21.04 M +14 -10 playlistbox.cpp https://invent.kde.org/multimedia/juk/commit/83cb227ee4e7b2b713f3c0799960adc04c1ffd5e -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 420500] Please make "Use random play" toggleable
https://bugs.kde.org/show_bug.cgi?id=420500 Michael Pyne changed: What|Removed |Added Resolution|--- |WORKSFORME Status|REPORTED|RESOLVED --- Comment #1 from Michael Pyne --- Please try using the "Random Play" action for the toolbar. This is a dropdown button or a context menu that contains all the potential random playback toggles (there are 3 rather than 2 so it's not directly suitable to be a toggle button). -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 166711] random playing is not so random
https://bugs.kde.org/show_bug.cgi?id=166711 Michael Pyne changed: What|Removed |Added Status|REPORTED|RESOLVED Resolution|--- |FIXED Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/6aef3be86 ||82d829f07ab86901ceff8755fee ||8610 Version Fixed In||21.04 --- Comment #6 from Michael Pyne --- Git commit 6aef3be8682d829f07ab86901ceff8755fee8610 by Michael Pyne. Committed on 23/03/2021 at 02:59. Pushed by mpyne into branch 'master'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 302250, bug 303901, bug 336637, bug 353259, bug 404157, bug 417551 FIXED-IN:21.04 M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +0-1juk.cpp M +2-21 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +14 -18 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/6aef3be8682d829f07ab86901ceff8755fee8610 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 303901] Restart album when using random album
https://bugs.kde.org/show_bug.cgi?id=303901 Michael Pyne changed: What|Removed |Added Resolution|--- |FIXED Status|REPORTED|RESOLVED Version Fixed In||21.04 Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/6aef3be86 ||82d829f07ab86901ceff8755fee ||8610 --- Comment #4 from Michael Pyne --- Git commit 6aef3be8682d829f07ab86901ceff8755fee8610 by Michael Pyne. Committed on 23/03/2021 at 02:59. Pushed by mpyne into branch 'master'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 302250, bug 336637, bug 353259, bug 404157, bug 417551 FIXED-IN:21.04 M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +0-1juk.cpp M +2-21 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +14 -18 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/6aef3be8682d829f07ab86901ceff8755fee8610 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 417551] Juk ignores the queue if a file gets dragged there
https://bugs.kde.org/show_bug.cgi?id=417551 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/6aef3be86 ||82d829f07ab86901ceff8755fee ||8610 Resolution|--- |FIXED Status|REPORTED|RESOLVED Version Fixed In||21.04 --- Comment #1 from Michael Pyne --- Git commit 6aef3be8682d829f07ab86901ceff8755fee8610 by Michael Pyne. Committed on 23/03/2021 at 02:59. Pushed by mpyne into branch 'master'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 302250, bug 303901, bug 336637, bug 353259, bug 404157 FIXED-IN:21.04 M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +0-1juk.cpp M +2-21 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +14 -18 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/6aef3be8682d829f07ab86901ceff8755fee8610 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 404157] Track length is not updated when advancing in the playlist
https://bugs.kde.org/show_bug.cgi?id=404157 Michael Pyne changed: What|Removed |Added Version Fixed In||21.04 Resolution|--- |FIXED Status|REPORTED|RESOLVED Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/6aef3be86 ||82d829f07ab86901ceff8755fee ||8610 --- Comment #1 from Michael Pyne --- Git commit 6aef3be8682d829f07ab86901ceff8755fee8610 by Michael Pyne. Committed on 23/03/2021 at 02:59. Pushed by mpyne into branch 'master'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 302250, bug 303901, bug 336637, bug 353259, bug 417551 FIXED-IN:21.04 M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +0-1juk.cpp M +2-21 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +14 -18 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/6aef3be8682d829f07ab86901ceff8755fee8610 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 353259] Juk player will not play tracks added to play queue
https://bugs.kde.org/show_bug.cgi?id=353259 Michael Pyne changed: What|Removed |Added Version Fixed In||21.04 Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/6aef3be86 ||82d829f07ab86901ceff8755fee ||8610 Status|REPORTED|RESOLVED Resolution|--- |FIXED --- Comment #2 from Michael Pyne --- Git commit 6aef3be8682d829f07ab86901ceff8755fee8610 by Michael Pyne. Committed on 23/03/2021 at 02:59. Pushed by mpyne into branch 'master'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 302250, bug 303901, bug 336637, bug 404157, bug 417551 FIXED-IN:21.04 M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +0-1juk.cpp M +2-21 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +14 -18 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/6aef3be8682d829f07ab86901ceff8755fee8610 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 100356] double-click to play item only works, till play-queue was used
https://bugs.kde.org/show_bug.cgi?id=100356 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/6aef3be86 ||82d829f07ab86901ceff8755fee ||8610 Status|CONFIRMED |RESOLVED Version Fixed In||21.04 Resolution|--- |FIXED --- Comment #6 from Michael Pyne --- Git commit 6aef3be8682d829f07ab86901ceff8755fee8610 by Michael Pyne. Committed on 23/03/2021 at 02:59. Pushed by mpyne into branch 'master'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 166711, bug 302250, bug 303901, bug 336637, bug 353259, bug 404157, bug 417551 FIXED-IN:21.04 M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +0-1juk.cpp M +2-21 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +14 -18 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/6aef3be8682d829f07ab86901ceff8755fee8610 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 302250] juk album random play does not play album
https://bugs.kde.org/show_bug.cgi?id=302250 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/6aef3be86 ||82d829f07ab86901ceff8755fee ||8610 Status|CONFIRMED |RESOLVED Version Fixed In||21.04 Resolution|--- |FIXED --- Comment #4 from Michael Pyne --- Git commit 6aef3be8682d829f07ab86901ceff8755fee8610 by Michael Pyne. Committed on 23/03/2021 at 02:59. Pushed by mpyne into branch 'master'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 303901, bug 336637, bug 353259, bug 404157, bug 417551 FIXED-IN:21.04 M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +0-1juk.cpp M +2-21 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +14 -18 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/6aef3be8682d829f07ab86901ceff8755fee8610 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 336637] After playing queued items, juk returns to "collection" not previous playlist
https://bugs.kde.org/show_bug.cgi?id=336637 Michael Pyne changed: What|Removed |Added Status|REPORTED|RESOLVED Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/6aef3be86 ||82d829f07ab86901ceff8755fee ||8610 Resolution|--- |FIXED Version Fixed In||21.04 --- Comment #3 from Michael Pyne --- Git commit 6aef3be8682d829f07ab86901ceff8755fee8610 by Michael Pyne. Committed on 23/03/2021 at 02:59. Pushed by mpyne into branch 'master'. Improve track sequencing by removing the track sequencing classes. This removes one of my first contributions to JuK :( But it's worth it because the extra code is not worth the complexity, seeing as how the job is really pretty simple in the first place, even with album random play and randomized playback. I believe this also fixes some bugs, including some longstanding ones. Bug 417551 (being unable to drag and drop into Play Queue) had some related work in a recent commit but was still broken until now. Related: bug 100356, bug 166711, bug 302250, bug 303901, bug 353259, bug 404157, bug 417551 FIXED-IN:21.04 M +0-2CMakeLists.txt M +7-20 dynamicplaylist.cpp M +2-0dynamicplaylist.h M +0-1juk.cpp M +2-21 playermanager.cpp M +0-1playermanager.h M +62 -51 playlist.cpp M +14 -1playlist.h M +14 -18 playlistbox.cpp M +3-1playlistbox.h M +2-4playlistcollection.cpp M +4-0playlistcollection.h M +11 -0playlistitem.cpp M +1-0playlistitem.h M +0-2playlistsplitter.cpp D +0-316 tracksequenceiterator.cpp D +0-233 tracksequenceiterator.h D +0-182 tracksequencemanager.cpp D +0-187 tracksequencemanager.h M +36 -166 upcomingplaylist.cpp M +11 -133 upcomingplaylist.h https://invent.kde.org/multimedia/juk/commit/6aef3be8682d829f07ab86901ceff8755fee8610 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 276753] Time Slider freezes some times
https://bugs.kde.org/show_bug.cgi?id=276753 Michael Pyne changed: What|Removed |Added Resolution|--- |WAITINGFORINFO Status|REPORTED|NEEDSINFO --- Comment #3 from Michael Pyne --- No response from submitter. But in any event I've never been able to reproduce, and I've recently significantly restructured the playback code in a way that I think is pretty solid (I had encountered bugs in the time slider being wrong, which I've fixed, which did not match the reported behavior here but may have been related). -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 243105] No playback possible
https://bugs.kde.org/show_bug.cgi?id=243105 Michael Pyne changed: What|Removed |Added Resolution|--- |WORKSFORME Status|REPORTED|RESOLVED -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 413065] Untranslatable file dialog filter "Playlists"
https://bugs.kde.org/show_bug.cgi?id=413065 Michael Pyne changed: What|Removed |Added Version Fixed In||21.04 Latest Commit||https://invent.kde.org/mult ||imedia/juk/commit/14731917b ||abaa45afca49c24965401354066 ||212b Resolution|--- |FIXED Status|REPORTED|RESOLVED --- Comment #1 from Michael Pyne --- Git commit 14731917babaa45afca49c24965401354066212b by Michael Pyne. Committed on 21/03/2021 at 18:33. Pushed by mpyne into branch 'master'. mediafiles: Make Playlist Save As file type message translatable. Sorry for missing this for so long. :( FIXED-IN:21.04 M +1-1mediafiles.cpp https://invent.kde.org/multimedia/juk/commit/14731917babaa45afca49c24965401354066212b -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 165899] juk take to much CPU / Power when IDLE
https://bugs.kde.org/show_bug.cgi?id=165899 --- Comment #4 from Michael Pyne --- Git commit 62561ad031a2c1c03f9abc266c621d83e48f6b8b by Michael Pyne. Committed on 21/03/2021 at 18:19. Pushed by mpyne into branch 'master'. systemtray: Cleanups and modernization. Also a timer bugfix. The bug: When using the track popup announcement, a timer kicks off to tell the popup to fade out. A second timer is used to incrementally fade the popup a bit every few milliseconds until the popup can finally be hidden. However the first timer was never stopped, and would kick off the fadeout sequence again, over and over, only hidden to view since the popup is no longer visible. This might be related to the old bug 165899 (which I tried to fix a different way). I believe this happened in the Qt3/4 transition since the code seems to assume the first timer was a "single shot" timer, but it could have been wrong forever. M +80 -84 systemtray.cpp M +12 -14 systemtray.h https://invent.kde.org/multimedia/juk/commit/62561ad031a2c1c03f9abc266c621d83e48f6b8b -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kcoreaddons] [Bug 387663] Dolphin doesn't update view (doesn't show new files)
https://bugs.kde.org/show_bug.cgi?id=387663 --- Comment #30 from Michael Pyne --- NFS might be a different thing. But I have a merge request open for the potential inotify bug, at https://invent.kde.org/frameworks/kcoreaddons/-/merge_requests/81 if anyone who can reproduce the bug is able to take a look at testing. -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kcoreaddons] [Bug 433402] KJobsTest in test suite flags undefined behavior sanitizer
https://bugs.kde.org/show_bug.cgi?id=433402 Michael Pyne changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |FIXED Latest Commit||https://invent.kde.org/fram ||eworks/kcoreaddons/commit/c ||1f6b6f437cb6764875e1bf67cb8 ||59f10adcb4d7 --- Comment #2 from Michael Pyne --- Git commit c1f6b6f437cb6764875e1bf67cb859f10adcb4d7 by Michael Pyne. Committed on 23/02/2021 at 00:43. Pushed by mpyne into branch 'master'. autotests: Fix autotests to pass under gcc ubsan and leak sanitizer. The GCC undefined behavior sanitizer (ubsan) flags our deliberate usage of undefined behavior within the KJob autotests, and the leak sanitizer flags some memory leaks from the use of KAutoSaveFile. It would be good to be able to consistently run the autotests under the use of compiler sanitizers to catch bugs in kcoreaddons libraries, but this relies on the code still passing the test suite in this case. It appears to me that the use of undefined behavior in KJobTest is as a simple sanity check that KJob::isFinished is consistent with the fact that we just called a slot connected to `KJob::finished()`. But `KJob::isFinished()` is a protected method and in the context of a `KJob::finished()` signal emitted from `KJob::~KJob()` there can be no subclasses to interrogate `KJob::isFinished()` anyways. I try to go about the sanity check instead by counting specific number of `finished` calls from each individual job, ensuring that no job ever emits `finished` more than once, that the corresponding `slotResult` call (on success) comes after one (and only one) `finished` signal for the job, and that the number of KJob test that emit `finished` but do not emit `result` is exactly equal to the number of expected failed jobs It might be simpler just to use `qobject_cast` or `dynamic_cast` and only perform the sanity check when we know the `finished` call is happening from a subclass. For KAutoSaveFile, it's just a matter of remembering to call delete on the list of returned KAutoSaveFile* objects. This is worth fixing just because autotests are sometimes treated as documentation on proper usage, but I also make sure to clarify in the API documentation (the code examples in the API docs show correct usage but the per-method documentation does not make clear that the application owns the returned pointers). M +9-3autotests/kautosavefiletest.cpp M +16 -8autotests/kjobtest.cpp M +2-0autotests/kjobtest.h M +10 -0src/lib/io/kautosavefile.h https://invent.kde.org/frameworks/kcoreaddons/commit/c1f6b6f437cb6764875e1bf67cb859f10adcb4d7 -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kcoreaddons] [Bug 433402] New: KJobsTest in test suite flags undefined behavior sanitizer
https://bugs.kde.org/show_bug.cgi?id=433402 Bug ID: 433402 Summary: KJobsTest in test suite flags undefined behavior sanitizer Product: frameworks-kcoreaddons Version: 5.79.0 Platform: Compiled Sources OS: Linux Status: REPORTED Severity: minor Priority: NOR Component: general Assignee: mp...@kde.org Reporter: mp...@kde.org CC: kdelibs-b...@kde.org Target Milestone: --- SUMMARY The autotest "kjobtest" causes errors and warning messages about use of undefined behavior. This makes it problematic to run the test suite under aggressive compiler sanitizing because it will then be expected to fail, which reduces the viability of using sanitizers to catch bugs. STEPS TO REPRODUCE 1. Ensure that kcoreaddons is configured to build autotests by CMake. 2. Compile kcoreaddons with Address Sanitizer (asan) and Undefined Behavior Sanitizer (ubsan). I used in my kdesrc-buildrc: options kcoreaddons cmake-options -DBUILD_TESTING:BOOL=ON cxxflags -fsanitize=address -fsanitize=undefined -ggdb end options ***NOTE*** if you do this only for kcoreaddons and install the result, your other Qt/KDE applications may fail to launch. I believe this is due to something about the Qt platform plugin because running something like "XDG_CURRENT_DESKTOP=GNOME assistant" succeeds in this case when just running "assistant" would not. 2. Go to kcoreaddons build directory and run ./bin/kjobtest OBSERVED RESULT user@domain /kdesrc/build/kf5/frameworks/kcoreaddons $ ./bin/kjobtest * Start testing of KJobTest * Config: Using QtTest library 5.15.1, Qt 5.15.1 (x86_64-little_endian-lp64 shared (dynamic) release build; by GCC 10.2.0), gentoo unknown PASS : KJobTest::initTestCase() PASS : KJobTest::testEmitResult(no error) PASS : KJobTest::testEmitResult(error no text) PASS : KJobTest::testEmitResult(error with text) PASS : KJobTest::testProgressTracking() PASS : KJobTest::testExec(no error) PASS : KJobTest::testExec(error no text) PASS : KJobTest::testExec(error with text) PASS : KJobTest::testKill(killed with result) PASS : KJobTest::testKill(killed quietly) /kdesrc/src/kf5/frameworks/kcoreaddons/autotests/kjobtest.cpp:439:5: runtime error: downcast of address 0x60303be0 which does not point to an object of type 'TestJob' 0x60303be0: note: object is of type 'KJob' 02 00 00 45 b8 a2 71 5f 1c 7f 00 00 20 24 00 00 80 60 00 00 a0 24 00 00 80 60 00 00 00 00 00 00 ^~~ vptr for 'KJob' PASS : KJobTest::testDestroy() PASS : KJobTest::testEmitAtMostOnce(Start-Start-autoDelete) PASS : KJobTest::testEmitAtMostOnce(Start-KillQuietly-autoDelete) PASS : KJobTest::testEmitAtMostOnce(Start-KillVerbosely-autoDelete) PASS : KJobTest::testEmitAtMostOnce(KillQuietly-Start-autoDelete) PASS : KJobTest::testEmitAtMostOnce(KillQuietly-KillQuietly-autoDelete) PASS : KJobTest::testEmitAtMostOnce(KillQuietly-KillVerbosely-autoDelete) PASS : KJobTest::testEmitAtMostOnce(KillVerbosely-Start-autoDelete) PASS : KJobTest::testEmitAtMostOnce(KillVerbosely-KillQuietly-autoDelete) PASS : KJobTest::testEmitAtMostOnce(KillVerbosely-KillVerbosely-autoDelete) PASS : KJobTest::testEmitAtMostOnce(Start-Start) PASS : KJobTest::testEmitAtMostOnce(Start-KillQuietly) PASS : KJobTest::testEmitAtMostOnce(Start-KillVerbosely) PASS : KJobTest::testEmitAtMostOnce(KillQuietly-Start) PASS : KJobTest::testEmitAtMostOnce(KillQuietly-KillQuietly) PASS : KJobTest::testEmitAtMostOnce(KillQuietly-KillVerbosely) PASS : KJobTest::testEmitAtMostOnce(KillVerbosely-Start) PASS : KJobTest::testEmitAtMostOnce(KillVerbosely-KillQuietly) PASS : KJobTest::testEmitAtMostOnce(KillVerbosely-KillVerbosely) PASS : KJobTest::testDelegateUsage() PASS : KJobTest::testNestedExec() PASS : KJobTest::cleanupTestCase() Totals: 32 passed, 0 failed, 0 skipped, 0 blacklisted, 302ms * Finished testing of KJobTest * = ==131922==ERROR: LeakSanitizer: detected memory leaks Direct leak of 16 byte(s) in 2 object(s) allocated from: #0 0x7f1c60269b48 in __interceptor_realloc /var/tmp/portage/sys-devel/gcc-10.2.0-r5/work/gcc-10.2.0/libsanitizer/asan/asan_malloc_linux.cpp:164 #1 0x7f1c5e0d5f42 in d_growable_string_resize /var/tmp/portage/sys-devel/gcc-10.2.0-r5/work/build/x86_64-pc-linux-gnu/libstdc++-v3/libsupc++/cp-demangle.c:4036 #2 0x7f1c5e0d5f42 in d_growable_string_append_buffer /var/tmp/portage/sys-devel/gcc-10.2.0-r5/work/build/x86_64-pc-linux-gnu/libstdc++-v3/libsupc++/cp-demangle.c:4060 #3 0x7f1c5e0d5f42 in d_growable_string_callback_adapter /var/tmp/portage/sys-devel/gcc-10.2.0-r5/work/build/x86_64-pc-linux-gnu/libstdc++-v3/libsupc++/cp-demangle.c:4077 SUMMARY: AddressSanitizer: 16 byte(s) leaked in 2 allocation
[frameworks-kcoreaddons] [Bug 410851] K_PLUGIN_FACTORY_DECLARATION_WITH_BASEFACTORY_SKEL doesn't use basefactory
https://bugs.kde.org/show_bug.cgi?id=410851 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/fram ||eworks/kcoreaddons/commit/d ||aec110942baed6c7e8bcb7684e6 ||5411fe8acbe8 Status|REPORTED|RESOLVED Resolution|--- |FIXED --- Comment #1 from Michael Pyne --- Git commit daec110942baed6c7e8bcb7684e65411fe8acbe8 by Michael Pyne, on behalf of Adriaan de Groot. Committed on 19/02/2021 at 02:29. Pushed by mpyne into branch 'master'. kpluginfactory: Use the user-provided base factory in our factory macro. The existing macro always inherits from KPluginFactory -- making it hard to use when sub-classing KPluginFactory in order to extend it. The parameter baseFactory to the macro isn't used. Use baseFactory as the base class for the plugin. -- Commentary by committer: This closes out merge request !1. Due to the length of time between when the MR was authored and when we could finally land it, this also includes other commits added as part of the MR to: * Use @-style Doxygen comments consistently * Adapt other Doxygen API documentation to our style guide * Some other miscellaneous editorial improvements On top of those changes, I have ensured that the other changes to this header made in the meantime now conform to the consistent API documentation (i.e. other \ to @ conversions), and to update the @since version to be accurate again. M +148 -117 src/lib/plugin/kpluginfactory.h https://invent.kde.org/frameworks/kcoreaddons/commit/daec110942baed6c7e8bcb7684e65411fe8acbe8 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 420933] kdesrc-build fails to update modules randomly (git stash failure)
https://bugs.kde.org/show_bug.cgi?id=420933 Michael Pyne changed: What|Removed |Added Latest Commit||https://invent.kde.org/sdk/ ||kdesrc-build/commit/c4ad3db ||b9038f9e0339f898a75fa48ce4f ||dc9cea Resolution|--- |FIXED Status|ASSIGNED|RESOLVED --- Comment #6 from Michael Pyne --- Git commit c4ad3dbb9038f9e0339f898a75fa48ce4fdc9cea by Michael Pyne. Committed on 11/10/2020 at 22:36. Pushed by mpyne into branch 'master'. git: Automatically reapply stashed changes if possible. In the event that the stash does not apply a "post-build" message is issued to warn the user. Otherwise there is an in-situ message. This addresses user feedback I have received on Invent and IRC. In particular re-fixes #42 and bug 420933, and fixes #49. All that I do here is to use the existing logic and add "git stash pop" after the update phase has completed. I've previously tried --autostash (see !32 and !45) and that had issues of its own. Using this approach allows us to use rely on git directly without risk of missing a merge conflict that git sometimes seems to generate with `git pull --autostash` (see https://invent.kde.org/sdk/kdesrc-build/-/merge_requests/32#note_46926). The merge request !61 may still be applicable. I haven't looked long enough to tell if there's more on it than the --autostash feature which still causes us problems... it's possible that there's a more reliable method than what we're doing though we've already taken pains to simplify this routine because of the recurring stash failures that we've seen previously. M +18 -7modules/ksb/Updater/Git.pm https://invent.kde.org/sdk/kdesrc-build/commit/c4ad3dbb9038f9e0339f898a75fa48ce4fdc9cea -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 426219] Add a way to disable installing udev rule
https://bugs.kde.org/show_bug.cgi?id=426219 --- Comment #5 from Michael Pyne --- Git commit 2317c3a32e5aee1e926bd912275fd2149757f69a by Michael Pyne. Committed on 27/09/2020 at 19:15. Pushed by mpyne into branch 'master'. setup: Make include-dependencies optional, show module groups not chosen. My hope is to make it easier for new users to understand how to add module groups if they are desired later, since there has previously been no generated note or warning in the generated kdesrc-buildrc if the user chose no major groups. In this case we had a user who, even though they didn't select any module groups, still had kdesrc-build trying to build bluez-qt due to include-dependencies being set to 'true' (the default). However bluez-qt needs a CMake option to be passed to build with kdesrc-build; this option isn't set unless the kf5-frameworks-build-include file is read in. That fix will happen in a separate commit, this should hopefully make it easier to read through the generated kdesrc-buildrc in any case. M +74 -39 kdesrc-build-setup https://invent.kde.org/sdk/kdesrc-build/commit/2317c3a32e5aee1e926bd912275fd2149757f69a -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 426219] Add a way to disable installing udev rule
https://bugs.kde.org/show_bug.cgi?id=426219 --- Comment #6 from Michael Pyne --- Git commit ed1545b71654c7e9dc3f3ca65e0c29ef1f6d92c5 by Michael Pyne. Committed on 27/09/2020 at 19:52. Pushed by mpyne into branch 'master'. setup: Break out common 'options' blocks for kdesrc-build-setup to always use. This change should address the immediate breakage that a user had when using kdesrc-build-setup, choosing no major module groups, and then using kdesrc-build to eventually build bluez-qt. In that case, bluez-qt had an 'options' block in kf5-frameworks-build-include to address the error he'd experienced, however since kdesrc-build-setup didn't generate a include to that file, the options weren't picked up either. kdesrc-build later found the module via include-dependencies and built it anyways. To fix this, break out "always-set" options into a dedicated file (kf5-common-options-build-include) and include that from kdesrc-build-setup-generated files always. I moved these from kf5-frameworks-build-include so I added an include from that file back to kf5-common-options-build-include for backwards compatibility for existing users. This relies on duplicate options blocks continuing to work (similar to C++'s One Definition Rule) so I've documented that in the code for now. M +1-0CMakeLists.txt M +10 -5kdesrc-build-setup A +19 -0kf5-common-options-build-include M +4-15 kf5-frameworks-build-include M +2-0kf5-qt5-build-include M +2-0kf5-workspace-build-include M +5-0modules/ksb/Application.pm M +6-1modules/ksb/FirstRun.pm https://invent.kde.org/sdk/kdesrc-build/commit/ed1545b71654c7e9dc3f3ca65e0c29ef1f6d92c5 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 426219] Add a way to disable installing udev rule
https://bugs.kde.org/show_bug.cgi?id=426219 --- Comment #2 from Michael Pyne --- As of commit cbf96256, the suggested CMake option is already passed to bluez-qt. Someone else has reported this as a bug on IRC. Could I ask how kdesrc-build was installed (cloned from git or distribution packages) and how the kdesrc-build configuration was generated (kdesrc-build-setup, modification of the sample configuration, kdesrc-build --initial-setup, something else)? -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 420718] juk segfaut on every startup
https://bugs.kde.org/show_bug.cgi?id=420718 Michael Pyne changed: What|Removed |Added Resolution|--- |FIXED Version Fixed In||20.04.1 Status|REPORTED|RESOLVED Latest Commit||https://invent.kde.org/kde/ ||juk/commit/e843f457d5656d77 ||8396d8045289117443b70975 --- Comment #5 from Michael Pyne --- Git commit e843f457d5656d778396d8045289117443b70975 by Michael Pyne. Committed on 10/05/2020 at 15:52. Pushed by mpyne into branch 'master'. playlist: Fix crasher with playlists using reserved columns. I introduced this bug in commit a800c1b3ffeb trying to fix a different memory mis-using bug. Sigh. In this case playlists that reserve additional columns would get a valid memory assignment but would be copied into the memory block incorrectly. This needed a back_inserter as well. Thanks to Markus for pointing out the regression, then opening a bug, then leaving a comment to point out both of these, and then drafting a patch that fixes the bug. I can confirm it works in my testing, it also seems to address a bug relating to playback stopping abrupting at the end of a track instead of skipping to the next track, but who knows at this point. FIXED-IN:20.04.1 M +1-1playlist.cpp https://invent.kde.org/kde/juk/commit/e843f457d5656d778396d8045289117443b70975 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 420933] kdesrc-build fails to update modules randomly (git stash failure)
https://bugs.kde.org/show_bug.cgi?id=420933 --- Comment #2 from Michael Pyne --- Git commit 5019317f41977d67f1932cacb68850bda83ec73b by Michael Pyne. Committed on 02/05/2020 at 23:21. Pushed by mpyne into branch 'master'. git: Capture basic git-status info before error-prone ops. There have long been bug reports around git-stash handling. I just had to enter another one, but it's hard to troubleshoot what git was seeing after the fetch and rebase has already happened, even if you try to rewind with reflog. For now this will just run a simple 'git-status' as a smoke check before the update, and optionally if a stash or error occur. My hope is this will permit more detail to be provided with subsequent bug reports. M +6-0modules/ksb/Updater/Git.pm https://invent.kde.org/kde/kdesrc-build/commit/5019317f41977d67f1932cacb68850bda83ec73b -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 420933] kdesrc-build fails to update modules randomly (git stash failure)
https://bugs.kde.org/show_bug.cgi?id=420933 Michael Pyne changed: What|Removed |Added CC||fa...@kde.org --- Comment #1 from Michael Pyne --- David, I'm not able to immediately resolve so entering a bug. I will update kdesrc-build to include 'git-status' output before update and stash operations to potentially improve on the error reporting available. I suspect something to do with long periods of changes between updates. For clarity in case you look yourself, the basic sequence kdesrc-build for updates to existing modules is: cd $srcdir git fetch --tags $remote DO_STASH = git diff-index --quiet HEAD != 0 if DO_STASH: git stash save --quiet git checkout $branch ; unconditionally done git rebase origin/$branch if DO_STASH: git stash pop -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 420933] New: kdesrc-build fails to update modules randomly (git stash failure)
https://bugs.kde.org/show_bug.cgi?id=420933 Bug ID: 420933 Summary: kdesrc-build fails to update modules randomly (git stash failure) Product: kdesrc-build Version: Git Platform: Other OS: Linux Status: REPORTED Severity: normal Priority: NOR Component: general Assignee: mp...@kde.org Reporter: mp...@kde.org Target Milestone: --- Created attachment 128105 --> https://bugs.kde.org/attachment.cgi?id=128105&action=edit kdesrc-build logs from my failed kactivitymanagerd run SUMMARY dfaure reports that kdesrc-build can consider a module failed to update (even though it successfully fetches updates from the git server and successfully rebases the module to the updated 'master' branch). It complains about being unable to restore stashed local changes. See https://invent.kde.org/snippets/868. The pertinent error: user@localhost:~/release-tools$ cd ../src && ./kdesrc-build --src-only Updating kde-build-metadata (to branch master) Local changes detected, stashing them away... Module updated, reapplying your local changes. * * Unable to re-apply stashed changes to kde-build-metadata! * * These changes were saved using the name "kdesrc-build auto-stash at 2020-05-02-21:48" * and should still be available using the name stash@{0}, the command run * to re-apply was git stash pop --index. Resolve this before you run * kdesrc-build to update this module again. * * If you do not desire to keep your local changes, then you can generally run * git reset --hard HEAD, or simply delete the source directory for * kde-build-metadata. Developers be careful, doing either of these options will remove * any of your local work. * Unable to download required metadata for build process * Will attempt to press onward... * Exception message: Runtime Error: Failed to re-apply stashed changes for kde-build-metadata STEPS TO REPRODUCE 1. Change to desired config directory 2. Run kdesrc-build --src-only OBSERVED RESULT Modules randomly fail to update complaining of re-application of stashed changes. Different modules fail on subsequent builds. [18:17] thanks [18:17] it seems like ot happened randomly in various repos [18:17] it [18:17] git version 2.26.1, if it matters [18:33] mpyne happened twice in a row upon "kdesrc-build frameworks", but on different modules every time [18:34] thanks [18:34] mpyne: once kde-build-metadata, once kio and kdesrc-build itself... pretty random In my own testing from an old checkout, applying to master as of 2020-05-02, I had one module fail for this reason, kactivitymanagerd. I believe this may be related to a commit that removed a file, but I cannot understand what situation kactivitymanagerd had been in before that would have led kdesrc-build to think a local change had occurred. I was not able to reproduce random modules having this failure. EXPECTED RESULT The source code modules cleanly update, since there were no local modifications to worry about. SOFTWARE/OS VERSIONS kdesrc-build: v19.12-44-geaf569d ADDITIONAL INFORMATION There have been recent changes to git-stash handling to fix other "fake local change detected" bugs. -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 420486] Bump Qt5 requirement to match other packages requirements
https://bugs.kde.org/show_bug.cgi?id=420486 Michael Pyne changed: What|Removed |Added Resolution|--- |FIXED Status|REPORTED|RESOLVED --- Comment #1 from Michael Pyne --- Thanks. This has been applied on git master. See https://invent.kde.org/kde/kdesrc-build/-/merge_requests/28 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 402511] When using the --stop-on-failure argument, print contents of log file for failed build to the console
https://bugs.kde.org/show_bug.cgi?id=402511 Michael Pyne changed: What|Removed |Added CC||mklape...@kde.org --- Comment #4 from Michael Pyne --- *** Bug 331203 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 331203] Display error in 'less' after build set finishes
https://bugs.kde.org/show_bug.cgi?id=331203 Michael Pyne changed: What|Removed |Added Status|CONFIRMED |RESOLVED Resolution|--- |DUPLICATE --- Comment #4 from Michael Pyne --- *** This bug has been marked as a duplicate of bug 402511 *** -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 411419] kdesrc-build dolphin --include-dependencies fails with gpgme
https://bugs.kde.org/show_bug.cgi?id=411419 --- Comment #5 from Michael Pyne --- I've updated the Qt branch that is built by default to 5.13 now (for bug 410921). I've also reduced a bit of the error message spam you get for the first major failing module, libdbusmenu-qt. Unfortunately I've not been able to reproduce the libdbusmenu-qt failure myself. Someone else has reported a very similar bug (bug 413031). As your StackOverflow question mentioned, you may have to remove build directories (can be done using the --refresh-build flag to kdesrc-build) after you make potential fixes. You might also use the --stop-on-failure flag until things get stable on your system, that will stop the entire build if a single module fails to avoid wasting time trying to build modules where the dependencies didn't make it through. -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 410921] Respect XDG Base Directory Specification.
https://bugs.kde.org/show_bug.cgi?id=410921 --- Comment #1 from Michael Pyne --- For those who come across this bug, a merge request that would fix this is at https://invent.kde.org/kde/kdesrc-build/merge_requests/11 -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 410915] Arch Partial Upgrade
https://bugs.kde.org/show_bug.cgi?id=410915 Michael Pyne changed: What|Removed |Added Resolution|--- |FIXED Latest Commit||673ef32a28e79c6325ed24e4f65 ||84ea7deab0b49 Status|CONFIRMED |RESOLVED --- Comment #3 from Michael Pyne --- The improved command referenced in the Phabricator change request was merged to fix this bug back in August. Not sure why I missed this bug report in the meantime but this has been already fixed. Thanks for reporting! -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 412989] Please update Qt to 5.13
https://bugs.kde.org/show_bug.cgi?id=412989 Michael Pyne changed: What|Removed |Added Latest Commit||46e06a53fd04cd4e322de782aad ||2611b3d1d4600 Status|REPORTED|RESOLVED Resolution|--- |FIXED --- Comment #1 from Michael Pyne --- Addressed in commit 46e06a53fd04cd4e322de782aad2611b3d1d4600 I have not been able to get OpenSUSE Leap to build to verify this is all that's needed so expect some residual breakage that may be needed to make Qt 5.13 happy. -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 411595] kdesrc-build l10n crashes
https://bugs.kde.org/show_bug.cgi?id=411595 --- Comment #1 from Michael Pyne --- Well, it's supposed to build and install translations but on the other hand I've not used that myself in years and you're the first to mention it one way or the other. It shouldn't be far off as long as the l10n-kde4 stuff hasn't changed much in the meantime. I'm trying to remember why 'snapshots' would have made its way into the module list (what this part of the codebase is complaining about). I remember l10n-kde4 had a '/scripts' directory that had required special handling but I don't remember a 'snapshots' dir... -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 413031] libdbusmenu-qt fails to update with kdesrc-build
https://bugs.kde.org/show_bug.cgi?id=413031 --- Comment #1 from Michael Pyne --- Thanks for including the error.log. I've just tried it with recent kdesrc-build and bzr 2.7.0, and my output was: libdbusmenu-qt/bzr-pull.log === # kdesrc-build running: 'bzr' 'pull' # from directory: /kdesrc/src/kf5/libdbusmenu-qt Using saved parent location: http://bazaar.launchpad.net/~dbusmenu-team/libdbusmenu-qt/trunk/ No revisions or tags to pull. If I try forcibly removing the source directory for libdbusmenu-qt before running kdesrc-build, kdesrc-build is able to succeed at checking out libdbusmenu-qt, with output of: libdbusmenu-qt/bzr-branch.log = # kdesrc-build running: 'bzr' 'branch' 'lp:libdbusmenu-qt' '/kdesrc/src/kf5/libdbusmenu-qt' # from directory: /kdesrc/src/kf5/sysadmin/repo-metadata You have not informed bzr of your Launchpad ID, and you must do this to write to Launchpad or access private data. See "bzr help launchpad-login". Branched 271 revisions. Although the process is run from the wrong directory (sysadmin/repo-metadata) the full destination path is still passed to bzr so the operation still succeeds. At this point I would recommend completely deleting the existing source directory and trying again. I think bzr itself is throwing an error and that seems like the fastest potential option to unstick it. -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 379464] JuK 3.14 won't remember the "Dock in System Tray" setting
https://bugs.kde.org/show_bug.cgi?id=379464 Michael Pyne changed: What|Removed |Added Status|REPORTED|NEEDSINFO Resolution|--- |WAITINGFORINFO --- Comment #1 from Michael Pyne --- I remember running into this and I fixed it at some point, but I don't remember the exact commit that did it. If you're still using this could you report back with whether the bug has been fixed or not? -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 353124] Juk ignores folder removal
https://bugs.kde.org/show_bug.cgi?id=353124 --- Comment #2 from Michael Pyne --- To be honest JuK is only poorly maintained, that much is true. We've at least setup JuK so that its Bugzilla versions are added automatically at each major release. As for your bug, saying that removing a music folder doesn't remove its corresponding music from the Collection List, I have to admit that it's working as designed, the feature is meant to guide music scanning on startup, rather than to directly control the contents of the Collection List. But I think it would make sense to work as you describe it, and certainly it isn't easy to identify the files yourself otherwise without using the "Full Path" column and then filtering on it. -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 238941] Juk causes excessive IO and CPU load on startup
https://bugs.kde.org/show_bug.cgi?id=238941 Michael Pyne changed: What|Removed |Added Status|REPORTED|RESOLVED Resolution|--- |FIXED --- Comment #3 from Michael Pyne --- I've been able to reproduce that JuK does take a lot of I/O at start up (because it loads the metadata for all music in its collection, including music that may be present in default music folder paths). I've progressively worked to address the worst bits of that, including by moving the music scanner into a separate thread last year and by fixing a bug causing music to be unnecessarily rescanned after first startup. I think that should address much of what this bug is describing, although I'm sure there's even better that could be done. -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 406791] Cant find lyrics no songs anymore.
https://bugs.kde.org/show_bug.cgi?id=406791 Michael Pyne changed: What|Removed |Added Resolution|--- |FIXED Latest Commit||https://invent.kde.org/kde/ ||juk/commit/d161bae4530bc288 ||71c6951939100bf7a7a3b924 Version Fixed In||19.08 Status|REPORTED|RESOLVED --- Comment #3 from Michael Pyne --- Git commit d161bae4530bc28871c6951939100bf7a7a3b924 by Michael Pyne, on behalf of Luigi Toscano. Committed on 04/08/2019 at 21:58. Pushed by mpyne into branch 'Applications/19.08'. Adjust lyrics to use new URLs. This uses TLS to secure the lookup, avoids a required redirect that was impacting the lookup, and pulls out repeated uses of the same URL into a central constant. See !9, which can be closed. FIXED-IN:19.08 M +9-6lyricswidget.cpp https://invent.kde.org/kde/juk/commit/d161bae4530bc28871c6951939100bf7a7a3b924 -- You are receiving this mail because: You are watching all bug changes.
[juk] [Bug 409936] every start asks for "folder where you keep your music"
https://bugs.kde.org/show_bug.cgi?id=409936 Michael Pyne changed: What|Removed |Added Latest Commit||98ce99c2d7502a9eb2f0b96562e ||d68fb0c293276 Resolution|--- |FIXED Status|REPORTED|RESOLVED Version Fixed In||19.08 --- Comment #7 from Michael Pyne --- Guo Yunhe has, I think, fixed the most immediate issue (opening folder list even when folders have been selected), so closing this bug. If there are filename encoding issues I would open it as a separate bug, please. -- You are receiving this mail because: You are watching all bug changes.
[kdesrc-build] [Bug 315105] kdesrc-build should treat "obvious" dependencies as implicit
https://bugs.kde.org/show_bug.cgi?id=315105 --- Comment #2 from Michael Pyne --- Hi Aasif! We recently had a new contributor add a new dependency resolver to kdesrc-build which also helps set the right conditions for treating non-KDE modules as dependencies. Please see the long thread at https://invent.kde.org/kde/kdesrc-build/merge_requests/6 as a starter. You might also look at https://invent.kde.org/kde/kdesrc-build/tree/master/doc/source-reference which has a little bit of detail on kdesrc-build internals. Depending on your waking hours I'm also sometimes available on Freenode IRC in #kde-devel as the user "mpyne". I'm typically there from 1900-2300 New York time during the work week and somewhat longer on weekends. -- You are receiving this mail because: You are watching all bug changes.