> On Jan 21, 2017, at 5:08 AM, Alan Bateman <alan.bate...@oracle.com> wrote: > > On 21/01/2017 02:32, Mandy Chung wrote: > >> Updated webrev to put the temporary file in the same containing directory of >> the target file: >> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8173096/webrev.01 >> >> I want to fix this regression in jdk-9+154. We can revisit the approach >> after this fix if necessary. >> > This looks okay, I guess an alternative fix would be for CreateJmods.gmk to > set the permissions.
I prefer jmod tool to create a file with the proper permission in this fix. Thanks > There are a couple of issues that have come up here and I agree that we > should re-visit once this regression is out of the way. Thanks Mandy
