RFR: 8000780 back ported Fix zero fail to build in JDK7u-dev [updated]

[Alejandro E Murillo]

Hi Chris (Phillips)
I just got a snapshot of hs24 and bumped the build number to b31,
so we can resume pushing bug fixes to it.
Unfortunately the patch in the webrev at [1] does not apply cleanly anymore,
Can you guys reapply the changes to clean clone of [2] and send the new 
webrev?
Then  if the previous reviewers approve it, I will push that soon after 
so it makes
  it into next week snapshot. Apologies for the extra work

[1]http://integral-portal.lgonqn.org/temp/ChrisPhi/jdk7u-dev-zero-hs24b29_20121222/
[2] http://hg.openjdk.java.net/hsx/hsx24/hotspot



Thanks
Alejandro


On 1/2/2013 1:02 PM, Christian Thalinger wrote:
> On Dec 22, 2012, at 10:15 AM, Chris Phillips @ T O<ChrisPhi at lgonqn.org>  wrote:
>
>> Hi Alejandro,
>> Oops emails crossed in the net ...
>> See my latest email , I had to regenerate the webrev and patch -
>> This webrev s/b ok:
>>
>> Webrev:
>> http://integral-portal.lgonqn.org/temp/ChrisPhi/jdk7u-dev-zero-hs24b29_20121222/
> That looks good.  -- Chris
>
>> Chris
>>
>> On 22/12/12 01:03 PM, Alejandro E Murillo wrote:
>>> Hi Chris,
>>> In that case is not necessary.
>>> I initially thought the webrev was against jdk7u/jdk7u-dev/hotspot
>>> but looks like it is against hsx/hsx24/hotspot.
>>> BTW, I went to check the webrev link and is unreachable, can you check
>>> that?
>>> Thanks
>>> Alejandro
>>>
>>> On 12/21/2012 9:53 PM, Chris Phillips @ T O wrote:
>>>> Hi Alejandro,
>>>>
>>>> The only change since I tested this afternoon is the change
>>>> for the hotspot_version. I will re-run tests if you wish but it seems
>>>> unnecessary.
>>>>
>>>> Cheers!
>>>> Chris
>>>>
>>>> On 21/12/12 07:40 PM, Alejandro E Murillo wrote:
>>>>> On 12/21/2012 4:34 PM, Chris Phillips @ T O wrote:
>>>>>> Hi
>>>>>>
>>>>>> {Not sure if this needs a new bug or not, it definitely needs a
>>>>>> sponsor
>>>>>> - Twisti ? }
>>>>>>
>>>>>> Attached is a backport (essentially resurrection of the Permgen code)
>>>>>> of 8000780 now updated with Roman Kennke's comments. Checked and built
>>>>>> against hsx24b28. (Current jdk7u-dev hs)
>>>>>>
>>>>>> Please review.
>>>>>>
>>>>>> Cheers!
>>>>>> Chris
>>>>>>    PS
>>>>>> Webrev is here:
>>>>>> http://integral-portal.lgonqn.org/temp/ChrisPhi/jdk7u-dev-zero/hs24b28/
>>>>>>
>>>>> Hi
>>>>> jdk7u-dev/hotspot corresponds to hs24-b27.
>>>>> I just got the hs24 snapshot for b28 and bumped the number to b29,
>>>>> Can you rework the patch/webrev against this repo:
>>>>>
>>>>>   http://hg.openjdk.java.net/hsx/hsx24/hotspot
>>>>>
>>>>> once it's reviewed I can push it
>>>>>
>>>>> Thanks
>>>>>
>>>