openPMD / openPMD-api

:floppy_disk: C++ & Python API for Scientific I/O
https://openpmd-api.readthedocs.io
GNU Lesser General Public License v3.0
138 stars 51 forks source link

openpmd-pipe: set correct install permissions #1459

Closed franzpoeschel closed 1 year ago

franzpoeschel commented 1 year ago

openpmd-pipe is currently installed with odd permissions, since the install() command manually specifies the permissions. This implicitly strips all permissions that would normally be installed, but are not listed.

Before this fix:

$ ls install_dir/bin/ -lisah
total 16M
177477809 4.0K drwxr-xr-x 2 franzpoeschel franzpoeschel 4.0K Jun  7 17:26 .
177477392 4.0K drwxr-xr-x 5 franzpoeschel franzpoeschel 4.0K Jun  7 17:26 ..
177478070  16M -rwxr-xr-x 1 franzpoeschel franzpoeschel  16M Jun  7 17:07 openpmd-ls
177479928 4.0K -rwx------ 1 franzpoeschel franzpoeschel  411 May 22 10:36 openpmd-pipe

With the fix:

$ ls install_dir/bin/ -lisah
total 16M
177477809 4.0K drwxr-xr-x 2 franzpoeschel franzpoeschel 4.0K Jun  7 17:26 .
177477392 4.0K drwxr-xr-x 5 franzpoeschel franzpoeschel 4.0K Jun  7 17:26 ..
177478070  16M -rwxr-xr-x 1 franzpoeschel franzpoeschel  16M Jun  7 17:07 openpmd-ls
177479928 4.0K -rwxr-xr-x 1 franzpoeschel franzpoeschel  411 May 22 10:36 openpmd-pipe