RFR: 8164829 & 8164830: back out jdeprscan

[Stuart Marks]

OK, great, thanks Joe, this will be a lot less disruptive. I've filed a few bugs:

JDK-8164835 add a few tools tests to the problem list

JDK-8164834 remove jdeprscan from tools/launcher/VersionCheck.java

I'll post reviews shortly.

s'marks

On 8/25/16 9:05 PM, joe darcy wrote:
> Hi Stuart,
>
> I've looked over the failures. Three tests fail:
>
> * One that checks that commands have a -version option.
>
> * One new jdeprscan test on windows.
>
> * A langtools tests that checks for consistency of APIs using annotations.
>
> Rather than backing out all of jdeprscan, I'd advise to instead add the
> jdeprscan command to the whitelist for the -version test and temporarily problem
> list the other two tests on affected platforms.
>
> I trust the two problem lists test failures should then be able to be resolved
> and returned into active service within a day or so.
>
> Thanks for being diligent on checking for test failures,
>
> -Joe
>
>
> On 8/25/2016 8:43 PM, Stuart Marks wrote:
>> Sigh, I just pushed jdeprscan and our internal builds failed various source
>> code checks. I haven't fully determined why, so meanwhile I'll just back
>> everything out. Webrevs:
>>
>> langtools -
>>
>>     http://cr.openjdk.java.net/~smarks/reviews/8164829/webrev.0/
>>
>> jdk -
>>
>>     http://cr.openjdk.java.net/~smarks/reviews/8164830/webrev.0/
>>
>> Please review.
>>
>> Thanks.
>>
>> s'marks
>