Re: [Monotone-devel] Please review nvm.man-page

[Stephen Leake]

Thomas Keller <address@hidden> writes:

> Am 19.08.2010 14:14, schrieb Stephen Leake:
>> Stephen Leake <address@hidden> writes:
>> 
>>> For MinGW testing, I commented out the isatty check, so it always dumps
>>> the troff source. The tests/manpage test passes.
>>>
>>> Reading the output with 'man mtn' in a Cygwin bash shell works, and with
>>> 'M-x man mtn' in Emacs.
>> 
>> As another test, I commented out the 'isatty' check, but made it always
>> run the interactive command. Running in a Cygwin shell with nroff and
>> less in path, that gave:
>> 
>> bash-3.2$ ./mtn man
>> 'nroff' is not recognized as an internal or external command,
>> operable program or batch file.
>> mtn: fatal: error: ../monotone.man-page/cmd.cc:927: I(fprintf(fp, 
>> ss.str().c_str()) != -1)
>> mtn: this is almost certainly a bug in monotone.
>
> Interesting, I thought popen uses the same environment like the calling
> process, apparently this is not true here. Could you try to change the
> command for the hook "get_man_page_formatter_command" like this?
>
>    PATH=... nroff -man -Tutf8 | less -R
>
> and set ... to the needed path specification?

Didn't help. Apparently popen is just broken in MinGW.

In Cygwin bash under Emacs, running MinGW mtn, this works:

./mtn man | nroff -man -Tutf8 | less -R

(I'm not clear what changed from my previous test where this failed;
probably something in my PATH)

I put that in a shell script (so isatty is false), and it works nicely
in a Cygwin bash window.

-- 
-- Stephe