Bug#822246: ITP: qt5ct -- Qt5 configuration tool
On 02.09.2017 at 10:32 +0200, James Lu wrote: Hi again, With some experimentation, I replaced the contents of /etc/X11/Xsession.d/99qt5ct with the following: # Enable qt5ct by default unless another PLATFORMTHEME option is set # or KDE is running. if [ -z "$QT_QPA_PLATFORMTHEME" ] && [ "$XDG_CURRENT_DESKTOP" != "KDE" ] then export QT_QPA_PLATFORMTHEME=qt5ct fi This seemed to work: the end result is that qt5ct is enabled on Xfce, GNOME and Budgie, but not enabled on Plasma 5. I also went ahead and removed the QT_PLATFORMTHEME and QT_PLATFORM_PLUGIN defines because they aren't necessary based on the upstream readme and my own testing. A git-style patch with this change is attached. Best, James On 01/09/17 07:31 AM, Lisandro Damián Nicanor Pérez Meyer wrote: On 27 August 2017 at 17:03, James Lu wrote: Hi everyone, From what I read at https://forum.antergos.com/topic/6487/qt5ct-has-priority-over-standard-plasma-settings-app/2, Lisandro's concerns are correct and qt5ct does appear to override Plasma's settings entirely. However, since qt5ct requires an Xsession file[1] to be enabled by default, could one solution be having the script exit silently if it finds any KDE environment variables? (This is assuming that the order at which vars are set make this possible, but I haven't verified that) This is definitely something to try, yes. Alternatively, would it make sense for qt5ct to Break or Conflict with Plasma, at least until these issues are worked around? And a nice way around it in case the above fails. If you are interested in trying this I'll be more than happy to sponsor you. Dmitry/team: any comments from you? Merged: https://github.com/mati75/qt5ct/commit/bfee1ceabefe20ffb8a6086f1f4cf0f26632e483 Here are sources: https://mentors.debian.net/debian/pool/main/q/qt5ct/qt5ct_0.33-1.dsc -- .''`. Mateusz Łukasik : :' : https://l0calh0st.pl `. `' Debian Member - mat...@linuxmint.pl `-GPG: D93B 0C12 C8D0 4D7A AFBC FA27 CCD9 1D61 11A0 6851
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
Hi again, With some experimentation, I replaced the contents of /etc/X11/Xsession.d/99qt5ct with the following: # Enable qt5ct by default unless another PLATFORMTHEME option is set # or KDE is running. if [ -z "$QT_QPA_PLATFORMTHEME" ] && [ "$XDG_CURRENT_DESKTOP" != "KDE" ] then export QT_QPA_PLATFORMTHEME=qt5ct fi This seemed to work: the end result is that qt5ct is enabled on Xfce, GNOME and Budgie, but not enabled on Plasma 5. I also went ahead and removed the QT_PLATFORMTHEME and QT_PLATFORM_PLUGIN defines because they aren't necessary based on the upstream readme and my own testing. A git-style patch with this change is attached. Best, James On 01/09/17 07:31 AM, Lisandro Damián Nicanor Pérez Meyer wrote: > On 27 August 2017 at 17:03, James Lu wrote: >> Hi everyone, >> >> From what I read at >> https://forum.antergos.com/topic/6487/qt5ct-has-priority-over-standard-plasma-settings-app/2, >> Lisandro's concerns are correct and qt5ct does appear to override >> Plasma's settings entirely. However, since qt5ct requires an Xsession >> file[1] to be enabled by default, could one solution be having the >> script exit silently if it finds any KDE environment variables? (This is >> assuming that the order at which vars are set make this possible, but I >> haven't verified that) > > This is definitely something to try, yes. > >> Alternatively, would it make sense for qt5ct to Break or Conflict with >> Plasma, at least until these issues are worked around? > > And a nice way around it in case the above fails. > > If you are interested in trying this I'll be more than happy to sponsor you. > > Dmitry/team: any comments from you? > > From d0738c2146e02a5dd553f1d96ba37b506f6e8727 Mon Sep 17 00:00:00 2001 From: James Lu Date: Sat, 2 Sep 2017 01:29:41 -0700 Subject: [PATCH] Don't enable qt5ct on KDE, or when another platform theme is set --- debian/99qt5ct | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/debian/99qt5ct b/debian/99qt5ct index 0362b72..32d7105 100644 --- a/debian/99qt5ct +++ b/debian/99qt5ct @@ -1,3 +1,6 @@ -export QT_QPA_PLATFORMTHEME=qt5ct -export QT_PLATFORMTHEME=qt5ct -export QT_PLATFORM_PLUGIN=qt5ct +# Enable qt5ct by default unless another PLATFORMTHEME option is set +# or KDE is running. +if [ -z "$QT_QPA_PLATFORMTHEME" ] && [ "$XDG_CURRENT_DESKTOP" != "KDE" ] +then +export QT_QPA_PLATFORMTHEME=qt5ct +fi -- 2.14.1 signature.asc Description: OpenPGP digital signature
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
Hi everyone, From what I read at https://forum.antergos.com/topic/6487/qt5ct-has-priority-over-standard-plasma-settings-app/2, Lisandro's concerns are correct and qt5ct does appear to override Plasma's settings entirely. However, since qt5ct requires an Xsession file[1] to be enabled by default, could one solution be having the script exit silently if it finds any KDE environment variables? (This is assuming that the order at which vars are set make this possible, but I haven't verified that) Alternatively, would it make sense for qt5ct to Break or Conflict with Plasma, at least until these issues are worked around? Best, James [1]: https://github.com/mati75/qt5ct/blob/master/debian/99qt5ct signature.asc Description: OpenPGP digital signature
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
I would really prefer if you didn't, at least not before you coordinate with us the Qt/KDE team. By the way qt5ct works it would probably interfere with Plasma and it's still another package using qt's private headers. -- Lisandro Damián Nicanor Pérez Meyer http://perezmeyer.com.ar/ http://perezmeyer.blogspot.com/ El abr 23, 2017 1:33 PM, "Mateusz Łukasik" escribió: Control: owner mat...@linuxmint.pl After year without any changes I go to upload this package to repository with myself as maintainer and Lisandro as uploader. I have really massive spam for request this package in repository since google show my github repository as second result. -- .''`. Mateusz Łukasik : :' : https://l0calh0st.pl `. `' Debian Member - mat...@linuxmint.pl `-GPG: D93B 0C12 C8D0 4D7A AFBC FA27 CCD9 1D61 11A0 6851
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
Control: owner mat...@linuxmint.pl After year without any changes I go to upload this package to repository with myself as maintainer and Lisandro as uploader. I have really massive spam for request this package in repository since google show my github repository as second result. -- .''`. Mateusz Łukasik : :' : https://l0calh0st.pl `. `' Debian Member - mat...@linuxmint.pl `-GPG: D93B 0C12 C8D0 4D7A AFBC FA27 CCD9 1D61 11A0 6851
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
On 23.04.2016 18:14 +0200, Lisandro Damián Nicanor Pérez Meyer wrote: The packaging is ready, but I would like to see what to do with it as requiring private headers is really not agood thing. -- Lisandro Damián Nicanor Pérez Meyer http://perezmeyer.com.ar/ http://perezmeyer.blogspot.com/ I understand. Here is: https://github.com/mati75/qt5ct my sources for Debian to this package. Go ahead if you want something copy. -- .''`. Mateusz Łukasik : :' : http://mati75.eu `. `' Debian Member - mat...@linuxmint.pl `-GPG: D93B 0C12 C8D0 4D7A AFBC FA27 CCD9 1D61 11A0 6851
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
On 22.04.2016 14:58 +0200, Lisandro Damián Nicanor Pérez Meyer wrote: Package: wnpp Severity: wishlist Owner: "Lisandro Damián Nicanor Pérez Meyer" * Package name: qt5ct Version : 0.22 Upstream Author : Ilya Kotov , * URL : https://sourceforge.net/projects/qt5ct * License : BSD-2-clause Programming Lang: C++ Description : Qt5 configuration tool This program allows users to configure Qt5 settings like theme, fonts and icons under a desktop envrionment/window manager without a suitable Qt platform plugin. Hi, I can help with that. -- .''`. Mateusz Łukasik : :' : http://mati75.eu `. `' Debian Member - mat...@linuxmint.pl `-GPG: D93B 0C12 C8D0 4D7A AFBC FA27 CCD9 1D61 11A0 6851
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
The packaging is ready, but I would like to see what to do with it as requiring private headers is really not agood thing. -- Lisandro Damián Nicanor Pérez Meyer http://perezmeyer.com.ar/ http://perezmeyer.blogspot.com/
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
I'm afraid this app requires Qt's private headers, so I'm reluctant to add it. -- "A computer is like an air conditioner. It stops working when you open windows." Lisandro Damián Nicanor Pérez Meyer http://perezmeyer.com.ar/ http://perezmeyer.blogspot.com/ signature.asc Description: This is a digitally signed message part.
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
On Friday 22 April 2016 16:35:43 Dmitry Shachnev wrote: [snip] > Just a small note: this tool overrides QT_QPA_PLATFORMTHEME which means i.e. > that Plasma integration may not function properly with this tool. Indeed it does. This is material for the package description and README.Debian then. -- ...man had always assumed that he was more intelligent than dolphins because he had achieved so much -- the wheel, New York, wars and so on -- whilst all the dolphins had ever done was muck about in the water having a good time. But conversely, the dolphins had always believed that they were far more intelligent than man -- for precisely the same reasons. Douglas Adams, "The hitchhikers' guide to the galaxy" Lisandro Damián Nicanor Pérez Meyer http://perezmeyer.com.ar/ http://perezmeyer.blogspot.com/ signature.asc Description: This is a digitally signed message part.
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
On Friday 22 April 2016 16:18:02 Konstantin Khomoutov wrote: [snip] > To my knowledge, these packages doing the same task for older Qt > versions present in the archive are named "qt3-qtconfig" and > "qt4-qtconfig", so I'd expect this one to be named "qt5-qtconfig". But qtx-config where provided by upstream whereas this one isn't, so I don't think this is a good idea in case upstream decides to re ship the app (it's still possible). Thanks non the less for the thought :) -- The reasonable man adapts himself to the world; the unreasonable one persists in trying to adapt the world to himself. Therefore all progress depends on the unreasonable man. George Bernard Shaw Lisandro Damián Nicanor Pérez Meyer http://perezmeyer.com.ar/ http://perezmeyer.blogspot.com/ signature.asc Description: This is a digitally signed message part.
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
Hi Lisandro, On Fri, Apr 22, 2016 at 09:58:16AM -0300, Lisandro Damián Nicanor Pérez Meyer wrote: > Package: wnpp > Severity: wishlist > Owner: "Lisandro Damián Nicanor Pérez Meyer" > > * Package name: qt5ct > Version : 0.22 > Upstream Author : Ilya Kotov , > > * URL : https://sourceforge.net/projects/qt5ct > * License : BSD-2-clause > Programming Lang: C++ > Description : Qt5 configuration tool > > This program allows users to configure Qt5 settings like theme, fonts and > icons under a desktop envrionment/window manager without a suitable Qt > platform plugin. Just a small note: this tool overrides QT_QPA_PLATFORMTHEME which means i.e. that Plasma integration may not function properly with this tool. -- Dmitry Shachnev signature.asc Description: PGP signature
Bug#822246: ITP: qt5ct -- Qt5 configuration tool
Package: wnpp Severity: wishlist Owner: "Lisandro Damián Nicanor Pérez Meyer" * Package name: qt5ct Version : 0.22 Upstream Author : Ilya Kotov , * URL : https://sourceforge.net/projects/qt5ct * License : BSD-2-clause Programming Lang: C++ Description : Qt5 configuration tool This program allows users to configure Qt5 settings like theme, fonts and icons under a desktop envrionment/window manager without a suitable Qt platform plugin.