Skip to content
This repository was archived by the owner on Aug 5, 2022. It is now read-only.

README: Supported platforms: add kernel version and Debian details #35

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dpocock
Copy link

@dpocock dpocock commented Dec 8, 2017

No description provided.


Linux kernel v4.13.0 or later is required.

- Debian 9.0 (stretch) - [package status / QA page](https://packages.qa.debian.org/thunderbolt-tools) - [bug reports](https://bugs.debian.org/src:thunderbolt-tools)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First, thanks for adding it!

Until now, the 'Supported OSes' used to tell which distro we know this to be compiled and installed correctly on (and preferably, testing it on).
Maybe there should be a spearated section for listing links to the relevant pages on distros that include it (like you did here for Debian)?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To become an official package in Ubuntu, it has to go through Debian anyway, even Ubuntu's official documentation says that. So when people get it from Ubuntu in future, most of them won't be downloading from Github, they'll be installing the package that has gone through Debian into Ubuntu.

When something is in Debian, any unit tests you include are automatically run each time it is (re)compiled and you can see the build logs, including unit test output, on the build farm

Have you had any direct communication with Colin and Kamal, the maintainers of the Debian package or any other Debian developers?

@superm1
Copy link

superm1 commented Dec 9, 2017

I would propose that end up on a github wiki page and in the readme link to said wiki page. That way there doesn't have to be a PR every time a new distro adds it.

@@ -30,6 +30,10 @@ handling the ACL and more.


## Supported OSes

Linux kernel v4.13.0 or later is required.
Copy link
Contributor

@ybernat ybernat Dec 9, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just realized that this sentence is too restrictive.
This is true for the upstream ('vanilla') kernel, but distros may backport the driver to previous kernel versions.
Maybe something like The required Linux kernel driver was upstreamed in v4.13? Makes sense? Too much technical?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe it could say "Linux kernel v4.13.0 includes the necessary hardware driver. If using an older kernel then the standalone Thunderbolt driver version X.Y.Z or later needs to be installed as a module."

That should probably include a link to the driver too. Does it seem accurate?

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

Successfully merging this pull request may close these issues.

3 participants