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 >