search

Gemba / skyscraper

Powerful and versatile game data scraper written in Qt and C++.
https://gemba.github.io/skyscraper/
GNU General Public License v3.0
54 stars 14 forks source link

[Enhancement] Support "-p all" for cache commands #65

Open mrsilver76 opened 6 months ago

mrsilver76 commented 6 months ago

Cache commands (such as --cache validate or --cache purge) require a valid platform to be provided with -p.

It would be useful if use of -p all could be used to apply the appropriate command to all platforms within the cache.

For example: Skyscraper -p all --cache validate

The workaround is that I'll have to write a script that runs Skyscraper -p $PLATFORM --cache validate where $PLATFORM contains the name of each directory located in /home/pi/.skyscraper/cache/.

Gemba commented 5 months ago

Good idea. Leave it with me to probably implement it.

Gemba commented 2 weeks ago

I really like the idea, but I looked into the implementation options and that gave me a headache, @mrsilver76 . Mainly because the Settings are set on start and with every platform identified these would have to be reset and initialized with the per-platform settings. It would add another loop around the section where the settings are filled in. It would be quite a design change (from single platform per Skyscraper run to multiple platform on per Skyscraper run).

Thus, don't expect this feature to be implemented soon.

mrsilver76 commented 2 weeks ago

Totally understand and thanks for considering it!

I'll leave it up to you if you want to close the issue.