Mausberry Shutdown Script Doesn't Save Metadata

meleu

@lostless now I need you to do the same thing for /etc/rc.local and /etc/switch.sh (the last one is supposed to be my v1.7 script, but please, paste it on ghostbin so I can really check if everything is fine)

lostless

/etc/rc.local
https://ghostbin.com/paste/goucc

/switch.sh
https://ghostbin.com/paste/5agap

meleu

@lostless

ls -l /etc/switch.sh

lostless

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

ls -l /etc/switch.sh

-rwxrwxrwx 1 root root 1889 Jul 29 02:42 /etc/switch.sh

meleu

@lostless
damn! I'm not getting what's wrong! Let's try this:

sudo bash -x /etc/switch.sh & 2> /home/pi/script-debug.txt

And then press the f%#&ing button. Pretty please... :)

It should turn off your pi...

lostless

@meleu aaaaaaaaaaaaaannnnnnnddddd...................

...........NO GOOD. >:( sorry man.

lostless

@meleu

 + GPIOpin1=23
+ GPIOpin2=24
+ echo 23
/etc/switch.sh: line 22: echo: write error: Device or resource busy
+ echo in
+ echo 24
/etc/switch.sh: line 24: echo: write error: Device or resource busy
+ echo out
+ echo 1
+ file=/sys/class/gpio/gpio23/value
+ sleep 30
+ inotifywait -qq -e modify /sys/class/gpio/gpio23/value

the readout of that and its stuck. My terminal has not finished what its doing.

meleu

@lostless are you able to use IRC? maybe it's better than this forum for an online conversation... :P

edit I'm at #retropie at irc.freenode.net

lostless

@meleu never used IRC. But willing to learn. Just need a client I assume? and somehow the #retropie is relevant somehow.

meleu

@lostless let's continue here then. Is your terminal "frozen"? I mean, the script didn't stopped, right?

lostless

@meleu i figured out irc. I'm there

meleu

@lostless Great. On your irc client, perform these commands:

/server irc.freenode.net
/join #retropie

lostless

@meleu im in, i wasnt kidding. LOL

meleu

@lostless I can't see any lostless there. What's your nickname?

cyperghost

#!/bin/bash
emucall=$(sed -n 4p /dev/shm/runcommand.info)
emupid=${emucall#* }
pos=$(expr ${#emucall} - ${#emupid})
$emupid=$(pgrep -f -n ${emucall:0:$pos})
kill $emupid

The code sniplet above should still do the job as it was introduced for a few days and was titled "complex" - it isn't ;)
It's robust string operation and searches for first occurence for space and then kills the latest process :)
The using of this regex in that way isn't maintainable imho for someone who isn't deep in this thing :) That's really nerdy now.

But your outfindings are great but do this only for ScummVM? I think to introduce the -n switch in pkill command will do the job :) without playing around with RegEx.

cyperghost

@meleu

I've posted something above.
I'm happy with your outfindings but I would prefer two ways.

a version with the inotify and a version without :)

I will use your code sniplets also for my newest build (but I will no use a mausberry - the platine is much to big)
To introduce a new bash windows with selectable options... Is it forced to be in first instance. Mean I can see it as on overlay due an emualtor? Do you know?

hansolo77

Now testing v1.5b (with the loop fix):

Shutdown from ES normally (play game, return to ES, verify "Last Played" updated, then button)
PASS - Last Played saved prior to successful shutdown
Shutdown from ES during video screensaver (play game, return to ES, verify "Last Played" updated, wait for screensaver to start, then button)
PASS - Last Played saved prior to successful shutdown (although there was a delay after pushing the button before the video stopped...)
Shutdown system while playing a game (RetroArch) (play game, return to ES, verify "Last Played" updated, play a different game, then button while still in game)
PASS - Last Played saved prior to successful shutdown (the previous game as well as the one I was playing when I shut down both saved to the list. Sleep delay still needs tweaked, takes a long time to start shutting down ES after exiting a game)
Shutdown system while playing a game (non-RetroArch) (play game, return to ES, verify "Last Played" updated, play a different game, then button while still in game)
PASS - Last Played saved prior to successful shutdown (previous game and active game both saved metadata)

COMPLETE SUCCESS!

I'm very happy you guys! You came forth and solved my problem! I'm going to try tweaking the sleep command a little, but for the most part.. yay!! I don't have the ScummVM system installed on my Pi, so I can't test any further if you guys try to come up with a solution for that. Thank you so much!!! Should I mark as resolved or are you guys still going to try tackling this further?

lostless

@hansolo77 If its solved for you, then by all means. @meleu is just trying to make a script that is more efficient on the cpu

hansolo77

I think the delay I found with the video screensaver comes from it waiting to start shutting down until AFTER the current playing video finishes. I just went to shutdown the system to make a backup of the working SD card, and a new video had just started, so I watched and waited until the current video stopped, then it shut down. Not a big deal, just something I noticed.

meleu

@hansolo77 when you are on IRC and see me (meleu) there, feel free to contact me and we will make more tests to "fix" that infinite loop checking the button pressing every single second.

Feel free to mark it as solved if it works for you. I am (was?) just trying to find a strong solution, for every scenario.