search

sonik-br / GDMENUCardManager

Manage content on SD Card for Dreamcast's GDMenu
GNU General Public License v3.0
226 stars 18 forks source link

Version 1.3.3's Mac version has incorrect permissions #11

Closed ticky closed 3 years ago

ticky commented 3 years ago

The current Mac version does not run due to a permission error; the executables at GDMENUCardManager.app/Contents/MacOS/GDMENU and GDMENUCardManager.app/Contents/MacOS/GDMENUCardManager/GDMENUCardManager are not marked as executable, which causes the app to fail to open until manually fixed.

sonik-br commented 3 years ago

I have no means to flag it as executable. I don't even have access to a mac. It's not working as an "application bundle" ? Please see issue number #4

ticky commented 3 years ago

No, the application bundle does not work as provided; those two specific files need to be marked executable for it to work. The build provided by @Pistacchione in #4 does work because the permissions are already correct, but the one in GDMENUCardManager.v1.3.3-osx-x64-AppBundle.zip does not.

You might consider building the releases using GitHub's Actions CI, which provides free Mac environments to build on.

sonik-br commented 3 years ago

I have no idea how to do this. I will revert back to not using the bundle then. For now you can just extract it from the zip and set the correct attributes right?

ticky commented 3 years ago

Right, well looking at what Avalonia say they sadly don't have a good suggestion for this, other than doing chmod stuff on a Unix system. Do you have access to a Linux machine to do this? That should work, as long as the files are marked correctly.

ticky commented 3 years ago

It looks like there might be a way to fix this from Windows, using this person's tool

sonik-br commented 3 years ago

I've made a python script to create the linux tar version with the correct attribute. Passed the mac version through it and generated a tar.gz file. It's on the releases page. Can you test it please?

YunaVirtua commented 3 years ago

I've tested it with the tar.gz bundle, applying chmod +x to GDMENU makes it executable, but without doing that the issue persists

ticky commented 3 years ago

Your script might only be applying +x to GDMENUCardManager.app/Contents/MacOS/GDMENUCardManager/GDMENUCardManager, it'll need to apply it to GDMENUCardManager.app/Contents/MacOS/GDMENU as well.

I just want to say I appreciate you working to help with this!

sonik-br commented 3 years ago

Oh. I will need to apply the correct attribute to the GDMENU shell script too. Gonna do this in a few days. Will let you know. Also thanks for testing and reporting! It's really important to me as I don't have access to a mac.

Pistacchione commented 3 years ago

Hi to all,

as @ticky said it need to apply +x to GDMENUCardManager.app/Contents/MacOS/GDMENU.

Application start but with access denied error on every volume immagine

But if you start it via command line ./GDMENUCardManager.app/Contents/MacOS/GDMENU it works... GDMENU

We need to investigate more to find the issue.

By the way, this is a great tool and we hope to help you solve this issue and improve it further!

sonik-br commented 3 years ago

Uploaded AppBundle.tar.gz with execute attribute on the two required files. Please test again. :)

Pistacchione commented 3 years ago

Uploaded AppBundle.tar.gz with execute attribute on the two required files. Please test again. :)

Now the application start but all external volumes are not accessible immagine

sonik-br commented 3 years ago

One more time... Now using the linked "zip_exec" program. Please try again with the zip file.

Pistacchione commented 3 years ago

One more time... Now using the linked "zip_exec" program. Please try again with the zip file.

Nope, same error for me

sonik-br commented 3 years ago

Ouch. Getting out of options here. If you get the normal zip version, extract and set attribute on the executable, it works?

ticky commented 3 years ago

Now the application start but all external volumes are not accessible immagine

this is an entirely separate issue and not related to the executable flag, let's discuss that one in #12

it sounds like the tar.gz solution worked for this, but that version seems to have been deleted now, so I can't check!

ticky commented 3 years ago

The current zip version also has correct executable permissions, so I believe this particular issue is resolved.

sonik-br commented 3 years ago

@ticky tar file is back again but it should have the exactly same attributes as the zip one. If you want to test it.

Pistacchione commented 3 years ago

@ticky tar file is back again but it should have the exactly same attributes as the zip one. If you want to test it.

Maybe this doc could help to build a correct .app application https://docs.avaloniaui.net/docs/distribution-publishing/macos#notes-on-the-app-executable-file

ticky commented 3 years ago

@sonik-br okay, that version works, too 👍🏼 whichever way is easiest for you, this issue is resolved