Martin,

A couple of points of order here

On 6/03/2013 7:19 AM, Martin Buchholz wrote:
Thanks.
Pushed to /hg/jdk8/tl.

You needed to have a jdk8 Reviewer listed for this change. Erik is a Committer not Reviewer.

(only tested on Linux)

I sincerely hope Erik tested on Windows, BSD and Solaris before this was pushed!

David
-----

On Mon, Mar 4, 2013 at 11:45 PM, Erik Joelsson <erik.joels...@oracle.com>wrote:

**
On 2013-03-04 23:55, Martin Buchholz wrote:



On Mon, Mar 4, 2013 at 3:34 AM, Erik Joelsson <erik.joels...@oracle.com>wrote:

  Thanks for the suggestion Martin!

I created 8009376 for this and will try to get it done.


  There's already a bug for this:

    8006988 : build-infra: Configure fails if 'cl' is in path on linux

  Oop, thanks!

  I've refreshed my patch to implement the consensus.
http://cr.openjdk.java.net/~martin/webrevs/openjdk8/COMPILER_CHECK_LIST/

  And thanks again. I think this looks good.

/Erik

Reply via email to