search

astuff / kvaser_interface

A Kvaser CAN interface ROS abstraction layer.
MIT License
83 stars 51 forks source link

Edit README for install instructions on kernel 5.13+ #76

Closed Roosta078 closed 2 years ago

Roosta078 commented 2 years ago

Kernel no longer includes <config/modversions.h>, which is required by the Kvaser driver. This header file has been an empty file, at least since 4.4, but was outright removed in 5.13. I have fixed this issue in the driver source code, but Kvaser does not have a repo to contribute to. This fix will allow the driver and the kvaser_interface package to be installed.

icolwell-as commented 2 years ago

Thanks @Roosta078 for mentioning this! We've also noticed this same issue with latest kernel upgrades. This repo is just a ROS driver that depends on kvaser's canlib, so I don't see a need to change anything here, especially since this error will go away with a future canlib release. That being said, I will leave this PR open for information purposes until Kvaser releases a new canlib.

We host the kvaser canlib here for convenience. It is just a straight copy of what is available from their website. We then package their canlib using this repo and make the debs available on this launchpad for convenience.

I've been in touch with kvaser support regarding this issue and they intend to release a new 5.38 version of canlib soon which will resolve this issue. I don't have an ETA yet, but we'll be updating our launchpad with the new version once it's available.

Thanks again for documenting this temporary work-around.

Roosta078 commented 2 years ago

@icolwell-as The issue was resolved by Kvaser on Feb 15 with version 5.38. The astuff ppa still has version 5.36. Could you escalate updating the ppa?

icolwell-as commented 2 years ago

@Roosta078, the 5.38 release was completed by kvaser on the 15th, but it wasn't made available to the public until sometime late last week since it had to pass their internal QA testing. I hope to update the PPA sometime this week, but only after I get a chance to test it on our Kvaser hardware. If you are pressed for time, you can download the source from kvaser's website (or here, I updated it already) and follow the included README to install from source.

icolwell-as commented 2 years ago

@Roosta078 the latest kvaser drivers are now available from our launchpad: https://launchpad.net/~astuff/+archive/ubuntu/kvaser-linux/+packages

Thanks!