search

atar-axis / xpadneo

Advanced Linux Driver for Xbox One Wireless Controller (shipped with Xbox One S)
https://atar-axis.github.io/xpadneo/
GNU General Public License v3.0
1.87k stars 110 forks source link

hid-xpadneo, Makefile: remove requirement for kernel sources #421

Closed apprehensions closed 1 year ago

apprehensions commented 1 year ago

remove the need for needing kernel sources for building and installing xpadneo. when i first found this project i was very weirded by the fact that it required the kernel sources, when many modules i know only need the header files, or modules directory.

i can add the ability to use kernel sources if required, automatically via sourching kernel's Kbuild.

i've also changed it so that VERSION isn't a requirement, and can be set at runtime make VERSION=bleh, for package builds that don't have access to a git directory.

Tekudachi commented 1 year ago

Hi, how can I sign this driver for enabling secure boot? Thanks in advance.

kakra commented 1 year ago

Hi, how can I sign this driver for enabling secure boot? Thanks in advance.

This is specific to each distribution so it was removed from the docs some while ago (maybe there are a few leftover generic instructions). Usually, your distribution DKMS should take care of it. If it does not, you should ask your distribution maintainers. If this needs an optional hook in the dkms config, I can add that. Please create a new issue for this or use one of the reports that already handle that topic. Thanks. :-)

codeclimate[bot] commented 1 year ago

Code Climate has analyzed commit 9faf4dc7 and detected 0 issues on this pull request.

View more on Code Climate.

kakra commented 1 year ago

@apprehensions Please open a separate pull request about the version variable if this is still needed.