[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-pm-utils
Subject:    Re: [Pm-utils] [PATCH 1/1] Do not track the man pages.
From:       "Dan Nicholson" <dbn.lists () gmail ! com>
Date:       2008-04-28 14:48:36
Message-ID: 91705d080804280748j7b73801by559ad5d96a3939d8 () mail ! gmail ! com
[Download RAW message or body]

On Mon, Apr 28, 2008 at 5:09 AM, Stefan Seyfried <seife@suse.de> wrote:
> Michael Biebl wrote:
>  > 2008/3/8, Dan Nicholson <dbn.lists@gmail.com>:
>
> >> Yes, but I don't have docbook2man everywhere. The Makefile should be
>  >>  conditional on whether I have it so I don't bomb my build on the man
>  >>  pages. Something like:
>  >>
>  >>  configure.ac
>  >>  ...
>  >>  AC_PATH_PROG(DOCBOOK2MAN, docbook2man, no)
>  >>  AM_CONDITIONAL(HAVE_DB2MAN, test "x$DOCBOOK2MAN" != xno)
>  >>  ...
>  >>
>  >>  and then surround the man_MANS in Makefile.am with "if HAVE_DB2MAN".
>  >>
>  >
>  > The problem with that is, that the one, creating the official dist
>  > tarballs, could miss to include the man pages.
>
>  Then just kill "make dist" if it is not there but not a plain make. I just had
>  to download and install 500MB of stuff, just to be able to do a test build.

I just committed a patch that makes docbook2man optional, but forces
the built man pages to be included in `make dist'. You'll get a weird
error if you try to run `make dist' and you don't have docbook2man
installed. I think this covers all the situations.

Also, `make distcheck' is broken by the ChangeLog rule. Looking into it...

--
Dan
_______________________________________________
Pm-utils mailing list
Pm-utils@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/pm-utils
[prev in list] [next in list] [prev in thread] [next in thread]