<div>ant can no longer be invoked directly (i.e. not via makefile) in langtools/make because ${warnings.fatal} is undefined.</div><div><br></div><div>cd langtools/make</div><div>ant -Djavac.warnings.fatal= -Dboot.java.home=$jdk6 -Dtarget.java.home=$jdk7 build </div>
<div>...</div><div><div>    [javac] Compiling 295 source files to /usr/local/google/home/martinrb/ws/icedtea7-forest/langtools/build/bootstrap/classes</div><div>    [javac] javac: invalid flag: ${warnings.fatal}</div></div>
<div><br></div><div>Caused by this icedtea-only patch:</div><div><br></div><div># HG changeset patch</div><div># User andrew</div><div># Date 1345649970 -3600</div><div># Node ID 26f5c80eb792e8536e15c82670dc38a140bb1ec3</div>
<div># Parent  b19919b8e86d7f17317d7cf6ca440ee4d3d4e14d</div><div>PR1095: Allow -Werror to be turned off.</div><div><br></div><div>diff --git a/make/Makefile b/make/Makefile</div><div>--- a/make/Makefile</div><div>+++ b/make/Makefile</div>
<div>@@ -111,6 +111,12 @@</div><div>   ANT_OPTIONS += -Ddebug.classfiles=true</div><div> endif</div><div> </div><div>+ifeq ($(JAVAC_WARNINGS_FATAL), true)</div><div>+  ANT_OPTIONS += -Dwarnings.fatal=-Werror</div><div>+else</div>
<div>+  ANT_OPTIONS += -Dwarnings.fatal=</div><div>+endif</div><div>+</div><div> # Note: jdk/make/common/Defs.gmk uses LANGUAGE_VERSION (-source NN)</div><div> # and the somewhat misnamed CLASS_VERSION (-target NN)</div><div>
 ifdef TARGET_CLASS_VERSION</div><div>diff --git a/make/build.properties b/make/build.properties</div><div>--- a/make/build.properties</div><div>+++ b/make/build.properties</div><div>@@ -68,7 +68,7 @@</div><div> # set the following to -version to verify the versions of javac being used</div>
<div> javac.version.opt =</div><div> # in time, there should be no exceptions to -Xlint:all</div><div>-javac.lint.opts = -Xlint:all,-deprecation -Werror</div><div>+javac.lint.opts = -Xlint:all,-deprecation ${warnings.fatal}</div>
<div> </div><div> # options for the &lt;javadoc&gt; task for javac</div><div> #javadoc.jls3.url=<a href="http://java.sun.com/docs/books/jls/">http://java.sun.com/docs/books/jls/</a></div><div><br></div><div><br></div><div>
Here&#39;s a possible fix, that preserves the openjdk default of -Werror, renames warnings.fatal to javac.warnings.fatal, and allows escape via JAVAC_WARNINGS_FATAL=false</div><div><br></div><div><br></div><div><div>diff --git a/make/Makefile b/make/Makefile</div>
<div>--- a/make/Makefile</div><div>+++ b/make/Makefile</div><div>@@ -111,10 +111,8 @@</div><div>   ANT_OPTIONS += -Ddebug.classfiles=true</div><div> endif</div><div> </div><div>-ifeq ($(JAVAC_WARNINGS_FATAL), true)</div><div>
-  ANT_OPTIONS += -Dwarnings.fatal=-Werror</div><div>-else</div><div>-  ANT_OPTIONS += -Dwarnings.fatal=</div><div>+ifeq ($(JAVAC_WARNINGS_FATAL), false)</div><div>+  ANT_OPTIONS += -Djavac.warnings.fatal=</div><div> endif</div>
<div> </div><div> # Note: jdk/make/common/Defs.gmk uses LANGUAGE_VERSION (-source NN)</div><div>diff --git a/make/build.properties b/make/build.properties</div><div>--- a/make/build.properties</div><div>+++ b/make/build.properties</div>
<div>@@ -68,7 +68,8 @@</div><div> # set the following to -version to verify the versions of javac being used</div><div> javac.version.opt =</div><div> # in time, there should be no exceptions to -Xlint:all</div><div>-javac.lint.opts = -Xlint:all,-deprecation ${warnings.fatal}</div>
<div>+javac.warnings.fatal = -Werror</div><div>+javac.lint.opts = -Xlint:all,-deprecation ${javac.warnings.fatal}</div><div> </div><div> # options for the &lt;javadoc&gt; task for javac</div><div> #javadoc.jls3.url=<a href="http://java.sun.com/docs/books/jls/">http://java.sun.com/docs/books/jls/</a></div>
</div><div><br></div>