Hi Martin,
Does your JDK 10 build have the following fix?
http://hg.openjdk.java.net/jdk/jdk10/rev/107413b070b9
I tried and the build indeed fails when using JDK 10 without this fix as
a bootstrap, but does not (seem to) fail when using JDK 10 with this fix.
Jan
From: *Martin Buchholz* <marti...@google.com <mailto:marti...@google.com>>
Date: Sat, Feb 10, 2018 at 7:36 AM
Subject: Re: RFR: 8194651: javadoc: mark the com.sun.javadoc API for removal
To: Jonathan Gibbons <jonathan.gibb...@oracle.com
<mailto:jonathan.gibb...@oracle.com>>
Cc: javadoc-dev <javadoc-dev@openjdk.java.net
<mailto:javadoc-dev@openjdk.java.net>>
I can confirm that it is this commit:
changeset: 48840:5e2d2067da48 tip
user: ksrini
date: 2018-02-09 13:58 -0800
8194651: javadoc: mark the com.sun.javadoc API for removal
Reviewed-by: jjg
that causes a build failure when rebuilding from scratch.
I bisected my configure flags - it's the boot jdk
bash ./configure --with-boot-jdk=/home/martin/jdk/jdk10
fails while
bash ./configure --with-boot-jdk=/home/martin/jdk/jdk9
succeeds.
Y'all are probably still using jdk9 as bootstrap.
But as you all know, it's been the rule since time immemorial to use JDK
N-1 to bootstrap JDK N.
On Fri, Feb 9, 2018 at 4:22 PM, Jonathan Gibbons
<jonathan.gibb...@oracle.com <mailto:jonathan.gibb...@oracle.com>> wrote:
Martin,
I'm not hearing any CI alarm bells ringing here, and I've just
completed a clean build of the latest bits in jdk/jdk repo.
There was a related javac fix that went back a few days ago, from
Jan Lahoda. Is it possible that either you didn't pick up that fix,
or that you have a locally modified copy of javadoc that needs to be
updated?
-- Jon
On 02/09/2018 04:06 PM, Martin Buchholz wrote:
This seems to break the build
* For target
buildtools_interim_langtools_modules_jdk.javadoc.interim__the.BUILD_jdk.javadoc.interim_batch:
/home/martin/ws/jdk/src/jdk.javadoc/share/classes/com/sun/tools/javadoc/main/SourcePositionImpl.java:31:
warning: [removal] SourcePosition in com.sun.javadoc has been
deprecated and marked for removal
import com.sun.javadoc.SourcePosition;
^
error: warnings found and -Werror specified
1 error
1 warning
On Fri, Feb 9, 2018 at 10:51 AM, Jonathan Gibbons
<jonathan.gibb...@oracle.com <mailto:jonathan.gibb...@oracle.com>>
wrote:
OK
-- Jon
On 1/23/18 2:02 PM, Kumar Srinivasan wrote:
Hi,
Please review fix for 8194651,
https://bugs.openjdk.java.net/browse/JDK-8194651
<https://bugs.openjdk.java.net/browse/JDK-8194651>
Webrev:
http://cr.openjdk.java.net/~ksrini/8194651/webrev.01/
<http://cr.openjdk.java.net/%7Eksrini/8194651/webrev.01/>
Please note this is blocked by: JDK-8194764
Thanks
Kumar