Development of module-script generator for lr-mess, lr-mame and mame standalone

Folly

Basically, that's a difficult one.
We only know that the part for generating module-scrips is something that they don't like and therefor it will never be accepted.
At some point I have been thinking, can I just install a system or even remove a system without generating a module script.
It might be possible to just use, for example, the functions mkRomDir, addEmulator and addSystem from within the script without the use of a generated module script.
Though, when above is possible, installing a system via the commandline using ./retropie_packages would, probably, not be possible because of the different structure of the module-script.
So that is also something that, I think, will not be liked too.
So basically what I am saying is that it's very difficult to make something that will be liked. unless there is a proper discussion and everything is build from the ground up.
I think that is the only way to prevent unnecessary loss of time and commitment.

Folly

@Widge

I would like to ask you, are you already familiar with the arcade "categories" I implemented in the script ?
( shooter, puzzle etc )

Soon I will be able to update the database to mame 0.245.
This means that I probably can add the category "lightgun" to the database too.

Would that be a nice option ?

Widge

@Folly
Thanks for asking me. I apologise that it's taken so long to respond.

I'm not too familiar with the categories. All I did was use the script to install mess for Atari800, so I could see if I could use mess to play the Atari lightgun games.

Ultimately, I found that whilst mess has lightgun support the drivers for Atari800 do not appear to make use of it.
This file seems to indicate that all types of mouse-like input for A800 are completely off the table: https://github.com/libretro/libretro-atari800/blob/master/libretro/core-mapper.c

Adding a lightgun category to your database does sound like a good option to have, but at present I'm not familiar enough with your script to give a qualified answer.
But again, thank you for thinking of me.

Folly

@Widge

Ultimately, I found that whilst mess has lightgun support the drivers for Atari800 do not appear to make use of it.

indeed, seems to be a problem with some drivers for MAME/lr-mame/lr-mess.

This file seems to indicate that all types of mouse-like input for A800 are completely off the table: https://github.com/libretro/libretro-atari800/blob/master/libretro/core-mapper.c

Correct, for that a800 libretro-core it looks like mouse is disabled when no android is used.
You could open a new topic, and try to change or patch the source to test adding mouse support to it.

Adding a lightgun category to your database does sound like a good option to have, but at present I'm not familiar enough with your script to give a qualified answer.

When I can I will try to update the database with lightgun info.
I am still waiting for updates on : https://www.progettosnaps.net/index.php .

Folly

The script has been updated to use the mame 0.245 data.

Within the 0245 database the lightgun data has been added.
So it's now possible to install the arcade category "lightgun".
Because there are no vertically playable lightgun roms the "lightgun90º" entry's are missing.

Although the category lightgun exists now, there are no artwork files or overlays files yet to support the sinden lightgun.
So if you want to make use of that, you still have to add that manually.

From the restricted part 122 of 329 lightgun games are available.

testudo

@Folly
Added the latest add-mamedev-systems.sh scriptmodule (0245.02) on my Ubuntu system and I am getting this error when trying to add new systems:

/home/testudo/RetroPie-Setup/scriptmodules/supplementary/add-mamedev-systems.sh: line 806: /home/testudo/RetroPie-Setup/scriptmodules/supplementary/add-mamedev-systems.sh: line 791: /usr/bin/dialog: Argument list too long: syntax error: operand expected (error token is "/home/testudo/RetroPie-Setup/scriptmodules/supplementary/add-mamedev-systems.sh: line 791: /usr/bin/dialog: Argument list too long")
/home/testudo/RetroPie-Setup/scriptmodules/supplementary/add-mamedev-systems.sh: line 809: /home/testudo/RetroPie-Setup/scriptmodules/supplementary/add-mamedev-systems.sh: line 791: /usr/bin/dialog: Argument list too long: syntax error: operand expected (error token is "/home/testudo/RetroPie-Setup/scriptmodules/supplementary/add-mamedev-systems.sh: line 791: /usr/bin/dialog: Argument list too long")

