Skip to content

add SampleModule example#109

Merged
bimmerlabs merged 8 commits intoReyeMe:mainfrom
bimmerlabs:main
Feb 5, 2026
Merged

add SampleModule example#109
bimmerlabs merged 8 commits intoReyeMe:mainfrom
bimmerlabs:main

Conversation

@bimmerlabs
Copy link
Collaborator

Demonstrate use of MODULES_EXTRA

note: there was some discussion on changing include module.mk to make module.mk in shared.mk. I don't fully understand what needs to change for that to work - I'm open to suggestions, but this does work as-is.

demonstration using the modules_extra to add custom plugins to SRL
@bimmerlabs
Copy link
Collaborator Author

there's a typo in the file copy, let me fix that..

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not cross-compatible bat, use the same as run_with_mednafen.bat, just replace mednafen with kronos

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not cross-compatible bat, use the same as run_with_mednafen.bat, just replace mednafen with ymir.

see folder ./tools/scripts/run.bat and ./tools/scripts/run.sh, and add ymir support there

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pointless change?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pointless if case

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pointless for, only once coue per project exists

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove unrelated comment on this line and also on line 14

will deal with the addition of ymir batch scripts in a separate PR
ended up being simple enough so I just included it here
@ReyeMe ReyeMe added the enhancement New feature or request label Feb 1, 2026
or at least, attempt to...
@bimmerlabs bimmerlabs merged commit 40bdbb4 into ReyeMe:main Feb 5, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments