| 25 Nov 2022 |
samueldr | what kind of merging do you mean here? | 07:06:35 |
samueldr | because different "scopes" like /etc/xdg vs. ~/.config merge fine AFAIUI without manual intervention by the consumer of the config | 07:07:01 |
samueldr | e.g. kcminput, which is what I was looking at for kwin, doesn't load it from the different sources manually | 07:07:55 |
samueldr | but it does work | 07:08:02 |
samueldr | unless I totally missed something | 07:08:14 |
samueldr | though here it can't merge "dicts" from a config to another config [or the same config] | 07:08:43 |
K900 | mmm | 09:18:43 |
K900 | I just tried to update MythTV to make it build with Qt 5.15 | 09:18:55 |
K900 | https://github.com/MythTV/mythtv/blob/v32.0/mythtv/configure | 09:19:04 |
K900 | Don't you just love completely bespoke configure scripts | 09:19:12 |
K900 | And of course they have a vendored ffmpeg, which comes with another 8k lines of bespoke configure script | 09:24:31 |
K900 | OK, I give up | 09:36:18 |
K900 | Officially | 09:36:23 |
K900 | Should just drop it and alias it to Kodi | 09:37:22 |
K900 | 6head | 09:37:25 |
samueldr | https://api.kde.org/frameworks/kconfig/html/classKConfigGroup.html#a754c98908f1fa22924b8a8a987cd25ba | 22:53:22 |
samueldr |
You should not call any functions on an invalid group.
| 22:53:28 |
samueldr | so, with my understanding of the code, m_config could be invalid, m_defaultConfig valid, and it would only call functions on the invalid KConfigGroup! fun! | 22:54:03 |
samueldr | rebuilding with kwin changed sure is fun :( | 23:02:34 |
| 26 Nov 2022 |
samueldr | oh, there are other logic issues... a touchpad is a pointer device... so it's picking the pointer device group and setting it as the new default config for that device, than looking on that pointer group to find the touchpad group | 03:36:51 |
samueldr | so while [Libinput][Defaults][Touchpad] is desired, [Libinput][Defaults][Pointer][Touchpad] needs to be used if there is a Pointer defaults group! | 03:37:36 |
samueldr | so I guess I'll send that patch since now it works | 05:34:35 |
samueldr | https://gist.github.com/samueldr/085abaeaa0f35f66f8c238d7308af8e4 | 05:35:23 |
samueldr | ignore the first commit, it's not needed in the end, though I still don't get why it's checking for validity at that place... it ends-up used within defaultValue | 07:00:08 |
samueldr | and, for anyone following at home, change sent https://invent.kde.org/plasma/kwin/-/merge_requests/3244 | 07:23:19 |
K900 | https://pointieststick.com/2022/11/25/this-week-in-kde-humongous-ui-improvements/ | 07:44:14 |
K900 | Oh god they finally made Filelight usable | 07:44:24 |
K900 | https://github.com/NixOS/nixpkgs/issues/201765#issuecomment-1328019626 | 10:19:29 |
K900 | Did a small review on the leftover Qt 5.1{2,4} packages | 10:19:45 |
K900 | Basically I think there's three that may be worth fixing | 10:27:33 |