Any idea on what I have done wrong?

Folly

@testudo

I am aware that there is somehow one bug in the script, only when using a x86 or x86_64 cpu or other OSes.
The bug was there from the start of the front-end.

The issue is that when you want to display all the systems, and install one from there, then the error occurs.

When selecting system in alphabetical order there should be no problem

So :
Are you able to go into front-end ?
Did you select to install a default system from the whole list ?
Do the other menu options work ?

testudo

@Folly

Yes, I was attempting to access the full list of default systems. It has been awhile since I added anything new, so I may have forgotten that bug existed.

Anyway, I am able to use the alphabetical listing options and get systems added, so all good now.

Thanks for the instructions.

Folly

@testudo

Not sure why the error occurs because on a raspberry pi it's no issue.
Perhaps I will dig in it once more.
On the other hand, is it a solution to just remove the menu options 1 and 2 ?
What do you think ?

testudo

@Folly said in Development of module-script generator for lr-mess, lr-mame and mame standalone:

@testudo

Not sure why the error occurs because on a raspberry pi it's no issue.
Perhaps I will dig in it once more.
On the other hand, is it a solution to just remove the menu options 1 and 2 ?
What do you think ?

Removing the menu options seems like a logical fix, be it temporary or permanent.

Folly

@testudo

Seems I found the issue and did an update :
https://github.com/FollyMaddy/RetroPie-Share/commit/e23e2a058872b2a8c05f391ef4020c8bcde092e1

Can you test it for me ?

EDIT :
Just tested both "description list" and "systems list".
Seems the issue is in the"description list".
So the error is not fixed yet.
I will have a second look ;-)

Folly

@testudo

I turned "Display all upon descriptions" off for x86 and x86_64 PC's.
You will now get a message.
Can you test it for me ?

https://github.com/FollyMaddy/RetroPie-Share/commit/8bbaa26a88521183fe1330c1e31ec02ba190e36f

testudo

@Folly said in Development of module-script generator for lr-mess, lr-mame and mame standalone:

@testudo

Seems I found the issue and did an update :
https://github.com/FollyMaddy/RetroPie-Share/commit/e23e2a058872b2a8c05f391ef4020c8bcde092e1

Can you test it for me ?

EDIT :
Just tested both "description list" and "systems list".
Seems the issue is in the"description list".
So the error is not fixed yet.
I will have a second look ;-)

@Folly

I received the following errors:
script errors.png

Folly

@testudo

Yes, I know, the earlier commit didn't fix anything.
In the last commit "Display all upon descriptions" will now only work on arm cpu's.
Can you update to 245.05 and see if you get a message instead of the error ?

testudo

@Folly said in Development of module-script generator for lr-mess, lr-mame and mame standalone:

@testudo

Yes, I know, the earlier commit didn't fix anything.
In the last commit "Display all upon descriptions" will now only work on arm cpu's.
Can you update to 245.05 and see if you get a message instead of the error ?

Looks like it worked:

Screenshot from 2022-08-09 08-32-23.png

dmmarti

@Folly … I don’t use my RetroPie build much anymore (mainly use my Windows PC now), but I had a look at some of your previous posts and decided to test a few things out.

I’ve now been able to add support for Videoton TV 64 using floppy disks. This opens up a few more games using the latest MAME software list for tvc_flop. I’ve been playing Galaxy alot lately now.

Here’s the stand alone MAME command I used:

mame tvc64 -exp1 hbf -flop1 “Path/To/Rom/game.dsk”

Once the system is started, instead of using the cassette command of:

LOAD <enter>

Use this instead:

LOAD"*" <enter>

You might have already did this system, but I wasn’t sure. By looking at the -listslots for tvc64 I was able to deduce how to add floppy support. There are quite a few games in the tvc_flop software list that are pretty fun.