free5lot / hid-apple-patched

Allows to swap the Fn key and left Control key and other tweaks on Macbook Pro and Apple keyboards in GNU/Linux
GNU General Public License v2.0
350 stars 61 forks source link

Update Makefile #15

Closed dmontaner closed 8 years ago

dmontaner commented 8 years ago

the $(uname -r) in the first line did not work for me because the variable comes empty.

`uname -r` 

seems to do the job.

Thanks for the patch.

free5lot commented 8 years ago

Hi, David, please tell us your GNU/Linux and versions of stuff that do not support $(uname -r) in this case. PS. I'm merging your PR, thanks.

free5lot commented 8 years ago

Well, you've not only changed the $() part, but also changed the operator. Users who set LINUX_HEADER_DIR externally may have issues with it. So more investigation is required, for now I'm reverting the changes back. Please check if really $() -> `` did the thing for you and not operator change or something else?