Re: [patch] binutils/: Remove the dependency on ancient automake

[Daniel Jacobowitz <drow at false dot org>]

On Thu, Apr 14, 2005 at 05:52:16PM +0100, Maciej W. Rozycki wrote:
> Hello,
> 
>  I've noticed binutils/Makefile.in has been generated with a severely 
> obsolete version of automake and actually a snapshot of some sort rather 
> than a release (1.4-p6; my records show it's about four years old).  
> Having processed it with version 1.9.4 I've discovered a few minor 
> glitches in the doc/ subdirectory resulting from using fragile constructs 
> that do not work anymore.
> 
>  Here is a fix.  It changes the problematic constructs to ones that have 
> been documented on automake info pages.  They should still work will 
> 1.4-p6, but I think there is no point in staying tied to old bugs, so I'd 
> like to regenerate scripts using a recent version of automake.  Besides 
> ld/ this is the only subdirectory still using a pre-1.9 automake and I 
> have a separate patch for ld/ as well.

There's actually a reason why I haven't done binutils and ld yet.  They
both use ylwrap; binutils uses it via automake rules, ld may in some
circumstances but usually uses it via explicit rules, and gdb off in
its non-automake corner uses it explicitly.

I'm hesitant to move them over separately; the last time I looked at
this I decided the only viable option was to update all three and
import a new ylwrap, all simultaneously.  Which is not hard to do, of
course!  But I never got around to it.

Can you check out my reasoning here?  Did your tests try to regenerate
the lexers/parsers?  I did all the above from memory from last year, so
it might easily be bogus.

> 2005-04-14  Maciej W. Rozycki  <macro@linux-mips.org>
> 
> 	* doc/Makefile.am (binutils.dvi, binutils.info): Remove, moving
> 	the dependency on config.texi to...
> 	(binutils_TEXINFOS): ... here.
> 	(install): Rename goal to...
> 	(install-data-local): ... this, to preserve implicit dependencies.
> 	(info): Likewise, to...
> 	(info-local). ... this.
> 	* doc/Makefile.in: Regenerate with automake 1.9.4.
> 	* aclocal.m4: Regenerate.
> 	* config.in: Regenerate.
> 	* configure: Regenerate.

The patch itself looks fine, by the way.

-- 
Daniel Jacobowitz
CodeSourcery, LLC