Missing permission sets in mx_graal.py

[Vladimir Kozlov]

Yes, I use promoted jdk and it does not have write permission on files. 
When mx_graal.py copies it locally, it changes only permission of 
directories but not files.

Thanks,
Vladimir

On 12/19/14 12:29 PM, Tom Rodriguez wrote:
> This fixes the problem with using a read-only JDK as your base image, right?
>
> tom
>
> On Dec 19, 2014, at 12:09 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>
>> --- a/mx/mx_graal.py	Thu Dec 18 18:34:21 2014 -0800
>> +++ b/mx/mx_graal.py	Fri Dec 19 11:37:53 2014 -0800
>> @@ -435,6 +435,8 @@
>>              chmodRecursive(jdk, JDK_UNIX_PERMISSIONS_DIR)
>>              shutil.move(join(_vmLibDirInJdk(jdk), defaultVM), join(_vmLibDirInJdk(jdk), 'original'))
>>
>> +            if mx.get_os() != 'windows':
>> +                os.chmod(jvmCfg, JDK_UNIX_PERMISSIONS_FILE)
>>              with open(jvmCfg, 'w') as fp:
>>                  for line in jvmCfgLines:
>>                      fp.write(line)
>> @@ -447,6 +449,8 @@
>>                      for line in f:
>>                          releaseFileLines.append(line)
>>
>> +                if mx.get_os() != 'windows':
>> +                    os.chmod(releaseFile, JDK_UNIX_PERMISSIONS_FILE)
>>                  with open(releaseFile, 'w') as fp:
>>                      for line in releaseFileLines:
>>                          if line.startswith("SOURCE="):
>>
>> Thanks,
>> Vladimir
>