search

blake502 / balatro-mobile-maker

Create a mobile Balatro app from your Steam version of Balatro
270 stars 21 forks source link

0.8.3 permissions on iOS #67

Open eszoke opened 1 week ago

eszoke commented 1 week ago

Using the new BMM build to make an IPA, I'm getting a permissions error when I go to install it in Altstore. Going back to 0.8.2 to make an IPA with the same .exe and installs fine. Anyone else seeing similar on 0.8.3?

blake502 commented 1 week ago

Hmm the Python dependency was removed in 0.8.3, based on this PR https://github.com/blake502/balatro-mobile-maker/pull/61#issuecomment-2094901151 I never tested it myself since I don't have iOS. If others are experiencing the same issue, then I'll roll back that change.

kylefmohr commented 1 week ago

@eszoke can you share the error message? What OS are you building it on?

kylefmohr commented 1 week ago

IMG_F30138425D92-1

Just tested and I'm unable to recreate, installation via AltStore worked just fine for me.

eszoke commented 1 week ago

Cool, yeah did a little bit of scenario testing with it but will try some more when I get home a little later. OS was 17.4.1 I believe

eszoke commented 1 week ago

Ok tested again with 0.8.3 and the default exe from the current 1.0.1f Balatro build, as well as with an older build 1.0.1c, and both are giving me this error: IMG_0021.jpg

IMG_0022.jpg

And then tested 0.8.2 with the current 1.0.1f exe again and was able to side load without error.

OS is iPad OS 17.4.1 on my 6th gen iPad. The BMM parameters were 60FPS cap, yes to widescreen/high DPI patches and no to the CRT shader patch. Altstore version is 1.7

Super weird yeah, I'll keep playing around with it too might just be something weird with Altstore.

kylefmohr commented 1 week ago

That's interesting! I'm not familiar with that error message, but I have a few ideas I'm going to look into this evening. I think the issue stems from the limitations Apple puts on free Developer accounts, which, as you might know, limits you to 3 sideloaded apps on your device at the same time. However including AltStore and the old version of Balatro, this should still leave you with a free slot, so it wouldn't completely explain this error.

Have you tried uninstalling the old Balatro before installing the new one via AltStore?

If that still doesn't work, it may work in a few days, when the week-long validity of the apps you signed expires.

I still want to look into how BMM's IPA base file assigns an App ID.

blake502 commented 1 week ago

Thanks for looking into this @kylefmohr ! I'll assign the issue to you if you're okay with that.

wygoda commented 1 week ago

I'm facing the same issue, building on Windows using BMM 0.8.3, targeting iPhone with iOS 17.4.1 via AltStore 1.7. Works fine with BMM 0.8.2. Attaching a screenshot with more details about the error. Hope it helps! IMG_5359 - Copy

kylefmohr commented 1 week ago

I'm facing the same issue, building on Windows using BMM 0.8.3, targeting iPhone with iOS 17.4.1 via AltStore 1.7. Works fine with BMM 0.8.2. Attaching a screenshot with more details about the error. Hope it helps! IMG_5359 - Copy

Thanks for posting the error, I think I see what the issue is, I'll post a new build in a little bit once I figure out a good way to implement this.

kylefmohr commented 1 week ago

@eszoke did you also initially run BMM on Windows to build the .ipa file? We might be running into an incompatibility that I'm not sure can be fixed:

tl;dr game.love doesn't have the 'execute' file attribute, and there isn't a good way to do this that I know of on Windows.

I started working on a potential fix for the issue, however .NET's "File.SetUnixFileMode" doesn't support Windows either, so we might be out of luck. It may be the case that either Linux or OSX is required to create the .ipa file. Not 100% sure on this, but this is what I'm thinking so far.

eszoke commented 1 week ago

Yup I did on Windows, that makes sense though. I have a dual boot of Linux so I'll try creating the IPA file there tomorrow, it would be a perfectly solid solution for me! Cheers