-
Notifications
You must be signed in to change notification settings - Fork 20
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
Universal Binary for Apple Silicon #139
Comments
Thank you for the input. It is actually a bit of work to port, and given that I don't have hardware to test on, and any donations haven't come close to even a down payment, it is doubtful. If I am going to make an effort in this area, I will probably do a complete rewrite in a new commercial app. |
@robaho Is this closed because you don't have a method to test it? |
Pretty much. It’s hard to accept a PR that I have no way of testing. |
If you can provide a list of specific features to test along with some sample files, I can create a callout for testers to create a screen recorded review of the Universal Binary. |
I volunteer to test on my Mac mini M1. |
@thebitstick That's awesome!! |
I'm sure @robaho will have more specific things to try, but if you could post a screen recording of the following that would be a good start:
If you need a screen recording app you can simply have Quicktime record the screen by pressing: Command + Shift + 5 |
There is no PR to support Apple silicon. I don't have the time to create and test one. It is more than just setting some complier flags - there is some C code and other memory manipulation operations that I assume need to be changed for direct compilation. Since Seashore is released on the App Store, I cannot submit a partially working, or partially tested App. So there would need to be a full PR with complete testing and test reports of all operations. It is a lot of work IMO. |
As I mentioned, if I invest the time to support Apple silicon it will probably be on a completely new version written from the ground up in Swift. That being said, I will accept a PR to support Apple silicon, but the bar is very high for what needs to be included - since I can't break the App Store version. |
I am re-opening the issue so a PR can be created against it. |
Fair enough! |
@robaho, I couldn't find a way to contact you other than through issues, to thank you so much for what you have done for Seashore. I have been using it since it was first released, then resorted to Gimp when seashore died; I have not had to open Gimp again since you resurrected this, and I use it daily. This level of quality for open source software on the Mac is phenomenal. Thanks again |
Thanks for the kind words @mugsuk |
fixed in 3.01 |
Seashore works great through Rosetta, but a native binary would be nice!
The text was updated successfully, but these errors were encountered: