fqlx / XboxKeyboardMouse

Keyboard and mouse for Xbox One streaming on Windows
Apache License 2.0
317 stars 68 forks source link

Improved SlimDX Handling #204

Closed DavidRieman closed 3 years ago

DavidRieman commented 3 years ago

While going through my repositories to adopt the industry's master->main branch movement, I checked in on the status of this repo on a machine that I think hasn't ever run XboxKeyboardMouse before. The app wasn't launching successfully, and found it to be the SlimDX library not loading correctly. I found that although SlimDX is deprecated (no longer supported), that the latest version of it was released on NuGet as a package. Switching from a direct DLL reference to a NuGet reference fixed the DLL loading issue, and the app works great again. So this feels like a cleaner, more trustworthy way to reference SlimDX DLL.

On top of that, this feels like it may even be eliminating the need to run the "SlimDX Runtime" Installer? So perhaps this PR could be followed by testing running on another definitely-fresh machine to see if the whole "Prerequisites" part of the README can be deleted!