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

Add support for .NET 8 build of Fan Control #20

Open
ChaseDDevelopment opened this issue Feb 26, 2023 · 15 comments
Open

Add support for .NET 8 build of Fan Control #20

ChaseDDevelopment opened this issue Feb 26, 2023 · 15 comments
Assignees
Labels
enhancement New feature or request in-prerelease Available or fixed in a pre-release
Milestone

Comments

@ChaseDDevelopment
Copy link

ChaseDDevelopment commented Feb 26, 2023

Hey there,

I am having the same behaviour as in issue (#2), though I believe it's a different exception:

I've attached the logs.
I've tried the latest Release (v 1.0.0) as well as the latest beta release.

I have ensured both zip's and dll's are unblocked as per the instructions. Please let me know if I can help in any way.

Corsair Commander PRO could not initialize or has no sensors.
2/26/2023 12:10:12 PM: [CorsairLink] DeviceManager: Found 2 Corsair device(s)
  name=CORSAIR iCUE Commander CORE, devicePath=\\?\hid#vid_1b1c&pid_0c1c&mi_01#7&26ae1e37&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}
  name=CORSAIR iCUE Commander CORE, devicePath=\\?\hid#vid_1b1c&pid_0c1c&mi_00#7&2d6e075&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}

2/26/2023 12:10:12 PM: [CorsairLink] DeviceManager: Found 1 supported Corsair device(s)
  name=CORSAIR iCUE Commander CORE, devicePath=\\?\hid#vid_1b1c&pid_0c1c&mi_00#7&2d6e075&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}

2/26/2023 12:10:12 PM: [CorsairLink] An exception occurred attempting to initialize device 'CORSAIR iCUE Commander CORE (e20f10a0948871aa8559c2063091005f)' (\\?\hid#vid_1b1c&pid_0c1c&mi_00#7&2d6e075&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}):
2/26/2023 12:10:12 PM: [CorsairLink] System.TypeInitializationException: The type initializer for 'CorsairLink.Synchronization.CorsairDevicesGuard' threw an exception.
 ---> System.MissingMethodException: Method not found: 'Void System.Threading.Mutex.SetAccessControl(System.Security.AccessControl.MutexSecurity)'.
   at CorsairLink.Synchronization.CorsairDevicesGuard.CreateMutex()
   at CorsairLink.Synchronization.CorsairDevicesGuard..cctor()
   --- End of inner exception stack trace ---
   at CorsairLink.Synchronization.CorsairDevicesGuard.Acquire()
   at CorsairLink.Synchronization.CorsairDevicesGuardLock..ctor()
   at CorsairLink.Synchronization.CorsairDevicesGuardManager.AwaitExclusiveAccess()
   at CorsairLink.CommanderCoreDevice.RefreshImpl(Boolean initialize)
   at CorsairLink.CommanderCoreDevice.Connect()
   at FanControl.CorsairLink.CorsairLinkPlugin.FanControl.Plugins.IPlugin.Initialize()
2/26/2023 12:10:14 PM: [CorsairLink] DeviceManager: Found 2 Corsair device(s)
  name=CORSAIR iCUE Commander CORE, devicePath=\\?\hid#vid_1b1c&pid_0c1c&mi_01#7&26ae1e37&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}
  name=CORSAIR iCUE Commander CORE, devicePath=\\?\hid#vid_1b1c&pid_0c1c&mi_00#7&2d6e075&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}```
@ChaseDDevelopment ChaseDDevelopment changed the title Corsair Commander CORE Not Detected Corsair Commander CORE Not Detected New Exception Feb 26, 2023
@EvanMulawski EvanMulawski self-assigned this Feb 26, 2023
@EvanMulawski EvanMulawski added the triage Issue has not yet been fully identified label Feb 26, 2023
@EvanMulawski
Copy link
Owner

Hi @ChaseDDevelopment, are you using the .NET Framework 4.8 build of Fan Control or the .NET 7 build?

@ChaseDDevelopment
Copy link
Author

Hey @EvanMulawski I am using the .NET 7 build with runtime 7.0.3

@EvanMulawski
Copy link
Owner

There isn't a .NET 7-compatible build yet, so the .NET Framework build of Fan Control is required (and somewhat recommended; most of the available plugins only target .NET Framework anyway). I'll add this to the README.

@EvanMulawski EvanMulawski changed the title Corsair Commander CORE Not Detected New Exception MissingMethodException using .NET 7 build of Fan Control Feb 27, 2023
@EvanMulawski EvanMulawski added enhancement New feature or request and removed triage Issue has not yet been fully identified labels Feb 27, 2023
@Rem0o
Copy link

Rem0o commented Feb 28, 2023

@EvanMulawski you could create a multi-target (net48;net7) and use the FanControl.Plugin.dll from the net48 and net7 release with a prefix to bundle both version if you want to do that.

@ChaseDDevelopment
Copy link
Author

@EvanMulawski Just got finished trying the Framework build v147 and it worked like a charm. Much appreciated!!

As a note, I did have to stay on the v147 build as when prompted by Fan control to update to v148, the plugin broke.

@EvanMulawski
Copy link
Owner

@ChaseDDevelopment Yeah, there's a problem with v148. See Rem0o/FanControl.Releases#1577 (comment)

@ChaseDDevelopment
Copy link
Author

@ChaseDDevelopment Yeah, there's a problem with v148. See Rem0o/FanControl.Releases#1577 (comment)

No problem! Thanks for calling that issue out. Appreciate the work you've put into this, made sure One of the PC's in the house could have it's fans controlled nicely through corsair's core! This can be closed, unless you want it open for the enhancement.

@CalculonPrime
Copy link

There isn't a .NET 7-compatible build yet, so the .NET Framework build of Fan Control is required (and somewhat recommended; most of the available plugins only target .NET Framework anyway). I'll add this to the README.

Isn't .NET 7 a .NET Framework? If .NET 7 is not a .NET Framework, that's confusing.

@Rem0o
Copy link

Rem0o commented Mar 10, 2023

Isn't .NET 7 a .NET Framework? If .NET 7 is not a .NET Framework, that's confusing.

@CalculonPrime Tell Microsoft. It is ;P

@EvanMulawski EvanMulawski changed the title MissingMethodException using .NET 7 build of Fan Control Add support for .NET 7 build of Fan Control Apr 14, 2023
@EvanMulawski EvanMulawski changed the title Add support for .NET 7 build of Fan Control Add support for .NET 8 build of Fan Control Feb 9, 2024
@EvanMulawski EvanMulawski added this to the v2.0 milestone Feb 9, 2024
@p1r473
Copy link

p1r473 commented Mar 17, 2024

Tried this with the latest version of .NET 8 of FanControl - didnt work - had to roll back to 4.8
I guess this is still not implemented yet right?

@EvanMulawski
Copy link
Owner

Tried this with the latest version of .NET 8 of FanControl - didnt work - had to roll back to 4.8 I guess this is still not implemented yet right?

Not yet - will be in v2.0 which is currently being designed.

@Rem0o
Copy link

Rem0o commented Mar 17, 2024

FanControl.Plugins.dll is netstandard2.0 by the way @EvanMulawski, so you can use the same for both.

@TannerBragg
Copy link

FanControl.Plugins.dll is netstandard2.0 by the way @EvanMulawski, so you can use the same for both.

All of my fans and pumps are run through core commanders. Without this plugin working with the .NET 8, I can't explore the software. I'd love to use this plugin, but it appears it is still incompatible with .NET 8.

I suppose it isn't being compiled against .NET Standard 2.0. I feel like this should be a relatively easy change to retarget the project? Can this be done so those of us on the latest can enjoy the plugin?

@EvanMulawski
Copy link
Owner

I am working on multi-targeting - will be after v1.6.0 comes out of beta.

@EvanMulawski EvanMulawski modified the milestones: v2.0, v1.7 May 5, 2024
@EvanMulawski
Copy link
Owner

Support added in v1.7.0-beta.1

@EvanMulawski EvanMulawski added the in-prerelease Available or fixed in a pre-release label Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request in-prerelease Available or fixed in a pre-release
Projects
None yet
Development

No branches or pull requests

6 participants