Bug#822246: ITP: qt5ct -- Qt5 configuration tool

2017-09-02 Thread Mateusz Łukasik

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

2017-09-02 Thread James Lu
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

2017-08-27 Thread James Lu
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

2017-04-23 Thread Lisandro Damián Nicanor Pérez Meyer
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

2017-04-23 Thread Mateusz Łukasik

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

2016-04-23 Thread Mateusz Łukasik

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

2016-04-23 Thread Mateusz Łukasik

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

2016-04-23 Thread Lisandro Damián Nicanor Pérez Meyer
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

2016-04-22 Thread Lisandro Damián Nicanor Pérez Meyer
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

2016-04-22 Thread Lisandro Damián Nicanor Pérez Meyer
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

2016-04-22 Thread Lisandro Damián Nicanor Pérez Meyer
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

2016-04-22 Thread Dmitry Shachnev
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

2016-04-22 Thread Lisandro Damián Nicanor Pérez Meyer
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.