Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Writes path info of menu selection #416

Merged
merged 1 commit into from Jul 15, 2021
Merged

Conversation

kaicherry
Copy link
Contributor

@kaicherry kaicherry commented Jun 24, 2021

Writes to current path and "full path" of whatever is highlighted in the menu for external scripts to monitor.

This was implemented specifically to support a dynamic marquee solution for MiSTer in arcade cabinets, but in a way that it is not "vendor specific". See https://pixelcade.org/mister/ for an example of a practical application.

Writes to current path and "full path" of whatever in highlighted in the menu for external scripts to monitor.
@sorgelig
Copy link
Member

sorgelig commented Jun 26, 2021

not every user likes a spy behavior

@kaicherry
Copy link
Contributor Author

kaicherry commented Jun 26, 2021

Please see this discussion/topic on the forum here: https://misterfpga.org/viewtopic.php?f=29&t=2011 - I genuinely do not understand what you mean by that. This was specifically designed to mimic the behavior the software has, eg writing the current core name to /tmp/CORENAME.

This is all this code does. I don't understand the 'spy behavior' connotation. These files are "monitored" the way everything else is...via inotify-type scripts.

@alinke
Copy link

alinke commented Jul 15, 2021

Hi Alexey, I am the creator of Pixelcade, a line of LED and LCD based marquees for arcade machines http://pixelcade.org. We have had many asks from MiSTer users for Pixelcade marquee support, Pixelcade/MiSTer beta user here ==>https://www.youtube.com/watch?v=hhrPBrxSyRQ. This PR simply adds to the file path the current game that has been selected in addition to the console that is already there so there should be no concerns there. This is the only way we know of to add Pixelcade support to MiSTer so please consider merging this PR and of course let me know if you have any questions. Thanks much, Al

@sorgelig sorgelig merged commit 2a8d227 into MiSTer-devel:master Jul 15, 2021
dentnz pushed a commit to dentnz/Main_MiSTer that referenced this pull request May 12, 2022
Writes to current path and "full path" of whatever in highlighted in the menu for external scripts to monitor.
dentnz pushed a commit to dentnz/Main_MiSTer that referenced this pull request May 12, 2022
Writes to current path and "full path" of whatever in highlighted in the menu for external scripts to monitor.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants