On 4/8/15 12:29 PM, Phil Race wrote: > Isn't it sufficient to comment out this one line ? > > 1215 ALL_OTHER_TARGETS += jaccessdocs > > .. and add a comment as to why ? I'm not familiar with this make file so I took the straightforward approach.
If you are confident that will work I'll redo the patch. Let me know. > > -phil. > > > On 04/08/2015 10:25 AM, Pete Brunet wrote: >> Here is an updated patch. >> http://cr.openjdk.java.net/~ptbrunet/JDK-8076552/webrev.02/ >> >> It simply removes the com.sun.java.accessibility.util part of the >> javadoc generation. >> >> How to better deal with the javadoc generation can be left to later. >> >> Please let me know if this patch meets with your approval. >> >> I have started a local Win build and will start JPRT builds on Linux, >> Windows, Solaris, and Mac shortly. >> >> Thanks, >> Pete >> >> On 4/8/15 12:51 AM, Pete Brunet wrote: >>> Please review/approve the following patch. >>> >>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076552/webrev.01/ >>> >>> The recent push for JDK-8076182 caused a build break, i.e. a problem >>> for >>> the creation of the Javadoc in the environment used by the nightly >>> build. This was because a newly opened package >>> com.sun.java.accessibility.util was mistakenly located in a windows >>> directory. This patch moves the package's files from >>> jdk/src/windows/classes to jdk/src/share/classes and this should >>> resolve >>> the build break for the jdk8u-dev nightly. >>> >>> JPRT builds run OK on solaris, mac, and linux. As of this writing the >>> Win jobs haven't started yet but the 64 bit build completed OK on my >>> local machine. >>> >>> This patch also had to include the fix for JDK-8051297 "Remove >>> com.sun.java.accessibility.util.java.awt.ChoiceTranslator". That file >>> is dead code and its existence in jdk/src/share/classes causes a >>> compilation failure, access of a non-existent enum, the reason the file >>> was planned to be removed. >>> >>> Thanks, Pete >>> >>> >
