[xmonad] Dropped patches

Gwern Branwen gwern0 at gmail.com
Thu Aug 28 16:45:50 EDT 2008

OK, since we aren't applying any patches until after 0.8, I thought I would make a list for when we do. This is a long list, so I hope people read it through (it certainly took long enough to compile).

9 patches outstanding. Let's go through the interesting ones:

* "add warpGeometry field to XConf"
Applicable. No comments on it. Useful? I don't know Mirror so I can't say.

* "XDG_CONFIG_HOME support"
Applicable. Commented upon. The discussion never came to any conclusion I can see. The 5 non-patch emails discussing it:
** sjanssen, opposed (or at least skeptical): <http://www.haskell.org/pipermail/xmonad/2008-May/005765.html>
** dons, noncommittal: <http://www.haskell.org/pipermail/xmonad/2008-May/005770.html>
** Brandon Allbery, in favor of (?): <http://www.haskell.org/pipermail/xmonad/2008-May/005771.html>
** Devin Mullins, opposed: <http://www.haskell.org/pipermail/xmonad/2008-May/005772.html>
** Isaac Jones, in favor of: <http://www.haskell.org/pipermail/xmonad/2008-May/005774.html>.
** the patch author, in favor of (obviously)

So: 1 neutral, 2 opposed, 3 in favor of. I don't know much about this, but the code change looks fairly clear to me, and the one real disadvantage adduced ('This has one disadvantage I can think of: for DEs that set $XDG
automatically, users will have to know about it in order to know where
to put their .xmonad.hs -- otherwise the file won't be detected.
Perhaps that's not worth worrying about.') doesn't sound terribly serious?

* "darcs patch: pass mouse clicks on to focused windows (experimental)"
Applicable. No comments on it. Marked experimental; perhaps Lukas Mai can comment on whether this is still good?

* "Use threadWaitRead to avoid blocking in nextEvent in the main loop"
Allbery commented with advice <http://www.haskell.org/pipermail/xmonad/2008-June/005897.html>, and Adam Sampson took it and apparently wrote a new & improved patch <http://www.haskell.org/pipermail/xmonad/2008-June/005899.html>. This second patch is applicable, but I see no feedback on it. A good idea? Not being able to usefully run threads because of blocking strikes me as a problem.

* "keyActions moved to XState"
Applicable. No comments. Confusingly, the second patch does not seem to exist on the mailing list, although Darcswatch has it at <http://darcswatch.nomeata.de/20080715143436-7641b-979a1f6ea132b4dceba775c8d96bb2bc8c1aa629.dpatch>.
A good idea? Looks like it; it enables a interesting looking patch to XMC <http://www.haskell.org/pipermail/xmonad/2008-July/006041.html>:
  "* XMonad.Actions.DynamicKeys: utilities to update key bindings at runtime"

Summary: 5 outstanding patches of the 9 total are worth considering; 2 or 3 look worthwhile to me.

13 patches outstanding. Let's go through the interesting ones:

* "Actions.Search: added {deb,debbts,debpts,images,isohunt} search engines"
Not applicable. No comments. I've emailed intrigeri asking him to re-record and send, since it is definitely a worthwhile patch to Action.Search. If he doesn't I probably will (assuming I don't forget).

* "Fix window region checking in UpdatePointer", "Fix window region checking in UpdatePointer (and 1 more)"
<http://www.haskell.org/pipermail/xmonad/2008-June/005843.html>, <http://www.haskell.org/pipermail/xmonad/2008-June/005845.html>
Applicable. No comments.  Joachim reported no bugs.

** "Only move pointers over managed windows"
See previous.

* "XMonad.Actions.CopyWindow runOrCopy"
<http://www.haskell.org/pipermail/xmonad/2008-June/005799.html>, patch <http://darcswatch.nomeata.de/20080602205742-20747-65c3b7fcab114e84def368c54056a8f58cadec2e.dpatch>
Applicable. I commented and criticized it a little; I think it should be applied - I don't mind grappling with the code duplication later.

* "Make prompt controlMask keybindings work when other modifiers are active"
???; patch <http://darcswatch.nomeata.de/20080608222350-18f27-aa3de81015e9eb69d4bdd91a21968d9881d80dfa.dpatch>
Not applicable. I don't know anything about this one. I include it because I don't actually know for certain that the current Prompt code does have working 'controlMask keybindings' when other modifiers are active.

* "hide the implementation type in "EwmhDesktopsLayout""
Not applicable (due to a Darcs bug?). Was a response to this thread <http://www.haskell.org/pipermail/xmonad/2008-June/thread.html#5837>, but while the other patches got applied, this one got ignored - no comments on it.

* "ManageDocks: keep struts on top"
An attachment on Google Code (?); I can't find the right bug report. Patch: <http://darcswatch.nomeata.de/20080626200455-18f27-727aac59e80057468e07299cea1f3516d2652abf.dpatch>
Applicable. No discussion that I know of.

* "darcs patch: Improvements in documentation (and 3 more)"
** "XMonad.Actions.Plane: removed unneeded hiding"
** "XMonad.Config.Gnome: using XMonad.Actions.Plane"
** "XMonad.Actions.Plane.planeKeys: function to make easier to configure"
Applicable. No discussion. (Poor Marco! Counting his Xmonad patch, that makes 5 or so patches that just got ignored.)

* "XMonad.Actions.DynamicKeys: utilities to update key bindings at runtime"
Applicable. No comments. Requires the patch to XMonad core discussed previously.

Summary: 10 or 11 patches worth considering; and 3 or 4 especially so.


* "Fixes for Ssh Module"
???; patch: <http://darcswatch.nomeata.de/20080810121734-d4c7e-893bfc8762644b1f97a488b3185ff1dbfd1cd181.dpatch>
May be applicable, but a darcs bug is triggered. I don't know where this one came from. It looks like some substantial Ssh changes though and is worth looking at?

Kvashnin OC-12 World BROMURE nitrate Marxist ASIC intelligence NAIA Defcon
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://www.haskell.org/pipermail/xmonad/attachments/20080828/ba875697/attachment.bin

More information about the xmonad mailing list