This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
Re: [PATCH, libjava/classpath]: Fix overriding recipe for target 'gjdoc' build warning
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Andrew Hughes <gnu dot andrew at redhat dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <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: Thu, 20 Aug 2015 08:20:11 +0200
- 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> <946415796 dot 12688452 dot 1440038906822 dot JavaMail dot zimbra at redhat dot com>
On Thu, Aug 20, 2015 at 4:48 AM, Andrew Hughes <gnu.andrew@redhat.com> wrote:
> ----- 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 was looking at
http://www.gnu.org/software/classpath/
where the lik still points to the CVS server, with the latest
ChangeLog entry from 2012-03-25. The repository you listed above can't
be found on what looks like a classpath start page.
> I've taken the liberty of applying your patch to keep GCJ in sync with its
> upstream.
Thanks!
Uros.