[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#735934: marked as done (python-plwm: WM_STATE after programmatic de-iconify)



Your message dated Tue, 03 Sep 2019 10:59:36 +0000
with message-id <[🔎] E1i56XA-000Gam-Gc@fasolo.debian.org>
and subject line Bug#939320: Removed package(s) from unstable
has caused the Debian Bug report #735934,
regarding python-plwm: WM_STATE after programmatic de-iconify
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
735934: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=735934
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: python-plwm
Version: 2.6a+20080530-1.1
Severity: minor
File: /usr/share/pyshared/plwm/wmanager.py

If a window is programmatically de-iconified in the usual ICCCM way then
the WM_STATE is left at IconicState whereas I expected it to be
NormalState.

This can be seen with /usr/share/doc/python-plwm/examples/petliwm.py, an
xterm, and the xwit program to iconify and de-iconify,

    xwit -iconify -id $WINDOWID; sleep 2; xwit -pop -id $WINDOWID
    xprop -id $WINDOWID WM_STATE

The xterm is hidden and restored but xprop prints

    WM_STATE(WM_STATE):
                    window state: Iconic
                    icon window: 0x0

where I expected "window state: Normal".


Maybe wmanager.py handle_map_request() would use deiconify() instead of
map(), per below.  That seems to work for me.

(That MapReqest is the way a client asks for Iconic -> Normal, per
/usr/share/doc/xorg-docs/icccm/icccm.txt.gz under "Changing Window
State":
  * Iconic -> Normal - The client should map the window. The contents of
    WM_HINTS.initial_state are irrelevant in this case.


--- wmanager.py.orig	2014-01-18 14:40:18.000000000 +1100
+++ wmanager.py	2014-01-18 14:52:30.000000000 +1100
@@ -1156,7 +1156,7 @@
             # already mapped window, map it if the user allows it
             w = self.get_client(event.window)
             if self.allow_self_changes(w):
-                w.map()
+                w.deiconify()
 
     def event_to_change(self, event):
         change = {}

-- System Information:
Debian Release: jessie/sid
  APT prefers unstable
  APT policy: (990, 'unstable')
Architecture: i386 (i686)

Kernel: Linux 2.6.32-5-686 (SMP w/1 CPU core)
Locale: LANG=en_AU.utf8, LC_CTYPE=en_AU.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages python-plwm depends on:
ii  python          2.7.5-5
ii  python-support  1.0.15
ii  python-xlib     0.14+20091101-1

python-plwm recommends no packages.

python-plwm suggests no packages.

-- no debconf information

-- debsums errors found:
debsums: changed file /usr/share/pyshared/plwm/wmanager.py (from python-plwm package)

--- End Message ---
--- Begin Message ---
Version: 2.6a+20080530-2+rm

Dear submitter,

as the package python-plwm has just been removed from the Debian archive
unstable we hereby close the associated bug reports.  We are sorry
that we couldn't deal with your issue properly.

For details on the removal, please see https://bugs.debian.org/939320

The version of this package that was in Debian prior to this removal
can still be found using http://snapshot.debian.org/.

This message was generated automatically; if you believe that there is
a problem with it please contact the archive administrators by mailing
ftpmaster@ftp-master.debian.org.

Debian distribution maintenance software
pp.
Scott Kitterman (the ftpmaster behind the curtain)

--- End Message ---

Reply to: