[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Monotone-devel] Please review nvm.man-page
From: |
Stephen Leake |
Subject: |
Re: [Monotone-devel] Please review nvm.man-page |
Date: |
Fri, 20 Aug 2010 05:15:51 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (windows-nt) |
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
- [Monotone-devel] Please review nvm.man-page, Thomas Keller, 2010/08/18
- Re: [Monotone-devel] Please review nvm.man-page, Stephen Leake, 2010/08/19
- Re: [Monotone-devel] Please review nvm.man-page, Thomas Keller, 2010/08/20
- Re: [Monotone-devel] Please review nvm.man-page, Stephen Leake, 2010/08/21
- Message not available
- Message not available
- Re: [Monotone-devel] Please review nvm.man-page, Thomas Keller, 2010/08/21
- Re: [Monotone-devel] Please review nvm.man-page, Stephen Leake, 2010/08/23