m4-patches
[Top][All Lists]
Advanced

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

Re: [PATCH] build: fix bootstrapping


From: Pavel Raiskup
Subject: Re: [PATCH] build: fix bootstrapping
Date: Thu, 29 May 2014 17:29:30 +0200
User-agent: KMail/4.12.5 (Linux/3.14.4-200.fc20.x86_64; KDE/4.12.5; x86_64; ; )

On Tuesday 27 of May 2014 08:58:22 Pavel Raiskup wrote:
> On Monday 26 of May 2014 21:14:23 Eric Blake wrote:
> > That said, I'll probably apply this once I know more about why it is
> > needed.
>
> Please don't, the patch is not correct.  I'll repost better one.  When
> man pages are generated by help2man, the man page target should not be
> dependant on generated binary (when help2man is also not distributed,
> which is IMO bad idea in principle).

Sorry for the delay.  I tried to fix yet another problem blocking me from
building that in Copr [1].

Patch 0000:

I see that the code I wanted to post was already in source code.  The code
was (basically) reverted out by the commit a43fd66dc18.  This commit
caused that there are no guarantees doc directory (and man page then) are
generated after the source is built.

This causes problems especially after git clone.  According to
automake.info, we should either ship help2man (which is IMO bad because -
as described in docs - it breaks cross-compilation builds of manual pages)
or just guarantee that docs are generated later than source.

---

Patch 0001 enables module loading and file inclusions on my machine (and
also enables me to run testsuite).  What did not work previously:

  * ./src/m4 libtool wrapper
  * include(AUTHORS) when M4PATH was defined
  * m4_module_load of 'm4' when m4 directory was in search path

I'd be glad to prepare testsuite for those issues ^^ but I would like to
receive ack that we could go this way.

[1] http://copr.fedoraproject.org/coprs/praiskup/autotools/builds/

Pavel

Attachment: 0001-build-fix-bootstrap-fail.patch
Description: Text Data

Attachment: 0002-modules-inclusions-fix-path-searching-issues.patch
Description: Text Data

Attachment: testsuite.log.xz
Description: application/xz


reply via email to

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