review request (XS) - 7150454: add release jdk7u4 to jprt.properties

David Holmes david.holmes at oracle.com
Sun Mar 4 01:27:27 PST 2012


On 3/03/2012 9:38 PM, David Holmes wrote:
> Didn't we do a review to set the default to release 8 and at the same
> time get rid of all these 6uX values? I don't remember seeing that pushed.

It was pushed and it was 7069991 but it missed these properties.

David
-----

> David
>
> On 3/03/2012 6:33 AM, Kelly O'Hair wrote:
>> Looks ok, but I think you can delete these lines:
>>
>> 157 jprt.build.targets.jdk7temp=${jprt.build.targets.all}
>> 158 jprt.build.targets.jdk7b107=${jprt.build.targets.all}
>> 159 jprt.build.targets.jdk6=${jprt.build.targets.standard}
>> 160 jprt.build.targets.jdk6perf=${jprt.build.targets.standard}
>> 161 jprt.build.targets.jdk6u10=${jprt.build.targets.standard}
>> 162 jprt.build.targets.jdk6u14=${jprt.build.targets.standard}
>> 163 jprt.build.targets.jdk6u18=${jprt.build.targets.standard}
>> 164 jprt.build.targets.jdk6u20=${jprt.build.targets.standard}
>> 165 jprt.build.targets.ejdk6=${jprt.build.targets.all}
>> 166 jprt.build.targets.ejdk7=${jprt.build.targets.all}
>>
>>
>> -kto
>>
>> On Mar 2, 2012, at 10:33 AM, John Coomes wrote:
>>
>>> Hi,
>>>
>>> I'd appreciate reviews of a small change to allow jprt to build
>>> hotspot with '-release jdk7u4'.
>>>
>>> http://cr.openjdk.java.net/~jcoomes/7150454-jprt-jdk7u4/
>>>
>>> Tested with 'jprt submit -release jdk7u4'.
>>>
>>> -John
>>


More information about the hotspot-dev mailing list