-
Notifications
You must be signed in to change notification settings - Fork 72
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
Asus: Remove System.Management #335
Asus: Remove System.Management #335
Conversation
what is the issue with the dependency? |
Error was like "Interop.AuraServiceLib.dll not found" or something. Research on internet said it was version conflict. I used various tools to figure out the conflict but none of them gave any clue. Difference between Aurora and Artemis was .net version and System.* dependencies. Removing System.Management from this library fixed my problem |
Also, these changes made sense |
This doesn't make any sense - if the error is "Interop.AuraServiceLib.dll not found" then it normally is exactly this - sometimes aura does not correctly register it in the GAC, that's why artemis is distributing it. |
I made sure to put the dll everywhere it can be found... And this change fixes that |
yeah, but this change is also breaking device detection for some devices and i really can't see any relation between that dependency and the capability of finding a library. |
from looking at it i'd say you have your version conflict in aurora as you're explicitely referencing 6.0.0 of System.Management, the provider uses 7.0.0. |
Well I'm out of arguments :) |
…to asus-remove-system-management
…to asus-remove-system-management
…to asus-remove-system-management
Thank you for your fix, it fixed the issue with the asus aura devices for me too, I was having the same issue as you had :) |
…sus-remove-system-management
I've had dependency problem with Aurora and my only solution was this. I know this is project specific change but I think this dependency is unnecessary anyway.
It should be removed so that it won't be problem for other projects in the future.