This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, libjava/classpath]: Fix overriding recipe for target 'gjdoc' build warning
- From: Andrew Hughes <gnu dot andrew at redhat dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: gcc-patches at gcc dot gnu dot org, java-patches at gcc dot gnu dot org, Andrew Haley <aph at redhat dot com>, Tom Tromey <tromey at redhat dot com>
- Date: Wed, 19 Aug 2015 22:48:26 -0400 (EDT)
- Subject: Re: [PATCH, libjava/classpath]: Fix overriding recipe for target 'gjdoc' build warning
- Authentication-results: sourceware.org; auth=none
- References: <CAFULd4YCSbA_2V8jNF1QtcM8b4EF8mJzTD7PyU9ETZ-uSyemsw at mail dot gmail dot com> <CAFULd4bj7q=U+yHrd=+U9NgB=wMQtarvAEw-uAW9xL6SC09keQ at mail dot gmail dot com>
----- Original Message -----
> On Fri, Aug 7, 2015 at 1:21 PM, Uros Bizjak <ubizjak@gmail.com> wrote:
>
> > Attached patch fixes:
> >
> > Makefile:871: warning: overriding recipe for target 'gjdoc'
> > Makefile:786: warning: ignoring old recipe for target 'gjdoc'
> >
> > build warning when compiling libjava.
> >
> > The problem was in configure.ac: we have to depend gjdoc build on
> > CREATE_WRAPPERS in the same way as other tools are dependent a couple
> > of lines above.
> >
> > While in this area, I also removed obsolete automake < 1.11
> > workaround. As mentioned in HACKING: "Make sure you have Automake
> > 1.11.1 installed. Exactly that version!"
> >
> > I have included all generated files in the diff. The changes are small
> > and they illustrate the effect of the patch.
> >
> > 2015-08-07 Uros Bizjak <ubizjak@gmail.com>
> >
> > * configure.ac (tools/gjdoc): Depend on CREATE_WRAPPERS.
> > * configure: Regenerate.
> > * tools/Makefile.am: Remove unneeded dependencies for Automake 1.11.
> > * tools/Makefile.in: Regenerate.
> >
> > Patch was bootstrapped on x86_64-linux-gnu, Fedora 22.
> >
> > OK for GCC mainline?
>
> I have committed this patch to GCC mainline SVN repository. Both
> issues can be considered obvious and the fix is trivial.
>
> Also, it looks like the official classpath repository is a dead place
> for a couple of years.
>
Hardly.
http://git.savannah.gnu.org/cgit/classpath.git/log/
I've taken the liberty of applying your patch to keep GCJ in sync with its
upstream.
> Uros.
>
--
Andrew :)
Senior Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)
PGP Key: ed25519/35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222
PGP Key: rsa4096/248BDC07 (hkp://keys.gnupg.net)
Fingerprint = EC5A 1F5E C0AD 1D15 8F1F 8F91 3B96 A578 248B DC07