Retroarch: Next attempt to enable save on exit

[gizmo98]

I like RGUI, but its annoying it cannot be used to setup things persistent. Newbies will be confused. Experienced users need our shell scripts or an editor to set things. All efforts enabling save on exit were disastrous by now. Replacing appendconfig or using #include does not help. We could also disable RGUI at all.

I've taken the liberty to write a startup script which makes a copy of our current system config, enables save on exit, starts retroarch with this copy and writes after exit all proved to be save settings to system and global config file.
https://github.com/RetroPie/RetroPie-Setup/compare/master...gizmo98:retroarch-save-on-exit?expand=1

Things like input/audio/video "driver" stuff will be written to configs/all. All shader/overlay/audio stuff will be written to configs/$system.

Don't know if this a viable way. It seems to work. What do you all mean?

[HerbFargus]

I like the idea, I don't have time to test anything right now but managing settings through the rgui seems a lot more intuitive than needing to always set configs manually.

As far as I see it: Basically all we need is for whatever edits are made in the RGUI to automatically save (either for the specific game or system and/or both) and then of course that newly saved retroarch.cfg file is what shows up when you go to edit the custom configs from the runcommand menu as well.

[gizmo98]

Integrate settings per rom should be easy. If $rom.cfg exists store settings in this file. Else use system.cfg file.

[dankcushions]

I like it! personally one of the first things I do with a retropie install is turn save after exit on. there's just too many things I need to change and the RGUI is the best way to do that IMO.

[joolswills]

Yeh the idea is nice. There might be cases where it saves something where someone doesn't want it but generally I think it covers what most people would want - maybe we should leave it off by default, and get some testing done first though as it's a fairly big functionality change.

[gizmo98]

Ok. I disabled it by default. Do you have any tip how to speed up ini read and write operations? It takes 2-3s to store all settings. I know python's configparser can do this in memory (read whole file --> edit everything --> write file back). I had another problem using arguments with whitespaces like rom name and rom config. I worked around it but there could be a better solution.

[joolswills]

for args just process the array of args you get and pass that on

eg the content of "$@" or so but the ini / read write will re-read the whole file each time, so there should be ways to speed it up.

[joolswills]

and of course there is the option of using python should shell scripting prove too slow.

[joolswills]

just did some experimentation - here's a way to speed it up.

You generate some sed expressions to extract the keys you want from the first file, replacing them with an expression to replace them in the second file.

eg

#!/bin/bash

source "scriptmodules/inifuncs.sh"

# system specific settings
local_vars=(
    video_scale
    video_vsync
    #video_force_srgb_disable
    video_hard_sync
    video_hard_sync_frames
    video_frame_delay
    #video_black_frame_insertion
    video_threaded
    #video_shared_context
    video_smooth
    video_force_aspect
    video_scale_integer
    video_aspect_ratio
    video_aspect_ratio_auto
    video_crop_overscan
    video_shader
    video_shader_enable
    #video_refresh_rate
    video_allow_rotate
    video_rotation
    audio_sync
    #audio_latency
    audio_rate_control
    #audio_rate_control_delta
    #audio_max_timing_skew
    input_player1_analog_dpad_mode
    input_player2_analog_dpad_mode
    input_player3_analog_dpad_mode
    input_player4_analog_dpad_mode
    input_player5_analog_dpad_mode
    input_player6_analog_dpad_mode
    input_player7_analog_dpad_mode
    input_player8_analog_dpad_mode
    input_overlay_enable
    #input_overlay_hide_in_menu
    input_overlay
    input_overlay_opacity
    input_overlay_scale
    custom_viewport_width
    custom_viewport_height
    custom_viewport_x
    custom_viewport_y
    fps_show
    rewind_enable
)

re=()
for var in "${local_vars[@]}"; do
    re+=("-e" "s|^$var = \"(.+)\"|s/^ *$var *=.*/$var = \"\1\"/|p")
done

nre=()
while read -r line; do
    nre+=("-e" "$line")
done < <(sed -rn "${re[@]}" /opt/retropie/configs/all/retroarch-tmp.cfg)

sed -ri "${nre[@]}" /opt/retropie/configs/all/retroarch.cfg

re+=("-e" "s|^$var = \"(.+)\"|s/^ *$var *=.*/$var = \"\1\"/|p") is not the most readable line of reg exp/code ever, but it works in my testing, and is significantly faster 0.35s vs 2.47s for doing it one value at a time with the ini functions.

[joolswills]

as retroarch always saves out it's configs with the same spacing etc, I have simplified the regular expression - more readable now :)

[joolswills]

this is not a complete solution actually, as it will only replace existing keys. but I'm sure it could be improved to replace existing keys, and then add any new ones. an additional grep on the destination, to work out which keys are missing, and some extra processing etc.. Or doing it in Python :)

[joolswills]

ok so a bit more tweaking - this should append new lines, and replace existing ones. Takes about 0.45s to run on my rpi2

for var in "${local_vars[@]}"; do
    grep_re+=("-e" "^[# ]*$var")
done

matches=($(grep -E -o "${grep_re[@]}" /opt/retropie/configs/all/retroarch.cfg | tr -d "# "))

for var in "${local_vars[@]}"; do
    if [[ " ${matches[*]} "  =~ " $var " ]]; then
        re+=("-e" "s|^$var = \"(.+)\"|s/^[# ]*$var *=.*/$var = \"\1\"/|p")
    else
        re+=("-e" "s|^$var = \"(.+)\"|$ a $var = \"\1\"|p")
    fi
done

while read -r line; do
    nre+=("-e" "$line")
done < <(sed -rn "${re[@]}" /opt/retropie/configs/all/retroarch-tmp.cfg)

sed -ri "${nre[@]}" /opt/retropie/configs/all/retroarch.cfg

[gizmo98]

Wow. This is really nice stuff.

[joolswills]

Thanks - well I'm not sure it's too pretty, but it works :)