nystudio107 / craft-scripts

Shell scripts to manage database backups, asset backups, file permissions, asset syncing, cache clearing, and database syncing between Craft CMS environments

Home Page:https://nystudio107.com/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Permission set with set_perms.sh on ./craft being 644

jayhlee opened this issue · comments

Question

set_perms.sh sets the Craft script (./craft) to 644, which ends up preventing any Composer Scripts that use Craft scripts from being executed. I know that changing the Craft script to have executable permissions will resolve this (chmod a+x craft), but I was wondering if setting the Craft script to 644 was intentional.

Without the Craft script being executable (specifically in the production environment), Craft scripts like clear-caches and migrating—which are part of the post-craft-update commands—won't work. So I'm figuring the 644 permission being set on the Craft script is not intentional? Not sure, but figured I'd ask the wizard directly. Thanks!

So the craft file is actually PHP code... and the best cross-platform way to run it is:

php craft xxx

...and this does not require that the executable bit is set on the file, since it is run through php.