<AWT Dev> RfR JDK-8076552 nightly build break fix

[Phil Race]

Isn't it sufficient to comment out this one line ?

1215 ALL_OTHER_TARGETS += jaccessdocs

.. and add a comment as to why ?

-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
>>
>>