On Fri, Jul 8, 2011 at 4:49 AM, Richard Shaw <[email protected]> wrote:
> On Thu, Jul 7, 2011 at 11:22 AM, Chris Foster <[email protected]> wrote:
>> Ok, I spent some time (more than expected) getting this working.  The
>> build system should now autogenerate man pages on unix systems when
>> txt2man and python can be found.  (patch attached. for reviewers, see the
>> pull request as before, https://github.com/OpenImageIO/oiio/pull/118)
>
> Ok, I ran into a couple of problems, the first one I think I fixed myself.
>
> 1. The man pages were not being generated at first, the problem, I
> think is this line in the patch:
>
> +find_program(txt2man txt2man)
>
> should be:
>
> +find_program(TXT2MAN txt2man)
>
> as it appears the cmake variables are case sensitive.

ugh, I did test this (multiple times) but I changed the TXT2MAN variable
to txt2man at some stage and forgot to update it in the test for whether
txt2man was found.  At that point I no doubt got bitten by the
persistent cmake cache as everything still seemed to work :(

I've fixed that now, it should be autoupdated in the diff online.
(Thanks Solomon for the tip about getting diffs out of pull requests.)

~Chris
_______________________________________________
Oiio-dev mailing list
[email protected]
http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org

Reply via email to