Re: RFR: 8136656: Check in blessed-modifier-order.sh

2015-09-18 Thread Magnus Ihse Bursie
On 2015-09-17 18:09, Chris Hegarty wrote: On 17 Sep 2015, at 17:08, Martin Buchholz wrote: Done! The updated webrev ( in-place ) looks good to me Martin. Agree. /Magnus -Chris. On Thu, Sep 17, 2015 at 1:14 AM, Chris Hegarty wrote: On 16 Sep 2015, at 20:36, Martin Buchholz wrote: H

Re: RFR: 8136656: Check in blessed-modifier-order.sh

2015-09-17 Thread Chris Hegarty
On 17 Sep 2015, at 17:08, Martin Buchholz wrote: > Done! The updated webrev ( in-place ) looks good to me Martin. -Chris. > On Thu, Sep 17, 2015 at 1:14 AM, Chris Hegarty > wrote: > On 16 Sep 2015, at 20:36, Martin Buchholz wrote: > > > Hi guys, > > > > Here's the requested script checkin:

Re: RFR: 8136656: Check in blessed-modifier-order.sh

2015-09-17 Thread Martin Buchholz
Done! On Thu, Sep 17, 2015 at 1:14 AM, Chris Hegarty wrote: > On 16 Sep 2015, at 20:36, Martin Buchholz wrote: > > > Hi guys, > > > > Here's the requested script checkin: > > > > https://bugs.openjdk.java.net/browse/JDK-8136656 > > > http://cr.openjdk.java.net/~martin/webrevs/openjdk9/blessed-m

Re: RFR: 8136656: Check in blessed-modifier-order.sh

2015-09-17 Thread Chris Hegarty
On 16 Sep 2015, at 20:36, Martin Buchholz wrote: > Hi guys, > > Here's the requested script checkin: > > https://bugs.openjdk.java.net/browse/JDK-8136656 > http://cr.openjdk.java.net/~martin/webrevs/openjdk9/blessed-modifier-order.sh/ Looks fine. Just add an appropriate licence header, move t

Re: RFR: 8136656: Check in blessed-modifier-order.sh

2015-09-17 Thread Magnus Ihse Bursie
On 2015-09-16 21:36, Martin Buchholz wrote: Hi guys, Here's the requested script checkin: https://bugs.openjdk.java.net/browse/JDK-8136656 http://cr.openjdk.java.net/~martin/webrevs/openjdk9/blessed-modifier-order.sh/ Martin, We're trying to retire the "make/scripts" directory, since this is

RFR: 8136656: Check in blessed-modifier-order.sh

2015-09-16 Thread Martin Buchholz
Hi guys, Here's the requested script checkin: https://bugs.openjdk.java.net/browse/JDK-8136656 http://cr.openjdk.java.net/~martin/webrevs/openjdk9/blessed-modifier-order.sh/