On Sat, 18 Feb 2012 at 12:16:09 +0100, Rodolfo García Peñas wrote: > > On Sat, 18 Feb 2012, Rodolfo García Peñas escribió: > > > On Sat, 18 Feb 2012, Carlos R. Mafra escribió: > > > > > On Sat, 18 Feb 2012 at 2:27:50 -0800, Doug Barton wrote: > > > > > > > As part of that process I double-checked that all of our patches had > > > > been applied to the base, and was left with one little one: > > > > > > > > http://www.freebsd.org/cgi/cvsweb.cgi/ports/x11-wm/windowmaker/files/patch-am?rev=1.2;content-type=text%2Fplain > > > > > > > > It would be great it this could go in so that I can remove it from our > > > > port at the next release. :) > > > > > > So it's up to you to write the patch for inclusion :-) > > > > > > > Meanwhile, I just want to say how impressed I am with the new version. > > > > All my old themes, scripts, config, etc. (some dating back to the 90's) > > > > worked without a hitch. > > > > > > From 19c36797d7f9998dc0f8d397b83108708a4ec8f2 Mon Sep 17 00:00:00 2001 > > From: Doug Barton <[email protected]> > > Date: Sat, 18 Feb 2012 12:10:22 +0100 > > Subject: [PATCH] utils: Include the SHEXEC in the wm-oldmenu2new script > > > > The script wm-oldmenu2new can now convert the SHEXEC lines. > > --- > > util/wm-oldmenu2new | 2 ++ > > 1 files changed, 2 insertions(+), 0 deletions(-) > > > > diff --git a/util/wm-oldmenu2new b/util/wm-oldmenu2new > > index 6cdc7c2..5ac8015 100755 > > --- a/util/wm-oldmenu2new > > +++ b/util/wm-oldmenu2new > > @@ -129,6 +129,7 @@ perl - $T <<-'EOF' > $T-p > > for (@foo2) { > > s/^(\s*)"([^"]*)"\s\s*END/\1),/; > > > > s/^(\s*)"([^"]*)"\s\s*EXEC\s\s*(.*)$/\1($v\n\1"\2",$v\n\1EXEC,$v\n\1"\3"$v\n\1),/; > > + > > s/^(\s*)"([^"]*)"\s\s*SHEXEC\s\s*(.*)$/\1($v\n\1"\2",$v\n\1SHEXEC,$v\n\1"\3"$v\n\1),/; > > push @foo,split "\n"; > > } > > @foo2=(); > > @@ -146,6 +147,7 @@ > > s/^(\s*)"([^"]*)"\s\s*OPEN_MENU\s\s*(.*)$/\1($v\n\1"\2",$v\n\1OPEN_MENU,$v\n\1"\ > > for (@foo) { > > s/^(\s*)([^ ]*)\s\s*END/\1),/; > > s/^(\s*)([^ > > ]*)\s\s*EXEC\s\s*(.*)$/\1($v\n\1"\2",$v\n\1EXEC,$v\n\1"\3"$v\n\1),/; > > + s/^(\s*)([^ > > ]*)\s\s*SHEXEC\s\s*(.*)$/\1($v\n\1"\2",$v\n\1SHEXEC,$v\n\1"\3"$v\n\1),/; > > push @foo2,split "\n"; > > } > > @foo=(); > > -- > > 1.7.8.3 > > Take a look in: > > From: Doug Barton <[email protected]> > > IMO is the correct.
Thanks for the patch, but I guess that the author is you :-) You can acknowledge his suggestion by writing something in the commit body. That's what I'll do now if nobody complains. -- To unsubscribe, send mail to [email protected].
