RFR 8214794 : java.specification.version should be only the major version number
Roger Riggs
Roger.Riggs at oracle.com
Tue Dec 4 19:32:00 UTC 2018
Hi Mandy, Martin,
The new test is unnecessary, the case is covered by
java/lang/System/Versions test
and uses the stronger comparison for the version numbers.
It would not detect the problem unless the version included more than
the major version.
Webrev: http://cr.openjdk.java.net/~rriggs/webrev-spec-ver-8214700-3/
Thanks, Roger
On 12/04/2018 01:41 PM, Mandy Chung wrote:
>
>
> On 12/4/18 8:16 AM, Roger Riggs wrote:
>> Please review correctly setting the java.specification.version property
>> with only the major version number. A test is added to ensure the
>> java spec version agrees with the major version.
>>
>> The symptoms are that jtreg would fail with a full version number.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~rriggs/webrev-spec-ver-8214700-2/
>>
>
> Looks okay. I agree with Martin to go with a stronger assertion
> without converting into a number.
> test/jdk/java/lang/System/Versions.java looks like also covering this
> test case. At some point it'd be good to consolidate these two tests.
>
> Nit: in GensrcMisc.gmk, I think VERSION_NUMBER and VERSION_PRE etc
> are a relevant group. VERSION_SPECIFICATION can be moved to group
> with VERSION_CLASSFILE_MAJOR and MINOR. Magnus may have an opinion.
>
> Mandy
More information about the core-libs-dev
mailing list