2ndalpha / gasmask

Hosts file manager for OS X
3.73k stars 252 forks source link

updated for Apple Silicon #220

Open sweetppro opened 1 year ago

sweetppro commented 1 year ago
sweetppro commented 1 year ago

Here's all the assets I used/created: Gas Mask Icons.zip

2ndalpha commented 1 year ago

Wow, it is a big PR. Has anyone tested it / is using it successfully daily?

oliveratgithub commented 1 year ago

Has anyone tested it / is using it successfully daily?

I tested it today – but my conclusion is, that its not "production ready" yet. Some key functionalities of the Gas Mask App are not working. I think the to-dos in #202 need to be addressed, too.

I tried it first with the configured Target platform settings (11.0) and with the most recent once (13.1) – the later giving me actually better results on an Apple Silicon platform.

Functionality + GUI issues

—> below are some screenshots

Xcode issues

Build Errors

None

Build Warnings

ApplicationController.m

HostsListView.m

HostsMainController.m

LoginItem.m

PrivilegedActions.m

Util.m

Screenshots

Screenshot 2023-03-12 at 16 26 12 Screenshot 2023-03-12 at 16 00 24

App GUI

Screenshot 2023-03-12 at 16 04 59 Screenshot 2023-03-12 at 16 06 45 Screenshot 2023-03-12 at 16 11 31

Xcode

Screenshot 2023-03-12 at 16 02 40 Screenshot 2023-03-12 at 15 30 00
2ndalpha commented 1 year ago

Thank you @oliveratgithub for testing. To be fair the current code in master does not work at all.

I will try to review it all.

lockieluke commented 1 year ago

merge this!

tlogik commented 10 months ago

Hi @2ndalpha Is there something we can do to help the process of getting the Apple Silicon version out. I see some testing is required and maybe some fixes. Please advise as it seems you placed you self in the eye of the storm and wanted to review :-D

2ndalpha commented 10 months ago

@tlogik I can review and test PR-s. The changes would need to be as small as possible. This PR is huge and if the author does not have to fix the issues then there is no real way to move forward with it. Unfortunately I don't have time to actively develop it.