Skip to content
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

Should not bundle CAN drivers #72

Open
ndantam opened this issue Nov 5, 2013 · 2 comments
Open

Should not bundle CAN drivers #72

ndantam opened this issue Nov 5, 2013 · 2 comments

Comments

@ndantam
Copy link
Member

ndantam commented Nov 5, 2013

These do not belong in this repo. Links to vendor websites / out-of-tree drivers in the README or INSTALL file would be better.

@thedancomplex
Copy link
Member

I include them because I do not trust venders to keep the versions that we
need available. I can post them on a website we control but it is purely
because it is tested with 7.7 and 7.8 and I want to make sure things keep
on working.

-Dan

On Mon, Nov 4, 2013 at 11:39 PM, Neil T. Dantam [email protected]:

These do not belong in this repo. Links to vendor websites / out-of-tree
drivers in the README or INSTALL file would be better.


Reply to this email directly or view it on GitHubhttps://github.com//issues/72
.

Daniel M. Lofaro Ph.D
Assistant Professor, George Mason University
Electrical and Computer Engineering Department
DARPA Robot Challenge Track-A Team DRC-Hubo Researcher
e: [email protected]
w: http://danLofaro.com http://danlofaro.com/

@ndantam
Copy link
Member Author

ndantam commented Nov 5, 2013

hubo-ach really just needs socketcan from the kernel (which should be supported forever, based on Linux kernel policy). That fact that some people use the PCAN driver for socketcan is almost incidental.

I think putting driver tarballs on a separate website is both reasonable and better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants