http://bugs.winehq.org/show_bug.cgi?id=30944
Bug #: 30944 Summary: Regression: Tray icon for furcadia is invisible and non-responsive Product: Wine Version: 1.3.7 Platform: x86 OS/Version: Linux Status: UNCONFIRMED Severity: minor Priority: P2 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: lethalfalcon@endlessrealities.com Classification: Unclassified Regression SHA1: 5d570706c6aca9acbff855c3ae497b35bdde4648
Starting from version 1.3.7, Furcadia's pounce icon disappears from the tray. The space is allocated to it, but the actual icon isn't visible, nor does right-clicking in the space do anything. The icon disappears as well.
Running a bisect results in this patch being the culprit:
5d570706c6aca9acbff855c3ae497b35bdde4648 is the first bad commit commit 5d570706c6aca9acbff855c3ae497b35bdde4648 Author: Alexandre Julliard julliard@winehq.org Date: Mon Nov 8 15:28:09 2010 +0100
winex11: Create Win32 windows corresponding to all the ancestors of embedded windows.
This is needed to keep track of the embedded window position by handling ConfigureNotify received by ancestors.
:040000 040000 7c3171d11930ae3e1dd7965245f6a90a2b115c7f c30937343ef6532a80879e808f118219e267030c M dlls z
If I revert this patch in master, the icon shows up. However, the right-click menu appears at the top of my screen instead of right above the icon as expected. I can assume this is due to one or more of the hunks that failed to patch.
I'm not sure what channels I should enable to get a useful log. I tried x11drv, but didn't see anything other than traces, and they looked very similar between versions.
Tested using a new profile and an existing profile with identical results. No extra winetricks used on the new profile.
This may be related to bug 24719, but the version reported in there is earlier than where I find the issue, so not sure.
http://bugs.winehq.org/show_bug.cgi?id=30944
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression CC| |julliard@winehq.org Summary|Regression: Tray icon for |Tray icon for furcadia is |furcadia is invisible and |invisible and |non-responsive |non-responsive
http://bugs.winehq.org/show_bug.cgi?id=30944
--- Comment #1 from Alexandre Julliard julliard@winehq.org 2012-06-19 03:22:40 CDT --- Please test with Wine 1.4, and if still present specify what window manager you are using and how to reproduce the problem.
http://bugs.winehq.org/show_bug.cgi?id=30944
--- Comment #2 from Joshua Gardynik lethalfalcon@endlessrealities.com 2012-06-19 09:25:56 CDT --- It's been tested using 1.5.5 and git's master branch, where it's still broken unless I revert this patch. Do you really want me to test every single version from 1.4.0 to 1.5.5 to see which versions break or not from then 'til now?
I'm using fluxbox 1.3.2 as a window manager.
To reproduce, just start furcadia and select a character (you'll have to create one on first run). Furcadia pounce starts automatically when the character loads. As an alternative, you can try running furc_on.exe (the pounce program) directly instead, but I'm not sure if it will let you with no characters. If it does run, that's all you have to do, since the icon is part of it.
http://bugs.winehq.org/show_bug.cgi?id=30944
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |UPSTREAM
--- Comment #3 from Alexandre Julliard julliard@winehq.org 2012-06-19 09:58:13 CDT --- (In reply to comment #2)
It's been tested using 1.5.5 and git's master branch, where it's still broken unless I revert this patch. Do you really want me to test every single version from 1.4.0 to 1.5.5 to see which versions break or not from then 'til now?
No, I thought you tested with 1.3.7 since that's what the version field said.
I'm using fluxbox 1.3.2 as a window manager.
To reproduce, just start furcadia and select a character (you'll have to create one on first run). Furcadia pounce starts automatically when the character loads. As an alternative, you can try running furc_on.exe (the pounce program) directly instead, but I'm not sure if it will let you with no characters. If it does run, that's all you have to do, since the icon is part of it.
Thanks. It works fine here with metacity, kwin, xfce, and stalonetray, so it's most likely a bug in fluxbox.
http://bugs.winehq.org/show_bug.cgi?id=30944
Jerome Leclanche adys.wh@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |adys.wh@gmail.com
--- Comment #4 from Jerome Leclanche adys.wh@gmail.com 2012-06-19 10:40:36 CDT --- If you choose to file an issue with fluxbox, please leave the URL to the bug here or in the See Also field.
http://bugs.winehq.org/show_bug.cgi?id=30944
--- Comment #5 from Joshua Gardynik lethalfalcon@endlessrealities.com 2012-06-19 11:49:51 CDT --- I don't want to start a blame game, but I would like to understand what this patch actually does, and why it alone is responsible for icons no longer showing up in fluxbox. Maybe it's a flux issue, but it works fine in flux at wine 1.3.6, and it ALMOST works again if I revert this patch, even with git master, and nothing else seems to be functionally degraded with the reversion either, so what was this patch designed to actually address? The commit comment doesn't seem like it matches with the issue at hand, to me anyway.
Fluxbox's development is... honestly, nowhere near as active as wine. I did find a bug open from over a year ago here:
https://sourceforge.net/tracker/index.php?func=detail&aid=3311806&gr...
It that appears to be the same issue. If I can find time, maybe I can go through the patch piecemeal and try to isolate the issue, but I'm not a c++ developer by any stretch, so I'll be flying rather blind. Luckily, it's only 3 files, so diffing them won't take a horribly long time. It's just having to rebuild after each one that kinda sucks.
http://bugs.winehq.org/show_bug.cgi?id=30944
--- Comment #6 from Alexandre Julliard julliard@winehq.org 2012-06-19 12:02:46 CDT --- The patch fixes the tracking of the window location, which as you noticed is broken when you revert it.
http://bugs.winehq.org/show_bug.cgi?id=30944
--- Comment #7 from Alexandre Julliard julliard@winehq.org 2012-06-19 12:25:46 CDT --- So the problem is that fluxbox sends a synthetic ConfigureNotify in window coordinates instead of root coordinates as per ICCCM:
trace:event:call_event_handler 47 ConfigureNotify for hwnd/window 0x100ea/1000002 trace:event:X11DRV_ConfigureNotify win 0x100ea/1000002 new X rect 0,0,18x18 (event 0,0,18x18) trace:event:X11DRV_ConfigureNotify 0x100ea moving from (0,0) to (-1900,-1180)
When mapped from root to window coords it is far off-screen, so the icon doesn't get displayed. Definitely a fluxbox bug.
http://bugs.winehq.org/show_bug.cgi?id=30944
--- Comment #8 from Joshua Gardynik lethalfalcon@endlessrealities.com 2012-06-19 13:08:12 CDT --- Cool, thank you very much for looking into it a little more! I'll pass that on, and if necessary, maybe I'll tear apart flux's tray to see if I can figure out how to make that change. Probably won't be any easier, but at least I know now not to blame wine at all. :)
https://bugs.winehq.org/show_bug.cgi?id=30944
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://sourceforge.net/tra | |cker/index.php?func=detail& | |aid=3311806&group_id=35398& | |atid=413960
https://bugs.winehq.org/show_bug.cgi?id=30944
Frédéric Delanoy frederic.delanoy@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED CC| |frederic.delanoy@gmail.com
--- Comment #9 from Frédéric Delanoy frederic.delanoy@gmail.com --- Closing
https://bugs.winehq.org/show_bug.cgi?id=30944
--- Comment #10 from Matteo Bruni matteo.mystral@gmail.com --- (In reply to Frédéric Delanoy from comment #9)
Closing
Looking at the linked bug report, this one might actually be fixed. Has anyone verified that's the case?
https://bugs.winehq.org/show_bug.cgi?id=30944
--- Comment #11 from Frédéric Delanoy frederic.delanoy@gmail.com --- (In reply to Matteo Bruni from comment #10)
(In reply to Frédéric Delanoy from comment #9)
Closing
Looking at the linked bug report, this one might actually be fixed. Has anyone verified that's the case?
Apparently fixed according to https://github.com/fluxbox/fluxbox/pull/19
-
wine-bugs＠winehq.org