Emulation Station game-start script not executable

alxld

Hi everyone. I'm running emulationstation v2.9.6rp and trying to call a game-start script, but getting an error in the es_log.txt that the script is not executable. As far as I can tell, all of the permissions are correct and the shebang is in place. I can run the script from the terminal and even from ES's file manager with no issues. It just won't run when it starts the games.

Feb 12 20:49:22 lvl2:   Attempting to launch game...
Feb 12 20:49:22 lvl1:   /home/myname/.emulationstation/scripts/game-start/UMMapper.py is not executable. Did you 'chmod u+x'?. Skipping this script.
Feb 12 20:49:22 lvl1:   /home/myname/.emulationstation/scripts/game-start/history is not executable. Did you 'chmod u+x'?. Skipping this script.

myname@retropie-pc:~/.emulationstation/scripts/game-start$ pwd
/home/myname/.emulationstation/scripts/game-start

myname@retropie-pc:~/.emulationstation/scripts/game-start$ ll
drwxrwxr-x 2 myname myname 4096 Feb 12 20:45 history/
-rwxrwxr-x 1 myname myname 2688 Feb 12 20:45 UMMapper.py*

myname@retropie-pc:~/.emulationstation/scripts/game-start$ head -4 UMMapper.py 
#! /usr/bin/python
import csv, sys, os

romname = sys.argv[2]

I've tried a simple .sh script, too, with the same results (runs from command line and ES file manager, 'not executable' from ES).

Thanks for taking a look!
-Aaron

mitu

Can't see anything wrong with the permissions.
Is the /home/myname folder on a Linux partition or is it mounted from a fat32 or ntfs partition ?

alxld

@mitu This is all installed on a single partition under Ubuntu. I checked permissions on all of the directories above scripts/game-starts, including the link back to /opt/retropie from ~/.emulationstation.

mitu

Not sure why ES thinks your script is not executable, given the permissions. The extension doesn't matter since ES checks just the execute bit before logging this message.

Can you try changing the script so that it's just user executable (mode 0755) ?

alxld

@mitu That was it! I had it as 775, but 755 must be required. Thanks so much!

-Aaron

Lolonois

@alxld Thanks for reporting. The warning message is confusing. My bad. Filed a small PR for this.

FWIW this mask is tested on user scripts, if one needs group writeable userscripts adding | S_IRWGRP would be a way.