Re: JAVAROOT was already defined

[Alexandre Duret-Lutz]

>>> "Simon" == Simon Josefsson <address@hidden> writes:

[...]

 >> 2004-03-19  Alexandre Duret-Lutz  <address@hidden>
 >> 
 >> Overriding JAVAROOT is legitimate, do not warn about it.
 >> * lib/Automake/Variable.pm (%_silent_variable_override): Add JAVAROOT.
 >> * tests/java2.test: Run automake without -Wno-override.
 >> Report from Simon Josefsson.

 Simon> Alas, I still have this problem.  I need to comment out JAVAROOT from
 Simon> every /usr/share/automake-*/am/java.am file I get from Debian, to
 Simon> avoid the re-definition error.  Debian is now up to version 1.9.4, and
 Simon> I can verify that the patch is part of their copy.

You can override JAVAROOT, but you have to do so in the same
conditions it has been defined.  That is: unconditionally.

So just changing

if JAVA
JAVAROOT = $(top_builddir)/java

to

JAVAROOT = $(top_builddir)/java
if JAVA

will also get rid of the diagnostic.


I'll look whether supporting the former in Automake is easy.  Thanks!
-- 
Alexandre Duret-Lutz