zhen-zen / YogaSMC

ACPI driver for OEM hardware.
GNU General Public License v2.0
237 stars 25 forks source link

YogaSMCNC - SMC Pane #189

Open pierpaolodimarzo opened 1 year ago

pierpaolodimarzo commented 1 year ago

Hi Zen, I'm having a problem with the latest version of the YogaSMCNC app. After installing *.prefPane and the YogaSMCNC application, clicking on the application at the top of the status bar, and clicking on the "Preferences" item, the "Please install YogaSMCPane" error popup appears, while opening it directly from the "Preferences System" panel opens without problems.

Screenshot 2023-01-08 alle 12 53 48 Screenshot 2023-01-08 alle 12 54 09

I am attaching screenshots of the problem.

macOS Ventura 13.1 (22C65)

CobanRamo commented 1 year ago

I have the same problem with Ventura, the system prefences can no longer be called up from the status bar.

statusbar

Einstonoy commented 1 year ago

It is interesting that my x1y3 appears opposite situation

dclive commented 1 year ago

@CobanRamo : The version in your picture is an older 152 version; you might want to get the latest 153 version. That said, with Ventura / 13.1, using the 153 version, I have the same problem with a X1C6.

CobanRamo commented 1 year ago

I don't think it's due to the version, it doesn't work under ventura. Under Catalina and BigSur on the same computer with the same config is no problem. YogaSMCPane1 YogaSMCPane2

https://user-images.githubusercontent.com/68192365/213952648-6abb91fc-2ccd-494d-8460-d4b06c86651e.mp4

bootlegseahorse commented 1 year ago

The System Preference app was renamed to System Settings in Ventura, and I think that is exactly what caused the problem.

bibarub commented 1 year ago

The System Preference app was renamed to System Settings in Ventura, and I think that is exactly what caused the problem.

nope, the bundle id is still the same

one thing that might actually be related is broken permission management because of disabled AMFI. OCLP graphics patches need AMFI to be disabled on Ventura

5T33Z0 commented 1 year ago

Same for me. The behavior is the same on Sonoma beta 2, btw.

jozews321 commented 7 months ago

I made a PR to hopefully fix the launching issue. There is also an issue with the latest release prefpane in macOS 13+, for some reason it doesn't work after opening once and then going to another settings page, its resolved when compiling it again targeting 10.13 minimum.

Here are some binaries with the fix

Builds.zip

CobanRamo commented 7 months ago

We're riding a dead horse but I'll test it out tonight after work. Thank you for your effort.

It's been years since @Zhen-Zen became active... :)

picture

5T33Z0 commented 7 months ago

I made a PR to hopefully fix the launching issue. There is also an issue with the latest release prefpane in macOS 13+, for some reason it doesn't work after opening once and then going to another settings page, its resolved when compiling it again targeting 10.13 minimum.

Here are some binaries with the fix

Builds.zip

Thanks for sharing. Will report back after trying it out.

CobanRamo commented 7 months ago

I made a PR to hopefully fix the launching issue. Builds.zip

I tested it, thanks for your effort @jozews321 With this binary the call from “Prefences” works again...

Ergo, the problem of calling the “preferences” would be solved. 211217181-b17739ca-2293-4ca4-9082-f6797895312f

But now the “Start at Login” no longer works. You always have to start it manually after a restart. Bildschirmfoto 2024-02-19 um 18 23 22

Otherwise it looks good.

5T33Z0 commented 7 months ago

Hmm, ok. For me, the "Start at Login" feature is of higher priority than the prefpane opening at first click.

jozews321 commented 7 months ago

I made a PR to hopefully fix the launching issue. Builds.zip

I tested it, thanks for your effort @jozews321 With this binary the call from “Prefences” works again...

Ergo, the problem of calling the “preferences” would be solved. 211217181-b17739ca-2293-4ca4-9082-f6797895312f

But now the “Start at Login” no longer works. You always have to start it manually after a restart. Bildschirmfoto 2024-02-19 um 18 23 22

Otherwise it looks good.

You need to disable it first in the release one, and then enable it in the new one for it to work, alternatively you can add the app to login items in settings to set it to open automatically

5T33Z0 commented 7 months ago

Working fine after replacing both the prefpane and the app.

@jozews321: I would say, just fork the repo and upload your own release, if the PR won't be merged.