Mausberry Shutdown Script Doesn't Save Metadata

cyperghost

@caver01 Well I think the bash script is downloadable by mausberry so the usecase is more on the bash side. I have to agree - I never watched the GPIO part. Feel free to post your python script here. We are open for every suggestion.

@hansolo77 and @lostless

Introduction v 1.5 of the shutdown script here

Much greetings to @meleu for his great help

NamErehWon

@meleu I was going off the script that was posted earlier in the thread. It could be easily changed to use inotify to check the "file" that is the GPIO value. I haven't had a chance to play around with the inotify stuff yet, but I can write something up once I've tried it.

@cyperghost The debounce also helps with preventing unintended shutdown due to the wires/traces picking up rf and registering a high on the GPIO pin. You know, make sure you have an actual rising edge and not a blip. I know that the GPIO pins have a mode where they report back rising or falling edges, but I haven't experimented with it to see if that is debounced already. If so, then switching to checking for a rising edge would negate the need for debouncing in the script that triggers your shutdown script.

cyperghost

@NamErehWon Can you please check out. Detecting flanks on GPIO within bash would be fine. I know it works with wiringPi...
So you can speed up GPIO change detection via IRQ - but I think that will be better discussed here

hansolo77

I wonder if you guys are just making it more complicated than it has to be. I also have a reset button wired up to the GPIO to act as a failsafe way to exit out a running game. The scripts are located here:
https://www.reddit.com/r/raspberry_pi/comments/2yw4fn/finally_set_up_retropie_complete_with_a_gpio/cpdhtpu/
So why not just incorporate that into the script? Just execute the escape script then initiate the shutdown? Of course if you wanted to make it beneficial for everybody, you'd probably want to add it into the Mausberry Script. The only problem I think is the type of scripts being used (shutdown is BASH why escape is PYTHON), so I don't think you can add it into the same script all at once..

cyperghost

@hansolo77 No it isn't more complicated. You asked for a bash script. With C or python it is indeed an easy approch :) because string operations are much easier to do.

You can see the results in different codestyle if you compare mine and @meleu.
meleu use the tools (commands) every unix offers in an elegant way. I don't know so much commands so I'm poking around with stringscuts and string compare. That's exact the way you would do it in any coding language.

BASH is immense powerful in working with files and data. As in every unix system hardware/devices are mounted into filesytem you can "communicate" to these devices via file manipulation :) So for fast results you can use BASH.

Of course you can use the same bash-commands to work in python and do not need the shutdown script anymore. Or you can use commands like emupid=$(pgrep -f ${emucall:0:$pos}) to extract emulator calls. There are 1000 solutions.

Our intention is to get an ultimate shutdown without using annother process in background to finsh ES and it's emulators (if they are running) as far as I see the python code it just kills obtained PID, like we do in version 1.5 of our bash script.

EDIT:
The python script will not work for zDoom for example. Because it just asks for "/opt/retropie/emulators/" path.

@meleus intention works in this case as long as the call is initiated by runcommand.sh (and this is the normal usecase)

hansolo77

And knowing is half the battle! -G.I.Joe!

meleu

@cyperghost said in Mausberry Shutdown Script Doesn't Save Metadata:

With C or python it is indeed an easy approch :) because string operations are much easier to do.

I doubt it! :-)

hansolo77

Testing v1.5 from here now....

