trulyspinach / SMCAMDProcessor

Power management, monitoring and VirtualSMC plugin for AMD processors
BSD 3-Clause "New" or "Revised" License
1.04k stars 90 forks source link

MacKernalSDK and Xcode 12.2 adaptations #112

Closed cgrazy closed 1 year ago

cgrazy commented 3 years ago

Hi,

I've created a change where I've adapted SMCAMDProcessor to MacKernelSDK and Xcode 12.2. Furthermore it's highly needed to provide macossdk 10.15 beneath /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs*)

*) at least on my MacOS Catalina 10.15.7, Xcode 12.2 that folder was missing.

--cgrazy

trulyspinach commented 3 years ago

Hi, thanks a lot for the pull request! I haven't use Xcode 12.2 yet, but from the file changes I can only see some removed header includes. Are they preventing compilation in the latest Xcode? The reason to use those older header files is to support linking in macOS 10.13.

cgrazy commented 3 years ago

Hi, no worries. They somehow are not available in LiLu.kext any longer, and moved to MaKernelSDK in a renamed version. So it was not compiling with the newest versions of lilu. I just saw that in "amdryzencpumuserclient.hpp" the adaptions are started, but commented out. So further changes are needed in that way as well.

Xcode 12.2 is using sdk 11, so there's still something to do to resolve that.

But I can only test on 10.15 so far.