search

libretro / retroarch-assets

Assets needed for RetroArch - e.g. menu drivers, etc. Also contains the official branding.
Creative Commons Attribution 4.0 International
170 stars 161 forks source link

Fix permissions and minor updates #314

Closed alfrix closed 5 years ago

baxysquare commented 5 years ago

I'm curious about the permission changes. Is this something we need to do periodically as new icons are added to themes?

alfrix commented 5 years ago

Is this something we need to do periodically...

it should not be needed, just don't make regular files executable

baxysquare commented 5 years ago

I had no idea I was setting the files executable in the first place. My workflow is to create the source PDF on a Mac and then use the script you made for me via a Linux Mint VM to batch create the SVGs and a Imagemagik mogrify command to make the PNGs. I never could quite get the script to work right for me on the Mac but the VM workflow has served me well.

If I were to get in the habit of running chmod -R 766 as part of my workflow, either by adding it to the script or by command, I assume that would that take care of the issue. I tried chmod -R 666, but that won't allow access for the GitHub app I use on my Mac.