Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bundled Growl and Sparkle frameworks don't have arm64 part #372

Open
ryandesign opened this issue Aug 22, 2022 · 4 comments
Open

Bundled Growl and Sparkle frameworks don't have arm64 part #372

ryandesign opened this issue Aug 22, 2022 · 4 comments

Comments

@ryandesign
Copy link

The bundled Growl framework is an x86_64/i386 universal binary and the bundled Sparkle framework is for x86_64 only. This prevents the software from building for arm64.

This was reported to MacPorts here: https://trac.macports.org/ticket/65690

@pmetzger
Copy link

pmetzger commented Sep 6, 2022

Growl is gone long enough that Hermes should probably not depend on it any more anyway.

@macOS-Mavericks
Copy link

So Sparkle supports Apple silicon - sparkle-project/Sparkle#2598 .

Growl is long gone.

Wasn't there so other component that needed to be updated to ARM64 too?

@dtseto
Copy link

dtseto commented Oct 31, 2024

So Sparkle supports Apple silicon - sparkle-project/Sparkle#2598 .

Growl is long gone.

Wasn't there so other component that needed to be updated to ARM64 too?
https://trac.macports.org/ticket/65690#no1
This person reported no. Ideally it should be updated to new macos notifications instead of growl but I have got it to work without growl. Right now testing a version for macos 11 and 12 and then updating latest sparkle. I don't have an arm mac to test though.

Here is my version for arm and x86 to test https://github.com/dtseto/Hermes-master/releases/tag/prereleasev132

@macOS-Mavericks
Copy link

macOS-Mavericks commented Oct 31, 2024

Here is my version for arm and x86 to test https://github.com/dtseto/Hermes-master/releases/tag/prereleasev132

I just downloaded your 1.3.2d1 version, but after unpacking it, it indicated it was for Intel after doing a Get Info, versus a Universal build. I haven't tested it since I have nothing on my M1 system that uses Rosetta 2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants