Approved.
In the interest of getting a usable build as soon as possible, issues
brought
up during the review will be addressed by (near) future pushes.
Paul
On 12/2/11 9:51 AM, Michael McMahon wrote:
Yes, the first two of those links were updated. This is now what I would
like to push.
Webrevs are located at:
(jdk modified files)
http://cr.openjdk.java.net/~michaelm/7113349/5/jdk7u-osx/modified/
(jdk new files)
http://cr.openjdk.java.net/~michaelm/7113349/5/jdk7u-osx/new/
(corba new and modified)
http://cr.openjdk.java.net/~michaelm/7113349/1/corba/webrev/
Thanks,
Michael.
On 02/12/11 14:37, Dalibor Topic wrote:
Thanks, Michael!
Since Paul is now the maintainer for the jdk7u-osx forest, you'll
need his approval for the push.
I assume you've updated the changesets since you posted this request
as a response to reviewers,
so I'm curious if you intend to push these changes or an updated
version - if the latter is the case,
please post the URLs to the webrevs in a reply.
cheers,
dalibor topic
On 12/2/11 12:16 AM, Michael McMahon wrote:
Hi,
This is a request to push the Macosx port of jdk into the jdk7u-osx
forest, for 7u4.
Webrevs are located at:
(jdk modified files)
http://cr.openjdk.java.net/~michaelm/7113349/4/jdk7u-osx/modified/
(jdk new files)
http://cr.openjdk.java.net/~michaelm/7113349/4/jdk7u-osx/new/
(corba new and modified)
http://cr.openjdk.java.net/~michaelm/7113349/1/corba/webrev/
Thanks,
Michael.
