New Idea/Help Needed - es_systems.cfg Selector

FlyingTomahawk

I haven't tested your script yet, will do as soon I get home. Is there a way to load a system file when changing themes?

TMNTturtlguy

@FlyingTomahawk There might be, but i think it would have to be coded into ES as you would need to tell the command that switches theme to also switch the config file. That is way over my head! This tool just makes it quick and easy to change what systems you are showing on your setup without having to go into command line or have a computer and ftp to do it.

meleu

@TMNTturtlguy EUREKA!

I've found what was causing the problem in the script and submitted a PR: https://github.com/TMNTturtleguy/change_es_systems/pull/2

When I found the bug I had one of those how-could-I-didn't-notice-that moments... :-)

cyperghost

@meleu Well and I thought how can meleu do that! An endless loop without break parameter
Like duck and cover
Well done brother :D
I already made a pull request with breaks loop with argument
https://github.com/crcerror/change_es_systems/blob/crcerror-patch-1/change_es_systems.sh

EUREKA

cyperghost

@meleu Can you cleanup code a bit more please?

Remove unneeded sudo on cp/rm >> done @meleu nice catch
Instead of copy and delete I would use mv command only >> sorry early in the morning I ment symbolic linking
No need to work in subfolders
Add path to point to es_systems.cfg path and then ln -s path$&"\es_syststems_hacked" path$&"\es_systems.cfg"

This will also help if someone loads this script and the systems are not installed then nothing happens and ES will not break because v1 up to v4 delete config file first - Can be problematic imho!

Then we push it together to v5 and the code is much cleaner and easier to maintain

I commented your push Done

meleu

@cyperghost said in New Idea/Help Needed - es_systems.cfg Selector:

I commented your push

and I replied there. :)

cyperghost

@meleu
I commented my post above. Got a hard week with some nightshifts.
You're right in moving system files ... therefore I still do not recommend not to copy/delete those files I would rather link those.

The original es_systems.cfg should be kept as es_systems.bak.
I also think that this file will be automatically be copied if es_systems.cfg gets lost. So it's alwyas fail safe!

I think it's just the right moment to thank you for investigate even small bash sniplets. I never used bash and I always learn lots from you and I appreciate the help from you (and lots of others users here!)

Öhmm... a downvote? I think you are using your mobile phone :D right?

meleu

@cyperghost said in New Idea/Help Needed - es_systems.cfg Selector:

a downvote? I think you are using your mobile phone :D right?

Damn! Sorry, it was an accident (again)! 😌

cyperghost

@meleu

Damn! Sorry, it was an accident (again)! 😌

De nada

Isn't the right thread here... How are your achivements in Terranigma?

FlyingTomahawk

Great that you guys got all figured out.
So what and where can I download this? I mean the fixed version.

cyperghost

@FlyingTomahawk
That's @meleu tidy version.
https://github.com/meleu/change_es_systems/tree/patch-1

FlyingTomahawk

@cyperghost

Thank you.

TMNTturtlguy

@cyperghost @meleu thank you both for your help. Unfortunately my schedule remains very busy with little time to actually sit and work on my pi. I will take a look at all of this, but it looks awesome!

@cyperghost i will look at you other methods, but I am not sure why you feel my method is "dangerous" as this works with Es_systems.cfg that are copied and moved to a new location. If they get deleted by accident, the main unedited es_systems.cfg should always remain in its original location and be used by default if the .cfg file does not exist in the alternate location.

cyperghost

@TMNTturtlguy It is dangerous. Think about someone loads your script from your repro and starts it but it is not configurated manually. Then he looses his es_systems.cfgbecause all funktions codes do rm /opt/retropie/configs/all/emulationstation/es_systems.cfg first and then copy the changed files (if it exists) :)

So it may be better to get all config files in one folder and just symlink them. Is there no destination file present then symlinking does not work. That should minimize the risk in breaking something. The risk now is at all minimum.... if you know what to do :)

@cyperghost @meleu thank you both for your help. Unfortunately my schedule remains very busy with little time to actually sit and work on my pi. I will take a look at all of this, but it looks awesome!

Thank you in your effords for the nice theme.
I still learn how to get familiar with bash.

TMNTturtlguy

@cyperghost

Thank you in your effords for the nice theme.
I still learn how to get familiar with bash.

This is why the community is so great, we all have so much we can offer and so many different skills, we can't do it all, but with a little help we can all create some really cool stuff!

@TMNTturtlguy It is dangerous. Think about someone loads your script from your repro and starts it but it is not configurated manually. Then he looses his es_systems.cfgbecause all funktions codes do rm /opt/retropie/configs/all/emulationstation/es_systems.cfg first and then copy the changed files (if it exists) :)

This should not be dangerous at all. Again, i haven't looked at your symlink solution, but my solution is not dangerous because if users do not follow the directions and try to run the script, the script will simply do nothing. The default and main es_systems.cfg file is located here: /etc/emulationstation/es_systems.cfg My script only runs in /opt/retropie/configs/all/emulationstation/es_systems.cfg which is the same as ~/.emulationstaiton/ just written out as the full path. As long as the user never deletes the main es_systems.cfg in the /etc/ folder, there is not risk or danger at all. Here is some more information on this from the wiki: Emulationstation Wiki

TMNTturtlguy

@meleu I just tried your script and it does not execute without the sudo commands. BuZz had updated the retropie startup script a few weeks back and he was the one who instructed that I add the sudo command as he changed user permissions when executing scripts from the retropie menu. Have you tried this script on your build? When did you last update retropie script? Thanks

Edit: adding the sudo command back in to both the copy and remove seems to have worked. With my brief testing it appears that the script works well and all other restart functions work properly. I have to take my girls out for a while so i will test more extensively later.

meleu

@TMNTturtlguy maybe you changed the ownership of those files with your previous script. If they're owned by user pi there's no need for sudo.

cyperghost

@TMNTturtlguy @meleu Is 100% correct! Check if some of your config files belong to root... Seems so because you worked with sudo in your last script. So I falsly said to him that he better removes sudo commands what he already did :)

But I think your script is very special ... I really can't imagine to use it :)
It was real fun to play a bit around with and learn a bit new. So I appreciate every comment/tip/hack from more advanced people. I wasn't able to trigger bugs because I commented every file operation out and tested just the behaviour of function call, loop break and restart ability. I think meleu did the same.

Two locations of es_systems....

Yes I rember now, but I alway work in /etc/emulationstation > Seems that I like risk

TMNTturtlguy

@cyperghost @meleu when I get back on my pi I will repudiate my setup script, bet here is the info from buzz when working on this in the past. Thanks link text

cyperghost

Yes but this behaviour was changed.
https://retropie.org.uk/forum/topic/11050/restart-es-via-bash-script/7

There is really no need to act as root. As long as your script works for you it is really no problem but it seems that user privileges causes some errors.