Please Test: Adding support for "All", "Favorites" and "Last Played" systems

[lilbud]

@meleu How did you get this to compile? It takes 20 min to compile, and spits out an error message.

[pjft]

@lilbud oh. What error message do you get?

Are you exiting EmulationStation before compiling?

[lilbud]

@pjft It says something to the effect of
error, failed to compile, did you see the error messages?

My answer to that is no, I was busy playing Cards Against Humanity

[lilbud]

@pjft This is what happens, it jumps from 43 to 71%

UPDATE: Got it to compile, using make clean. But is fails to install, claiming that the .sh file doesn't exist.

[meleu]

@lilbud please, do the following:

bash -x ./es-tests.sh 2> err.log

And try to install the this branch again. And when it fail, exit the script and post the err.log on ghostbin.com or something.

edit: by the way, I compiled here and everything seems to be fine.

[pjft]

@lilbud I wouldn't take it as a problem per se. Those missing ones are resources that you could have perhaps previously completed.

If it stops the compilation do let me know. And make sure to exit ES.

I might certainly have messed something in the PR so I'll gladly fix that to compile. Last minute code cleaning can result in that.

@cyperghost The "all" system will work regardless.

The last played will only work if it can tell what games you last played. That's stored in the metadata, which means gamelists. If you don't have any, no game will show up there.

As for the favorites, same thing applies for now: it's metadata so if you don't use gamelists it won't know what to show there.

In the future I plan on implementing custom systems not based on metadata, for themes folders that aren't in use, so conceptually you could create your own favorites system without having gamelists for that.

Unsure if this clarifies it for now, but I don't write any new gamelist or anything to file. Asides from the new favorite metadata field, I only store a setting stating which systems are enabled.

[lilbud]

@meleu I will post the err.log shortly

@pjft I added a few games to my favorites, I launched a game and, upon exiting, the favorite notification flashes for about 2 minutes

ADDDED SONIC 2 TO FAVORITES

[meleu]

@lilbud

@pjft I added a few games to my favorites

Then you compiled fine?

[pjft]

@lilbud well, I really wanted people to make sure they knew what games were their favorites. Is two minutes a long time? :P

That's obviously a bug, great catch. Thanks!

[lilbud]

@meleu Yeah, got it to compile. Installing supposedly fails, as I can't set it as the default, but it runs in the command line perfectly fine.

Here is the error log

https://ghostbin.com/paste/gnnfz

[cyperghost]

@pjft That is exactly what I thought... Thank you for explanation.
The code for future version without gameslist.xml is already made by me.

I "created" the FAV-file format. The FAV-file acts as descriptor of path and emulated system. It stores the info about ROM-path and emu-system to use in one simple file.
With this info you can load any system from any place. I'm sure it's easy to implent into your branch and this would offer all users a way to use favourites.

So maybe you can take a chance and have a look to the BASIC soource-code and translate it to C++. My suggesetion was to first create this in C but then I realised that heavy string manipultion is a better task for annother language.

Take this only as suggestion how a fav-system can be installed without using XML.

Or... that would be my deepest wish you can create a call to a Bash-file by pressing Y-button with marked ROM in menu ;) So you give user the decision what happens by pressing Y-button and maybe you can expand it to X-button also ;) What about this?

How a bash file call can be established can be seen under Prerequisite #5 of topic lfl-create v0.85

[meleu]

@lilbud caught the "bug": you're running the script as root! I'll change the script to avoid that. Thanks.

[UDb23]

@pjft said in Please Test: Adding support for "All", "Favorites" and "Last Played" systems:

I'm working with @UDb23 to create these in Carbon,

Next Weekend you'll get a proposal and "beta" version of the required SVGs ;-)

[lilbud]

My switch logos are already done

[cyperghost]

@lilbud Nice work ;)
I've created favs buttons for IO only maybe @mattrixk can design a set of new one

[lilbud]

What would the es_system.cfg look like for favorite, all games and last played?

[pjft]

@lilbud

There's nothing on es_system.cfg, as they're not hardcoded systems.

If you're looking for the theme folders, they're supposed to be auto-allgames, auto-lastplayed, and auto-favorites.

I've fixed the bug you ran into (thank you!). After launching a game and coming back to ES the application timer starts from zero, which meant that the fade animation would pretty much run until the application got to the same "time" as it had taken for you to get that particular popup to show the first time around. Funny one.

Will update the PR shortly.

[pjft]

@UDb23 No pressure, please - it was more to reassure people :)

[mattrixk]

@cyperghost The logos on IO are just the Carbon SVGs, recoloured to be bi-colour, I didn't actually make them.

[pjft]

I have updated the PR and my branch with a fix for @lilbud 's reported bug! Thanks for sending it over. :)