diff --git a/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/Extern.java b/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/Extern.java --- a/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/Extern.java +++ b/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/Extern.java @@ -223,7 +223,7 @@ } return true; } catch (Fault f) { - reporter.print(Diagnostic.Kind.WARNING, f.getMessage()); + reporter.print(Diagnostic.Kind.ERROR, f.getMessage()); return false; } } diff --git a/test/langtools/jdk/javadoc/doclet/testLinkOption/TestBadLinkOption.java b/test/langtools/jdk/javadoc/doclet/testLinkOption/TestBadLinkOption.java --- a/test/langtools/jdk/javadoc/doclet/testLinkOption/TestBadLinkOption.java +++ b/test/langtools/jdk/javadoc/doclet/testLinkOption/TestBadLinkOption.java @@ -23,8 +23,8 @@ /* * @test - * @bug 4625883 - * @summary Make sure that bad -link arguments trigger warnings. + * @bug 4625883 8180019 + * @summary Make sure that bad -link arguments trigger errors. * @author jamieh * @library ../lib * @modules jdk.javadoc/jdk.javadoc.internal.tool @@ -44,11 +44,10 @@ String out = "out"; javadoc("-d", out, "-sourcepath", testSrc, - "-link", out, + "-link", "a-non-existent-link", "pkg"); - checkExit(Exit.OK); + checkExit(Exit.ERROR); - // TODO: the file it is trying to read, out/out/package-list, warrants investigation checkOutput(Output.OUT, true, "Error reading file:"); } diff --git a/test/langtools/jdk/javadoc/doclet/testLinkOption/TestNewLineInLink.java b/test/langtools/jdk/javadoc/doclet/testLinkOption/TestNewLineInLink.java --- a/test/langtools/jdk/javadoc/doclet/testLinkOption/TestNewLineInLink.java +++ b/test/langtools/jdk/javadoc/doclet/testLinkOption/TestNewLineInLink.java @@ -44,7 +44,7 @@ void test() { javadoc("-d", "out", "-sourcepath", testSrc, - "-linkoffline", "http://www.java.sun.com/j2se/1.4/docs/api", testSrc, + "-linkoffline", "http://www.java.sun.com/j2se/1.4/docs/api", testSrc("jdk"), "testNewLineInLink"); checkExit(Exit.OK);