bug-hello
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Cross-Compiling fails due to man-page generation


From: Sami Kerola
Subject: Re: Cross-Compiling fails due to man-page generation
Date: Tue, 5 Aug 2014 21:58:09 +0100

On 24 July 2014 02:43, Assaf Gordon <address@hidden> wrote:
> Attached is my attempt at such patch, based on your suggestion.

Hi Assaf,

That's great. The weekend following when I wrote the reply I polished
a bit of the 'patch' I originally sent, that was completely broken,
that end up to be remarkably similar with the patch in your latest
mail. But not exactly the same, and I must admit yours is more
elegant. The greatest difference is that you skip manual page
generation when it's pointless.

> Few differences from your suggestion:
> 1. If building from tarball (as opposed to from .git), I've changed the
> build to never generate, and never clean the "hello.1" manpage. The downside
> is that if a developer works from a tarball and changes the help screen, the
> manpage will be stale.
> This is the "BUILD_FROM_GIT" in Makefile.am .
> I think that's reasonable, but I could be wrong.

I think skipping manual generation when doing tarball build is good
strategy. The manual is a frozen, like the code, and there really
should not be reason spend time regenerating it (or them, depending on
project).

> 2. If building from GIT and cross-compiling, don't add a rule to the
> makefile to generate the man-page (this is the "!GIT_CROSS_COMPILING" in
> Makefile.am).

Ack.

> 3. I've read somewhere (can't find the link now) that AM_CONDITIONAL should
> not be nested - so it is outside the "AS_IF".

Yes, that was one of the bits not quite perfect in initial mock
version of the change.

> 4. To detect the ".git" directory, I've used "$srcdir/.git" (not just
> ".git") - as this could be problematic in out of tree builds?

All I can say the 'make distcheck' works, and $srcdir/ sounds like
right thing to do, so no objections.

> I've tested it on several systems (including cross-compiling), but more
> testing is always good.

Excellent.

> Comments are welcomed,

None other than thank you for contribution. The change is merged.

-- 
Sami Kerola
http://www.iki.fi/kerolasa/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]