yoshisuga / MAME4iOS

Multiple Arcade Machine Emulator for iOS, iPadOS, tvOS, macOS (Catalyst)

Other

635 stars 103 forks source link

update to MAME 240, iOS 13.4, remote target for macOS #372

Closed ToddLa closed 2 years ago

ToddLa commented 2 years ago

update to MAME 240, iOS 13.4, remove target for macOS

MAME 240
update minimum version to iOS 13.4, tvOS 13.4, and macOS 10.15.4
remove MAME4mac target, just build Catalyst from MAME4iOS target
break the dam some more, added more swift code.

eduo commented 2 years ago

Will MAME for Mac be handled solely by the main MAME project from now on?

ToddLa commented 2 years ago

MAME4mac is now built from the iOS target, just build as Mac Catalyst

yoshisuga commented 2 years ago

I tried out the updated keyboard and it looks awesome! Thanks so much for the changes, they really made a huge difference in the look and feel. I'm using the keyboard in other projects like RetroArch and I'm planning on making it a dependency (maybe swift package manager).

yoshisuga commented 2 years ago

That's cool that the iOS target could be used directly to build on Mac with Catalyst - I didn't know that was possible, or maybe that was always there?

We should fix the menus though later :)

ToddLa commented 2 years ago

I tried out the updated keyboard and it looks awesome! Thanks so much for the changes, they really made a huge difference in the look and feel. I'm using the keyboard in other projects like RetroArch and I'm planning on making it a dependency (maybe swift package manager).

Sucking in random code from the internet randomly and automatically is never a good idea, so I am by default nervous about using a package manager.

ToddLa commented 2 years ago

Nice changes! I kinda glossed over the file handling stuff but I'll run the build on my iPhone to see how it looks.

The file handling changes are some pre-work for a future feature, that should not effect current version.

create software and shaders folders, they currently are ignored.
read the list of file extensions we import from our own Info.plist, that way they are in one place. (next PR this list will grow....)
add a version tag to our HASH.DAT database
add new key to HASH database to lookup a single rom file (not in a software zip) this key not used (yet) but it's presence in the database won't effect current build.

ToddLa commented 2 years ago

That's cool that the iOS target could be used directly to build on Mac with Catalyst - I didn't know that was possible, or maybe that was always there?

It was not possible until we bumped the minimum version to 13.0, I picked 13.4 cuz that is when proper keyboard support was added

It took some playing around to get Xcode-build to do the right thing, I hope I did that right.

We should fix the menus though later :)

I have seen this before, I think it is a macOS bug, but I will investigate and fix....