Thomas Adam wrote: > On Thu, Dec 29, 2011 at 05:51:34PM +0100, Michael Großer wrote: >> > (with other words, it will eliminate the shell as soon as it is ready >> > with spawning "foo"). >> >> I hope, my add-on is correct. It may be redundant, but this redundance >> would help the reader to classify these two statements > > I had to go through and correct this to how it was before, because even > though you used $SHELL, that's *only* correct if you don't set ExecUseShell,
Are you sure that you mean these diffs? http://git.xteddy.org/index.cgi?p=fvwmwiki.git;a=commitdiff;h=c78310ae93c53882d51db7a26eb1546d7b0c4090 http://git.xteddy.org/index.cgi?p=fvwmwiki.git;a=commitdiff;h=c727a3b7ca2895d521783f64b87135b787c0d8f9 http://git.xteddy.org/index.cgi?p=fvwmwiki.git;a=commitdiff;h=b5fa5cf393bb4a5bfcf3b1c618db652b84287e27 I didn't add a "$SHELL" anywhere. It must have been someone else then. > and it's hence enough to just use the word 'shell' to mean the shell FVWM is > using. No, mentioning ExecUseShell or $SHELL here, explicitly, confuses > things. > > You've also gone through and added several commas to different parts of the > text where they were not needed. I'm not going to correct your misuse of > them, I don't care to, but be aware that you've now completely changed the > flow of text as originally written by me to make it considerably worse in > some areas. > > The rest of your additions are mostly wiki-related; I don't mind those, > thanks. > > -- Thomas Adam >