[xmonad-contrib] DarcsWatch Unapplied Patch Review

Updated version of Gwern's fine review from a while back:
Down to 15 unapplied in XMC DarcsWatch, several of which are new, and
none obsolete but possibly[1] and [2]. Hurrah!
DW is unfortunately missing at least some patches from the mailing
list. Some manually attached patches are listed and some are not;
some are listed only on the 'DarcsWatch overview > Listing by user'
page, and marked as "unmatched", e.g. the *Master patches. Not
sure what causes this, but I did have some problems with a darcs get
--partial repo while testing patches, so maybe generating patches
against different lazy versions may have something to do with it?
= Needing comments,lobbying, committer decision =
* 20080710220410: XMonad.Actions.DynamicKeys: utilities to update key
bindings at runtime by Marco TM-CM-:lio Gontijo e S

Hi, Am Freitag, den 20.02.2009, 09:36 -0700 schrieb Wirt Wolff:
DW is unfortunately missing at least some patches from the mailing list. Some manually attached patches are listed and some are not; some are listed only on the 'DarcsWatch overview > Listing by user' page, and marked as "unmatched", e.g. the *Master patches. Not sure what causes this
in theory (that is, if there is no bug in darcswatch), this means that the patch bundle has a dependency on a patch not found in the repository. So if you can apply the patch cleanly against a current checkout of the repository, please tell me about the specific patch in question and I’ll try to see if it’s a bug. I guess I could to a more heuristical approach (if more than 50% or 10 patches are found in a repository, the patch is matched to that repository, and maybe marked as “missing depedency patch”). Greetings, Joachim -- Joachim "nomeata" Breitner mail: mail@joachim-breitner.de | ICQ# 74513189 | GPG-Key: 4743206C JID: nomeata@joachim-breitner.de | http://www.joachim-breitner.de/ Debian Developer: nomeata@debian.org

Wirt,
* 20090212170113: X.L.AddMaster: Master turned into a LayoutModifier
wmw: [2] (same as ff. patch but as new module? obsolete?)
is replaced by the following patch in your list, yo can mark it as
obsolete, byebye!
On Fri, Feb 20, 2009 at 2:36 PM, Wirt Wolff
Updated version of Gwern's fine review from a while back:
Down to 15 unapplied in XMC DarcsWatch, several of which are new, and none obsolete but possibly[1] and [2]. Hurrah!
DW is unfortunately missing at least some patches from the mailing list. Some manually attached patches are listed and some are not; some are listed only on the 'DarcsWatch overview > Listing by user' page, and marked as "unmatched", e.g. the *Master patches. Not sure what causes this, but I did have some problems with a darcs get --partial repo while testing patches, so maybe generating patches against different lazy versions may have something to do with it?
= Needing comments,lobbying, committer decision =
* 20080710220410: XMonad.Actions.DynamicKeys: utilities to update key bindings at runtime by Marco TM-CM-:lio Gontijo e S
wmw: Depends on core change. Own thread. * 20081007080041: UpdatePointer even to empty workspaces by Joachim Breitner
, 20081008154245: pointWithin has moved to the core by Joachim Breitner gwern: Maintainer plans to comment this weekend. Tested ok here. Own thread. * 20081019140330: FocusNth improvements fixed by Aleksey Artamonov
wmw: Maintainer still no comment. Tested ok here. Own thread. * 20090117164445: Properly encode destop names before sending them to X server in XMonad.Hooks.EwmhDesktops Khudyakov Alexey
gwern: Blocked on our slow-moving UTF discussion. wmw: Makes UTF-8 non-optional dependency but solves high impact problem for those it affects. Own thread. * 20090128013917: new XMonad.Layout.MessageControl module 20090128004726: NoWrap export patch for use with X.L.MessageControl
wmw: Comments requested from (yet to volunteeer) LayoutCombinators adopter. Own thread. * 20090209192718: add a regular expression matching manageHook by Justin Bogner
wmw: Looks good to me; several user requests and many use cases. * 20090213155441: H.DynamicLog Add format strippers for xmobar and dzen
wmw: No comments yet. Own thread. * 20090215212346: L.Cross: clarify documentation and fix for haddock
wmw: (!) Some fix for Cross should be applied; it blocks -ftesting for anyone with Cross in their repo. * 20090219022417: Cleanup X.L.Mosaic, without breaking it by Adam Vogt
wmw: Works well for me. Own thread. [1] 20090220054704: Add TabbedModifier layout modifer by Adam Vogt * 20090220061819: Add TabbedModifier layout modifer by Adam Vogt
wmw: Not sure if only last patch needed; DarcsWatch may have missed an amend-record. Test and comment. Own thread. = Not on XMC DarcsWatch =
* 20090212170113: X.L.AddMaster: Master turned into a LayoutModifier wmw: [2] (same as ff. patch but as new module? obsolete?) * 20090213020453: X.L.Master: turn it to a Layout modifier and update the code by Ismael Carnales
wmw: No comments yet. No opinion till I look more closely, seems like good idea. Own thread. = Needing action by author or new adopter=
20080926205838: dzenStatusBar by seanmce33@gmail.com, 20081002131304: layoutLogger, windowLogger, workspaceLogger,maildDir loggers by seanmce33@gmail.com gwern: byorgey tried to apply them, but failed. So we're blocked on sean re-sending them. wmw: if no other action next week or so, I'll submit versions against current repo, probably put dzenStatusBar in Util.Dzen instead.
= Summary = Obsoletes have been cleaned out. Not much action since last time on controversial patches. Several new patches have been added since last review, including adam's tabbed modifier response to tabbed subpane requests. -- wmw _______________________________________________ xmonad mailing list xmonad@haskell.org http://www.haskell.org/mailman/listinfo/xmonad

On Friday 20 February 2009 19:36:05 Wirt Wolff wrote:
* 20090215212346: L.Cross: clarify documentation and fix for haddock
wmw: (!) Some fix for Cross should be applied; it blocks -ftesting for anyone with Cross in their repo.
Please fix Cross documentation some way. Haddock currently breaks on it. Ve-e-ry annoying and patches are have to be recorded with --no-test. -- Khudyakov Alexey
participants (4)
-
Ismael Carnales
-
Joachim Breitner
-
Khudyakov Alexey
-
Wirt Wolff