Request for Review: Execute the build logger with the help of a shell if the x permission is not set

Kelly O'Hair kelly.ohair at oracle.com
Wed Apr 25 19:34:50 UTC 2012


I of course approve. ;^)

We need some code phrase for me when you want a patch integrated into the jdk8/build forest.
I'll deal with the CR, make the owner you in this case, and add any reviewers I have seen in the email
thread.  So how about a code phrase like "Kelly, Please integrate this patch."   ;^)

I await the code phrase. ;^)

-kto

On Apr 25, 2012, at 3:32 PM, Magnus Ihse Bursie wrote:

> On 2012-04-23 18:01, Dmitry Samersoff wrote:
>> Magnus,
>> 
>> I'm second to Kelly.
>> We shouldn't have executable scripts in repository.
> 
> Ok, so here comes a new, even simpler patch. :-)
> 
> http://cr.openjdk.java.net/~ihse/logger.sh-missing-x-permission/webrev.01/
> 
> /Magnus
> 




More information about the build-dev mailing list