From: Christian Couder <chriscool@tuxfamily.org>
Date: Tue, 11 Mar 2008 05:48:15 GMT
Subject: Re: [PATCH 1/2 resend] help: add "man.viewer" config var to use "woman" or "konqueror"
Message-ID: <200803110648.16117.chriscool@tuxfamily.org>
In-Reply-To: <200803110100.m2B102k9007026@localhost.localdomain>

Le mardi 11 mars 2008, Xavier Maillard a écrit :
>    This patch makes it possible to view man pages using other tools
>    than the "man" program. It also implements support for emacs'
>    "woman" and konqueror with the man KIO slave to view man pages.
>
>    Note that "emacsclient" is used with option "-e" to launch "woman"
>    on emacs and this works only on versions >= 22.
>
>    Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
>
> Acked-off-by: Xavier Maillard <xma@gnu.org>

Thanks. ("Tested-by:" may be better though.) 

> There is something wrong though. git cmd --help should wait for
> the server to return in my opinion.

When I try something like the following on the command line:

$ emacsclient -e '(woman "fork")'

emacsclient doesn't wait for the server to return.

So I don't see how and why we should wait.

Thanks,
Christian.