Request for approval: 7152519, 7159041, 7153343 Remove including of non-POSIX header link.h

[Jonathan Lu]

Thanks a lot for all your support

Regards
Jonathan

On 09/04/2012 11:02 PM, Neil Richards wrote:
> Now pushed to jdk7u-dev/jdk [1] [2] [3].
>
> Regards,
> Neil
>
> [1] http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/f8acc3222383
> [2] http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/aa04907cb729
> [3] http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/257dee3bea4e
>
> On Mon, 2012-09-03 at 11:53 +0100, Seán Coffey wrote:
>> one approval request for these 3 related issues should be fine.
>> Approved for jdk7u-dev. I've created bug records for all and will update
>> when I see your pushes. As Alan said, 3 changesets will be best and I
>> think that's your intention already.
>>
>> cc'ing the original jdk8 reviewers here also as an FYI.
>>
>> regards,
>> Sean.
>>
>> On 03/09/2012 09:28, Jonathan Lu wrote:
>>> On 09/03/2012 04:20 PM, Alan Bateman wrote:
>>>> On 03/09/2012 09:09, Jonathan Lu wrote:
>>>>> Hi all,
>>>>>
>>>>> Here's a request of porting the fix 7152519, 7159041, 7153343 to
>>>>> JDK7, could you please help to review?
>>>>>
>>>>> All the three patches serves for the same purpose of removing
>>>>> including of non-POSIX header link.h, so I suggest to contribute
>>>>> them in one commit.
>>>> I don't know if there is an agreed policy on this but I think it
>>>> would be better to just hg export/import the individual change-sets,
>>>> otherwise it can be confusing when comparing the history or doing bug
>>>> archeology.
>>>>
>>>> -Alan
>>>>
>>> Hi Alan,
>>>
>>> Sorry for the confusion, I think you are right, here I'm posting the
>>> independent 3 change sets, BTW, is it appropriate to raise one back
>>> port request for these three changes?
>>>
>>> 7152519: http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/604067ec3ced
>>> 7159041: http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/9383a3749940
>>> 7153343: http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/c4e66dc3222d
>>>
>>> Thanks & best regards
>>> Jonathan
>>>
>>>
>