search

rockowitz / ddcutil

Control monitor settings using DDC/CI and USB
http://www.ddcutil.com
GNU General Public License v2.0
946 stars 38 forks source link

Enhancement - consider packaging module loading #426

Open digitaltrails opened 3 months ago

digitaltrails commented 3 months ago

When I packaged ddcutil-service for OpenSUSE, I was advised to add module loading to the spec so that users would not have to deal with that themselves.

I added the following to the ddcutil-service.spec:

# Module loading on subsequent boots
install -m 0644 -D -t %{buildroot}%{_prefix}/lib/modules-load.d/ i2c-dev/%{name}.conf

# Module load for the current boot:
# The post modprobe is helpful for the user, but not really necessary, don't error if it fails
%post
%{_prefix}/sbin/modprobe i2c-dev ||:

You might like to consider whether ddcutil or ddcui should do something similar. Or whether this might be bundled in with the i2c rules package.

rockowitz commented 3 months ago

make install has done this since ddcutil release 2.0.0 and ddcui 0.5.4 See the [release notes](http://www.ddcutil.com/release_notes/#installation and ddcui release notes. Whether the distribution packages (rpm, deb, etc.) install the file depends on the distribution and its procedures. I supply the official debian packages which does this (or more precisely will do so once the packages are updated to the current version.

digitaltrails commented 3 months ago

This issue can therefore be closed?

In my case, I should raise this as an issue with the OpenSUSE package/maintainer on OpenSUSE build system.