RfR JDK-8076552 nightly build break fix

[Pete Brunet]

How's this?
http://cr.openjdk.java.net/~ptbrunet/JDK-8076552/webrev.03

On 4/8/15 12:47 PM, Mandy Chung wrote:
> I agree with Phil's suggestion and file a bug to follow up the javadoc
> build issue.
>
> You can verify the result from make docs that there is no javadoc
> generated for this package on windows build.
>
> Mandy
>
> On 4/8/2015 10:29 AM, 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 ?
>>
>> -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
>>>>
>>>>
>>
>