Exit from ES normally after playing a game
FAIL - Metadata did not save (although system did shut down)
Exit from running screensaver after playing a game
FAIL - Metadata did not save (although system did shut down)
Exit from playing a game (RetroArch)
FAIL - Metadata did not save (although RetroArch did close, return to ES, then shutdown.. Suggestion: shorten the wait time after returning to ES. I didn't know if it was working until it shut down lol)
Exit from playing a game (SuperMarioWar)
FAIL - Metadata did not save (although SuperMarioWar did close, return to ES, then shutdown. I also discovered that my Streets of Rage Remix game isn't working (crap) so I have to fix that soon)

Take away - Looks like the script is properly closing out of the game I'm playing and returning to ES for a shutdown, but it's not saving the metadata when it shuts that down. I wonder if it's getting hung up in the if functions, not finding a valid true and going to the bottom of the script to the original poweroff without exiting ES first...

So quick question... anybody know how to make a profile in Putty so it can automatically log in (username/password) so I don't have to keep typing it every time I try one of these tests? :)

meleu

@hansolo77 said in Mausberry Shutdown Script Doesn't Save Metadata:

Exit from ES normally after playing a game
FAIL - Metadata did not save (although system did shut down)

Did you succeed saving metadata in this scenario with some other script?

I'm asking because several posts above I said that killing ES wouldn't save metadta and, If I understood well, @cyperghost said that you succeeded.

I think I misunderstood his answer.

anybody know how to make a profile in Putty so it can automatically log in

Maybe this can help.

lostless

@meleu I can confirm the 1.2 script works fine if you shut off from emulation station. But not from game

meleu

@lostless said in Mausberry Shutdown Script Doesn't Save Metadata:

@meleu I can confirm the 1.2 script works fine if you shut off from emulation station.

Just to be really sure: "works fine" means "saves all new games I added to Favorites and/or Last Played", right?

lostless

@meleu yes.

@meleu said in Mausberry Shutdown Script Doesn't Save Metadata:

Just to be really sure: "works fine" means "saves all new games I added to Favorites and/or Last Played", right?

meleu

@lostless please, give me a few minutes and I'll tweak the script v1.2 a little and post it to you try.

meleu

@lostless please, install inotify-tools:

sudo apt-get install inotify-tools

and then try the script below:

#!/bin/bash
# Before using this script you need to install inotify-tools:
#
#    sudo apt-get install inotify-tools
#
# I'm just trying to help with the script logic here.
# I DON'T HAVE A MAUSEBERRY AND DIDN'T TEST THE GPIO STUFF HERE!
#
#########################################################
#  U S E   I T   A T   Y O U R   O W N   R I S K ! ! !  #
#########################################################
#
# meleu - July/2017
# kudos for @cyperghost , who is very persistent in help you guys! :-)

#this is the GPIO pin connected to the lead on switch labeled OUT
GPIOpin1=23

#this is the GPIO pin connected to the lead on switch labeled IN
GPIOpin2=24

echo "$GPIOpin1" > /sys/class/gpio/export
echo "in" > /sys/class/gpio/gpio$GPIOpin1/direction
echo "$GPIOpin2" > /sys/class/gpio/export
echo "out" > /sys/class/gpio/gpio$GPIOpin2/direction
echo "1" > /sys/class/gpio/gpio$GPIOpin2/value

file="/sys/class/gpio/gpio$GPIOpin1/value"

while inotifywait -qq -e modify "$file" ; do
    power="$(cat "$file")"
    [[ "$power" == 0 ]] && continue

    emu_command="$(sed -n 4p /dev/shm/runcommand.info)"
    [[ -n "$emu_command" ]] && pkill -f "${emu_command%% *}" && sleep 5

    espid=$(pgrep -f "/opt/retropie/supplementary/.*/emulationstation([^.]|$)")
    if [[ "$espid" ]]; then
        touch /tmp/es-shutdown && chown pi:pi /tmp/es-shutdown
        kill "$espid"
    fi

    sudo poweroff
done

lostless

@meleu Im giving it a shot. I really don't want to reconfigure my pi, but oh what the heck. Ive only had to reconfigure 10 times or so. whats another for the most perfect script for mauseberry (or other switch) users.

meleu

@lostless I edited the script, be sure to get the current one!

lostless

@meleu i saw that. whew.

lostless

@meleu killed all switch functionality. As in it no longer functions as a shutdown switch.

meleu

@lostless OK. I'll try a little further...

lostless

@meleu i see you removed some stuff from the original script

while [ 1 = 1 ]; do
power=$(cat /sys/class/gpio/gpio$GPIOpin1/value)
if [ $power = 0 ]; then
sleep 1
else

that was removed. any reason to that?