Re: Change in Install behaviour: no longer makes target file writable after copying, causing build errors

On 4/11/21 4:02 PM, Bill Deegan wrote:
> Indeed.
> Here's the old logic.
> 
>     shutil.copy2(source, dest)
>     st = os.stat(source)
>     os.chmod(dest, stat.S_IMODE(st[stat.ST_MODE]) | stat.S_IWRITE)
> 
> Interestingly previously the above would only apply to single file 
> Install()'s and not to whole directories worth which would use 
> shutil.copy2() (which copied permissions).
> So the current code is more consistent than the previous code, but I can 
> understand your expectations..
> 
> I think ideally you could set the permissions for installing?
> Maybe to be masked with original file..
> 

we maybe have to do some thinking about the philosophy within SCons in 
general with the readonly attribute - I knew I had seen some open issues 
about it and I with a quick check I  find #3206 and #2693.