[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries th
From: |
Ralf Wildenhues |
Subject: |
Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries they call. |
Date: |
Wed, 22 Sep 2010 22:41:18 +0200 |
User-agent: |
Mutt/1.5.20 (2010-08-04) |
* Gary V. Vaughan wrote on Wed, Sep 22, 2010 at 10:36:01PM CEST:
> On 23 Sep 2010, at 01:22, Ralf Wildenhues wrote:
> > * Eric Blake wrote on Wed, Sep 22, 2010 at 08:19:28PM CEST:
> >> On 09/22/2010 12:13 PM, Ralf Wildenhues wrote:
> >>>> Is it acceptable instead to use a nested $(MAKE) invocation prior to
> >>>> running help2man to ensure the binary is up-to-date?
> >>>
> >>> Can you show a patch so I can see what you mean?
> >>
> >> $(srcdir)/doc/libtool.1: $(srcdir)/$(auxdir)/ltmain.sh
> >> + $(MAKE) libtool
> >> $(update_mans) --help-option=--help-all libtool
> >
> > When -jN has been passed, the two makes may both try to update 'libtool'
> > at the same time, leading to a race.
> >
> >> $(srcdir)/doc/libtoolize.1: $(srcdir)/libtoolize.in
> >> + $(MAKE) libtoolize
> >> $(update_mans) libtoolize
> >
> > Likewise here.
>
> How about a putting the shell code for libtoolize.in -> libtoolize
> transformation and writing:
>
> $(srcdir)/doc/libtoolize.1: $(srcdir)/libtoolize.in
> $(generate_libtoolize)
> $(update_mans) libtoolize
That will not help, because 'make' may still run it in parallel with the
commands from the 'libtoolize' rule.
Cheers,
Ralf
- [PATCH 4/4] maint: simplify and improve safety of bootstrap process., (continued)
- [PATCH 4/4] maint: simplify and improve safety of bootstrap process., Gary V. Vaughan, 2010/09/22
- Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries they call., Ralf Wildenhues, 2010/09/22
- Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries they call., Eric Blake, 2010/09/22
- Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries they call., Ralf Wildenhues, 2010/09/22
- Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries they call., Eric Blake, 2010/09/22
- Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries they call., Ralf Wildenhues, 2010/09/22
- Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries they call., Eric Blake, 2010/09/22
- Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries they call., Ralf Wildenhues, 2010/09/22
- Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries they call., Gary V. Vaughan, 2010/09/22
- Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries they call.,
Ralf Wildenhues <=
Re: [PATCH 1/r47] maint: help2man targets should rely on the binaries they call., Gary V. Vaughan, 2010/09/22
[PATCH v2 01/11] maint: rearrange Makefile.am in preparation for a follow-up patch., Gary V. Vaughan, 2010/09/23
[PATCH v2 02/11] maint: don't leak developer GREP, SED etc into distribution file., Gary V. Vaughan, 2010/09/23
[PATCH v2 03/11] build: compare `revision' rather than `correctver' in Makefile.am., Gary V. Vaughan, 2010/09/23
[PATCH v2 04/11] build: avoid unnecessary directory changes in Makefile rules., Gary V. Vaughan, 2010/09/23
[PATCH v2 05/11] build: factor Makefile.am `m4sh' invocations to LT_M4SH., Gary V. Vaughan, 2010/09/23