[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-08 Thread Mathew Hodson
** No longer affects: qtbase-opensource-src (Ubuntu) ** Changed in: qjackctl (Ubuntu) Importance: Critical => Medium ** Changed in: ubuntustudio Status: New => Fix Released -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-07 Thread Dmitry Shachnev
> @Dmitry, I still require: > export XDG_CURRENT_DESKTOP=Unity > To make the icon show. Is this being fixed in other places? I hope I'll be able to get the fix into Qt before Xenial release (or in the worst case as an early SRU). So no need to add the workaround. -- You received this bug

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-07 Thread Len Ovens
I meant to add would be possible (advisable) to change the qjackctl.desktop file to use: export XDG_CURRENT_DESKTOP=Unity; qjackctl as the executable line as a temporary work around? -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-07 Thread Len Ovens
@Dmitry, I still require: export XDG_CURRENT_DESKTOP=Unity To make the icon show. Is this being fixed in other places? Also: the new version has changed the name of one of the settings that needs to be set by default for thing s to work ootb. see Bug #1567589 -- You received this bug

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-07 Thread Launchpad Bug Tracker
This bug was fixed in the package qjackctl - 0.4.2-0ubuntu1 --- qjackctl (0.4.2-0ubuntu1) xenial; urgency=medium * New upstream minor release. - Partially fixes issues with systray icon (LP: #1546328). * Refresh 90_dbus_setting.patch. -- Dmitry Shachnev

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-06 Thread Dmitry Shachnev
The new release is a minor/bugfix one, so I'll upload it now. -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to qtbase-opensource-src in Ubuntu. https://bugs.launchpad.net/bugs/1546328 Title: Systray option does not work.

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-06 Thread Len Ovens
Rui has announced a new version of qjackctl on LAA, LAD,LAU mailing lists. One of the updates/changes is to remove Start minimized to system tray as an option. There are some other fixes for indicator menu, but they say they are kde based, so I do not know if they will also work for xfce. I am no

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-03 Thread Len Ovens
@Dmitry, @Ross: I do remember qjackctl from before only allowing singltons :) one could close qjackctl to the systray and start it again from the menu with the result of two qjackctls. That while annoying, is not a real problem provided the user is using jackdbus as it is ok to have more than one

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-03 Thread Ross Gammon
Whoops. The new ticket upstream: https://sourceforge.net/p/qjackctl/tickets/27/ -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to qtbase-opensource-src in Ubuntu. https://bugs.launchpad.net/bugs/1546328 Title: Systray option

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-03 Thread Ross Gammon
OK - that re-factoring (Single/unique application instance control adapted to Qt5/X11) was listed in the release notes to 0.4.1. I am not sure that allowing more than one Qjackctl instance is a good idea. Some audio applications require JACK to be running, and other start it themselves. This may

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-03 Thread Dmitry Shachnev
Looks like the purpose of that code is to make the application singleton. There are lots of solutions for this task that do not require X11. Here is a very small library that qjackctl can probably use or embed: https://github.com/itay-grudev/SingleApplication. As a quick and temporary fix, maybe

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-03 Thread Ross Gammon
Thanks Dmitry for spending so much time on this. I had been meaning to create a separate bug for the freezing behaviour, and to try and get some debug output from X to send upstream to Qjackctl. But I had been busy, and it looks like you have found the issue anyway. Not knowing the X11 API nor

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-03 Thread Dmitry Shachnev
@Len, I could not reproduce it on my own system with Xfce installed, but I can now reproduce it on Xubuntu daily CD. According to D-Bus monitor, qjackctl doesn't even create a tray icon before everything hangs. Looking at the code, I suppose such behavior may be a result of this call in

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-02 Thread Len Ovens
@Dmitry, Sorry for not being clearer. "> - Setting "Start minimized to system tray" makes desktop unusable. Mouse moves, but no input is possible. One has to use c/a/f1 to a vterm in order to kill qjackctl to make the desktop usable again. " After setting "Start minimized to system tray", exit

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-02 Thread Dmitry Shachnev
@Len, > - The qjackctl menu does not work, instead an empty drop-down appears. (for the reasons you already mentioned) Right. I cannot do anything with this in Qt, qjackctl should really use the recommended APIs (i.e. QSystemTrayIcon::setContextMenu)… > - Setting "Start minimized to system

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-01 Thread Len Ovens
@Dmitry, doing: $ export XDG_CURRENT_DESKTOP=Unity $ qjackctl Does make the icon behave as expected. Good. However, the rest of the problems still exist: - The qjackctl menu does not work, instead an empty drop-down appears. (for the reasons you already mentioned) - Setting "Start minimized

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-01 Thread Dmitry Shachnev
The mentioned fix is https://codereview.qt-project.org/154459. If it gets accepted upstream, I'll try to make sure it gets included in Xenial (or at least Xenial-updates). -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-04-01 Thread Dmitry Shachnev
Dear Ross, >> Without appmenu-qt5, the menu works from the systray, but the icon is not >> animated. It just shows a red cross all the time, which is misleading for >> the user. > > Thanks for letting me know that, I will try to look at this issue a bit later. Can you please check if starting

Re: [Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-31 Thread Kaj Ailomaa
On Tue, Mar 22, 2016, at 04:48 PM, Ross Gammon wrote: > Disabling the system tray completely seems a bit of a backwards step. > > Like you, I am no QT expert. Although it is on my to-do list to learn. > So if anyone can help identify what makes Ubuntu Studio freeze if > Qjackctl was minimised

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-22 Thread Ross Gammon
Disabling the system tray completely seems a bit of a backwards step. Like you, I am no QT expert. Although it is on my to-do list to learn. So if anyone can help identify what makes Ubuntu Studio freeze if Qjackctl was minimised before upgrade, it would be appreciated. Otherwise, uploading the

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-20 Thread Len Ovens
Hmm, maybe for now just set ./configure --enable-system-tray=no or --disable-system-tray Honestly, I use the system tray settings with qjackctl all the time. I just don't have the screen real estate to deal with it. Screen 1 has the editor window and meter bridge and screen 2 has the mixer panel

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-20 Thread Len Ovens
Ross: "we would have to advise users to manually edit their conf file to set StartMinimised to false." They may not be able to. A not uncommon use of qjackctl is to have it set to start with session in order to run jack at that time. So the user upgrades, reboots and finds they can't do

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-20 Thread Ross Gammon
@ Len: Thanks for testing that. I didn't think to try leaving minimised first. I have to admit that uploading the version that builds with QT4 is my current favourite. It is a whole lot neater and has zero problems for the user as far as I can tell. But if we are forced to upload the QT5

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-20 Thread Dmitry Shachnev
@Ross: > Without appmenu-qt5, the menu works from the systray, but the icon is not animated. It just shows a red cross all the time, which is misleading for the user. Thanks for letting me know that, I will try to look at this issue a bit later. The goal is to make Qt's own dbustray

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-19 Thread Len Ovens
Note: The problem is starting qjackctl when "Start minimized to tray" has already select on a previous run. -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to qtbase-opensource-src in Ubuntu.

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-19 Thread Len Ovens
Reboot did get me a nice icon, but running qjackctl with "Start minimized to tray" still locks all screen input. It is interesting that with "Enable system tray icon" selected it is fine, I can start and stop with no problems. It is the Start minimized to tray option that causes trouble. Perhaps

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-19 Thread Len Ovens
Ross: Downloaded and installed qjackctl_0.4.1-1ubuntu2~ppa3_i386.deb from https://launchpad.net/~ubuntustudio-dev/+archive/ubuntu/dev- testing/+packages Setup->misc->Enable system tray icon seems to work Minus the icon... stop and then start works ok too. Setup->misc->Start minimized to system

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-19 Thread Dmitry Shachnev
I don't like Depends: appmenu-qt5. Can it be a recommendation? We want to drop appmenu-qt5 at some point. If it won't happen in Xenial, it will definitely happen in Xenial+1. -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-19 Thread Ross Gammon
A new debdiff with appmenu-qt5 demoted to "recommends". If appmenu-qt5 disappears from the archive, Qjackctl is still useable for people that minimise to the system tray (via the drop down indicator menu), but the icon shown in the tray is a red cross instead of reflecting the "stopped"/"started"

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-19 Thread Ross Gammon
@ Dmitry. I was just reading the discussion in Ubuntu-devel and wondering what to do. Without appmenu-qt5, the menu works from the systray, but the icon is not animated. It just shows a red cross all the time, which is misleading for the user. So I guess we could just recommend it, and then it

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-18 Thread Ross Gammon
I now upload a new debdiff which rather than dropping back to build with QT4, instead cherry-picks some upstream commits to get the systray menu working, and depend upon appmenu-qt5 to get the systray icon showing. The changes can be tested by upgrading to the verision uploaded to the Ubuntu

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-18 Thread Len Ovens
If not having a package breaks an app that package would be a depend by definition... or am I missing something? This small application, Qjackctl, is a major part of Ubuntustudio as the app that allows the user to control jackd/jackdbus from a gui. With out proper systray operation the user may

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-08 Thread Ross Gammon
Thanks Dmitry. That really helps make it clear. I have updated the upstream bug with the analysis. So it appears we have a choice: 1. Go with the original proposal (patch attached) and build qjackctl with QT4 for Xenial 2. Cherry pick two of the upstream commits and try and get it working that

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-03 Thread Dmitry Shachnev
A small typo in my previous comment: it doesn't *populate* the tray icon's contextMenu(). -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to qtbase-opensource-src in Ubuntu. https://bugs.launchpad.net/bugs/1546328 Title:

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-03 Thread Dmitry Shachnev
Actually there are two problems here. 1) The menu I have taken a look at qjackctl code. It makes some really wrong assumptions about how the system tray icons work. In particular, it doesn't call the tray icon's contextMenu(), but instead intercepts the events and tries to draw a custom menu.

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-02 Thread Mathew Hodson
** Also affects: ubuntustudio Importance: Undecided Status: New -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to qtbase-opensource-src in Ubuntu. https://bugs.launchpad.net/bugs/1546328 Title: Systray option does

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-02 Thread Ross Gammon
Thanks Dmitry. I can confirm that we have been testing with up-to-date Xenial boxes with the latest QT5. I just installed appmenu-QT5 and it makes things slightly better in that the icon in the tray is dynamic, but the menu that drops down from it is empty. -- You received this bug

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-03-02 Thread Ross Gammon
** Attachment added: "Qjackctl with QT5 and appmenu-qt5 installed" https://bugs.launchpad.net/ubuntu/+source/qjackctl/+bug/1546328/+attachment/4586886/+files/VirtualBox_UbuntuStudioXenial1604_02_03_2016_19_40_05.png -- You received this bug notification because you are a member of Ubuntu

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-02-29 Thread Dmitry Shachnev
@Ross: The difference between Debian and Xubuntu is that the latter has an indicators-based tray, so Qt tries to use that too. Anyway, have you tested if the bug still occurs with the latest qtbase upload? If that is the case, I will try to look at it. As a temporary workaround, try installing

[Touch-packages] [Bug 1546328] Re: Systray option does not work.

2016-02-29 Thread Ross Gammon
The Ubuntu changes to qtbase-opensource-src 5.5.1 do seem to include backports from the 5.6 branch which may be System Tray related: https://launchpadlibrarian.net/243321175/qtbase-opensource-src_5.5.1+dfsg-14ubuntu1_5.5.1+dfsg-14ubuntu2.diff.gz But I am not sure. I will add a task to see if