RFR (XXS) 8183238 Obsolete CheckEndorsedAndExtDirs and remove checks for lib/endorsed and lib/ext

[Ioi Lam]

On 3/29/18 5:15 PM, mandy chung wrote:
>
>
> On 3/30/18 7:24 AM, David Holmes wrote:
>> On 30/03/2018 4:16 AM, Ioi Lam wrote:
>>> Hi Alan,
>>>
>>> Thanks for the reminder of the other files. I checked all the files 
>>> in the open and closed repos with 'grep -r CheckEndorsedAndExtDirs'. 
>>> I found only 3 man pages in the open repo that contained references 
>>> to CheckEndorsedAndExtDirs. No test cases or any other files refer 
>>> to this option.
>>>
>>> Here's an updated webrev that added the changes to the mage pages. 
>>> It also removes the dead code in arguments.cpppointed out by Mandy 
>>> and David.
>>>
>>> http://cr.openjdk.java.net/~iklam/jdk11/8183238-obsolete-CheckEndorsedAndExtDirs.v02/ 
>>
>>
>
> Looks fine.
>
> Can you file a docs issue to update the java tool reference guide [1] 
> to take out the option?

Hi Mandy,

I've filed https://bugs.openjdk.java.net/browse/JDK-8200476 "Update java 
tool reference guide to remove CheckEndorsedAndExtDirs option"


Thanks

- Ioi
>
>> Seems okay, though I thought the man page sources were "dead" anyway. 
>> Surprised to still see them there.
>>
>
> These out-of-date man pages are for OpenJDK build and they were 
> generated by the docs team and dropped in the repo.
>
> Jon and I have discussed to include the man pages in our source in 
> markdown and the build can generate the man pages in proper formats 
> and include in the docs bundle.   Developers will update them for any 
> change to CLI options and the man pages will always be up-to-date.  I 
> think that's a good improvement and hope to happen some time.
>
> Mandy
> [1] https://docs.oracle.com/javase/10/tools/java.htm#JSWOR624