-
Notifications
You must be signed in to change notification settings - Fork 164
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
Move distribution specifics into the config #328
Comments
From #329 we're currently running
|
Another set of distro specific variations - module signing. In particular:
At a glance, if we are to use |
Moving the original module/collisions to standalone location:
Might be worth tracking down the reason behind this variance and removing it, in the mid/long run. At least initially preserving the behaviour seems like a good first step. |
The above are the distro-specific variations we have so far, although others may emerge at some point. They don't have to be fixed all together/in a single PR. So if anyone is bored and like to write some bash - patches welcome o/ |
It is very unclear why. The kernel simply traverses filesystem in alphabetic order, afaik. Which means not removing the original module leads to undefined behavior. |
@HanabishiRecca the original is stored outside of the normal search path. Note that the kernel does not care about any of this - kmod does all the path/priority handling as well as loading. The ultimate goal here is not to fix everything that's off with dkms, but migrate from a bunch of "running_distribution" hacks to a data/config driven flow. |
As mentioned previously #323 (comment), ideally dkms will not know (or care) about the distribution it's run against.
Any specifics like
DEST_MODULE_LOCATION
overrides will be set in/etc/dkms/framework.conf
and the user can override those with the usual/etc/dkms/framework.conf.d
mechanism.For this task, we should:
framework.conf
, honour that in dkmsinstall-foo
variant which produces creates/uses the correct config for the given distributionNice to have:
install-foo
files, update as necessarymake install
and manually removes kconf files and libdirThe text was updated successfully, but these errors were encountered: