Run chmod after all directories are made and files are copied.
https://builder.blender.org/download/experimental/ See tmp_T72605
It still needs some changes on builder to work
properly, but anything against the code here ?
Ref T72605
Details
Details
- Reviewers
James Monteath (jmonteath) - Group Reviewers
Platform: macOS - Maniphest Tasks
- T72605: macOS: Blender doesn't have execute permission for non admin accounts
Diff Detail
Diff Detail
- Repository
- rB Blender
- Branch
- tmp_T72605 (branched from master)
- Build Status
Buildable 15503 Build 15503: arc lint + arc unit
Event Timeline
Comment Actions
I ran the patch locally to investigate the permissions issues.
It only changes the executable on the blender file.
Then code sign will fail trying to update it.
jamesm-mpm-macos-bigsur-arm64-01 blender-vdev % ls -la install_release/Blender.app/Contents/MacOS total 506104 drwx------ 3 blender staff 96 Jun 30 09:19 . drwx------ 6 blender staff 192 Jun 30 09:19 .. ---x--x--x 1 blender staff 259123428 Jun 30 09:19 Blender
All folders need to be set with o775 and files to o664 that do not need execute permissions.
Comment Actions
This is a good edge case for failing on code sign.
I will add in the smoke-test a simple CLI test before running the GUI smoke tests.
This will ensure catching any file mode issues.
Comment Actions
The code here as of now should work.. I mean it sets the permission locally just fine (checked by setting a+rwx).
But buildbot build's permissions are still wrong..
Please see if something is wrong with creating the DMG/ copying the files.
Regarding the code sign failure, there was one more: existence of .cstemp files which caused a failure. So might want to add that too.