dkms >= 3.1.7 refuses to build intel-ipu6-dkms due to duplicated modules

[vicamo]

$ sudo dkms add .
$ sudo dkms install --force ipu6-drivers/0.0.0 -k 6.16.0-13-generic
dkms.conf: Error! Duplicate module 'drivers/media/i2c/hm2170' in 'BUILT_MODULE_NAME[8]'.
dkms.conf: Error! Duplicate module 'drivers/media/i2c/hm2172' in 'BUILT_MODULE_NAME[9]'.
dkms.conf: Error! Duplicate module 'drivers/media/i2c/hi556' in 'BUILT_MODULE_NAME[10]'.
dkms.conf: Error! Duplicate module 'hm2170' in 'DEST_MODULE_NAME[8]'.
dkms.conf: Error! Duplicate module 'hm2172' in 'DEST_MODULE_NAME[9]'.
dkms.conf: Error! Duplicate module 'hi556' in 'DEST_MODULE_NAME[10]'.

Error! Bad conf file.
File: /usr/src/ipu6-drivers-0~git202411190607.0ad49882/dkms.conf does not represent a valid dkms.conf file.

$ apt policy dkms
dkms:
  Installed: 3.2.0-1ubuntu1
  Candidate: 3.2.0-1ubuntu1
  Version table:
 *** 3.2.0-1ubuntu1 100
        100 http://tw.archive.ubuntu.com/ubuntu questing-proposed/main amd64 Packages
        100 /var/lib/dpkg/status
     3.0.11-1ubuntu15 500
        500 http://archive.ubuntu.com/ubuntu questing/main amd64 Packages

See also https://bugs.launchpad.net/ubuntu/+source/ipu6-drivers/+bug/2115145

[vicamo]

This affects only Ubuntu ipu6-drivers. Close. Sorry for the noices.

[anbe42]

Installing two modules with the same name to the same location is not a good idea. Especially it confuses dkms when backing up (and later restoring) modules already provided by the kernel (or something else) - where the dkms package is supposed to provide a newer version than what is in the kernel.

In your case with the conditionals in dkms.conf I suspect that the arrays are not contiguous. Something that has not been tested and is not supposed to be supported by dkms. The holes may get removed at some point resulting in inconsistent module numbering. Please file a new bug against dkms.
@scaronni: Arrays with holes need to be identified and outlawed.

PS: there is ongoing work in dkms to support per-module conditions that should simplify your dkms.conf

[scaronni]

Testing is welcome: dkms-project/dkms#533

To test you can just copy the dkms script over the distribution provided package. Ex:

make
sudo cp dkms /usr/sbin/dkms

[scaronni]

In your case with the conditionals in dkms.conf I suspect that the arrays are not contiguous. Something that has not been tested and is not supposed to be supported by dkms. The holes may get removed at some point resulting in inconsistent module numbering. Please file a new bug against dkms. @scaronni: Arrays with holes need to be identified and outlawed.

Sure!