Debian Bug report logs - #688904
DKMS not handling kernel versions with major > 9 ; breaking on kfreebsd-10

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Christoph Egger <christoph@debian.org>

To: Debian Bug Tracking System <submit@bugs.debian.org>

Subject: dkms: remove non-functional, breaks dh_dkms maintscript

Date: Wed, 26 Sep 2012 12:07:54 -0700

Package: dkms
Version: 2.2.0.3-1.1
Severity: serious
X-Debbugs-Cc: debian-bsd@lists.debian.org

Hi!

  I just noticed updating fuse4bsd-dkms fails on my kfreebsd-amd64. It
seems the `dkms remove` command used in the prerm script is a no-op at
least on kfreebsd

=====
 % sudo dkms status
cuse4bsd, SVN: added
fuse4bsd, 0.3.9~pre1.20080208: added

 % sudo dkms remove cuse4bsd/SVN --all

 % sudo dkms status
cuse4bsd, SVN: added
fuse4bsd, 0.3.9~pre1.20080208: added
=====

=====
 % sudo dpkg -i fuse4bsd-dkms_0.3.9\~pre1.20080208-5_all.deb 
(Reading database ... 398320 files and directories currently installed.)
Preparing to replace fuse4bsd-dkms 0.3.9~pre1.20080208-5 (using fuse4bsd-dkms_0.3.9~pre1.20080208-5_all.deb) ...
Unpacking replacement fuse4bsd-dkms ...
Setting up fuse4bsd-dkms (0.3.9~pre1.20080208-5) ...
Removing old fuse4bsd-0.3.9~pre1.20080208 DKMS files...
Loading new fuse4bsd-0.3.9~pre1.20080208 DKMS files...
Error! DKMS tree already contains: fuse4bsd-0.3.9~pre1.20080208
You cannot add the same module/version combo more than once.
dpkg: error processing fuse4bsd-dkms (--install):
 subprocess installed post-installation script returned error exit status 3
Errors were encountered while processing:
 fuse4bsd-dkms
=====

  I don't have a linux here to check and it somewhat sounds similar to
#659672 though I'm not sure it is the same

Regards

    Christoph

-- System Information:
Debian Release: wheezy/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'unstable'), (1, 'experimental')
Architecture: kfreebsd-amd64 (x86_64)

Kernel: kFreeBSD 10.0-0-amd64
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages dkms depends on:
ii  build-essential               11.5
ii  coreutils                     8.13-3.2
ii  dpkg-dev                      1.16.8
ii  gcc                           4:4.7.1-1
ii  kldutils [module-init-tools]  9.0+ds1-7
ii  make                          3.81-8.2
ii  patch                         2.6.1-3

Versions of packages dkms recommends:
ii  fakeroot                                                          1.18.4-2
pn  linux-headers-686-pae | linux-headers-amd64 | linux-headers-gene  <none>
pn  linux-image                                                       <none>
ii  sudo                                                              1.8.5p2-1

dkms suggests no packages.

-- no debconf information

-- 

Message #10 received at 688904@bugs.debian.org (full text, mbox, reply):

From: Christoph Egger <christoph@debian.org>

To: 688904@bugs.debian.org

Subject: dkms remove no-op

Date: Wed, 26 Sep 2012 12:40:02 -0700

Hi!

Just for the record: 
the patch in #659672 does *not* work

Regards

    Christoph

Message #15 received at 688904@bugs.debian.org (full text, mbox, reply):

From: Christoph Egger <christoph@debian.org>

To: 688904@bugs.debian.org

Subject: dkms remove no-op bash -x trace

Date: Wed, 26 Sep 2012 12:45:21 -0700

+ readonly 'dkms_conf_variables=CLEAN REMAKE_INITRD remake_initrd PACKAGE_NAME
       PACKAGE_VERSION POST_ADD POST_INSTALL POST_REMOVE PRE_BUILD
       PRE_INSTALL BUILD_EXCLUSIVE_KERNEL BUILD_EXCLUSIVE_ARCH
       build_exclude OBSOLETE_BY MAKE MAKE_MATCH MODULES_CONF
       modules_conf_array PATCH PATCH_MATCH patch_array BUILT_MODULE_NAME
       built_module_name BUILT_MODULE_LOCATION built_module_location
       DEST_MODULE_NAME dest_module_name MODULES_CONF_OBSOLETES
       DEST_MODULE_LOCATION dest_module_location
       modules_conf_obsoletes MODULES_CONF_ALIAS_TYPE
       modules_conf_alias_type STRIP strip MODULES_CONF_OBSOLETE_ONLY
       modules_conf_obsolete_only AUTOINSTALL'
+ dkms_conf_variables='CLEAN REMAKE_INITRD remake_initrd PACKAGE_NAME
       PACKAGE_VERSION POST_ADD POST_INSTALL POST_REMOVE PRE_BUILD
       PRE_INSTALL BUILD_EXCLUSIVE_KERNEL BUILD_EXCLUSIVE_ARCH
       build_exclude OBSOLETE_BY MAKE MAKE_MATCH MODULES_CONF
       modules_conf_array PATCH PATCH_MATCH patch_array BUILT_MODULE_NAME
       built_module_name BUILT_MODULE_LOCATION built_module_location
       DEST_MODULE_NAME dest_module_name MODULES_CONF_OBSOLETES
       DEST_MODULE_LOCATION dest_module_location
       modules_conf_obsoletes MODULES_CONF_ALIAS_TYPE
       modules_conf_alias_type STRIP strip MODULES_CONF_OBSOLETE_ONLY
       modules_conf_obsolete_only AUTOINSTALL'
+ readonly 'y_re=^(Y|y)'
+ y_re='^(Y|y)'
+ readonly 'mv_re=^([^/]*)/(.*)$'
+ mv_re='^([^/]*)/(.*)$'
+ readonly 'rh_kernels=(debug|summit|smp|enterprise|bigmem|hugemem|BOOT|vmnix)'
+ rh_kernels='(debug|summit|smp|enterprise|bigmem|hugemem|BOOT|vmnix)'
+ PATH=/bin:/sbin:/usr/bin:/usr/sbin:/usr/lib/dkms
+ umask 022
+ unset CC CXX CFLAGS CXXFLAGS LDFLAGS
++ uname -r
+ current_kernel=10.0-0-amd64
++ uname -s
+ current_os=GNU/kFreeBSD
+ dkms_tree=/var/lib/dkms
+ source_tree=/usr/src
+ install_tree=/lib/modules
+ tmp_location=/tmp
+ verbose=
+ dkms_frameworkconf=/etc/dkms/framework.conf
+ [[ ! -n '' ]]
+ [[ -e /etc/sysconfig/module-init-tools ]]
+ addon_modules_dir=
+ [[ ! -n '' ]]
++ distro_version
++ local LSB_DESCRIPTION DISTRIB_ID DISTRIB_RELEASE ver
++ '[' -r /etc/lsb-release ']'
++ type lsb_release
+++ lsb_release -i -s
++ DISTRIB_ID=Debian
+++ lsb_release -r -s
++ DISTRIB_RELEASE=testing
++ case ${DISTRIB_ID} in
++ [[ -n Debian ]]
++ [[ -n testing ]]
++ echo Debiantesting
+ running_distribution=Debiantesting
+ weak_modules=
+ '[' -e /etc/dkms/framework.conf ']'
+ . /etc/dkms/framework.conf
+ module=
+ module_version=
+ template_kernel=
+ distro=
+ media=
+ release=
+ conf=
+ kernel_config=
+ archive_location=
+ kernel_source_dir=
+ ksourcedir_fromcli=
+ action=
+ force=
+ no_prepare_kernel=
+ no_clean_kernel=
+ binaries_only=
+ source_only=
+ all=
+ module_suffix=
+ rpm_safe_upgrade=
+ size=1440
+ specfile=
+ legacy_postinst=1
+ directive_array=()
+ kernelver=()
+ arch=()
+ declare -a directive_array kernelver arch
+ weak_modules=
+ last_mvka=
+ last_mvka_conf=
+ try_source_tree=
+ die_is_fatal=yes
+ '[' -x /sbin/weak-modules ']'
+ '[' -x /usr/lib/module-init-tools/weak-modules ']'
+ action_re='^(remove|(auto|un)?install|match|mk(driverdisk|tarball|rpm|deb|dsc|kmp)|build|add|status|ldtarball)$'
+ (( 3 > 0 ))
+ case $1 in
+ [[ remove =~ ^(remove|(auto|un)?install|match|mk(driverdisk|tarball|rpm|deb|dsc|kmp)|build|add|status|ldtarball)$ ]]
+ action=' remove'
+ shift
+ (( 2 > 0 ))
+ case $1 in
+ [[ fuse4bsd/0.3.9~pre1.20080208 =~ ^(remove|(auto|un)?install|match|mk(driverdisk|tarball|rpm|deb|dsc|kmp)|build|add|status|ldtarball)$ ]]
+ [[ -f fuse4bsd/0.3.9~pre1.20080208 ]]
+ [[ -d fuse4bsd/0.3.9~pre1.20080208 ]]
+ [[ -f fuse4bsd/0.3.9~pre1.20080208 ]]
+ [[ ! -n '' ]]
+ parse_moduleversion fuse4bsd/0.3.9~pre1.20080208
+ [[ fuse4bsd/0.3.9~pre1.20080208 =~ ^([^/]*)/(.*)$ ]]
+ module=fuse4bsd
+ module_version=0.3.9~pre1.20080208
+ shift
+ (( 1 > 0 ))
+ case $1 in
+ all=true
+ shift
+ (( 0 > 0 ))
+ [[ -n '' ]]
+ ((  0 != 0 &&     0 > 1  ))
+ [[ -n '' ]]
+ [[ -n '' ]]
+ for action_to_run in '$action'
+ setup_kernels_arches remove
+ [[ -n true ]]
+ [[ remove != status ]]
+ local i=0
+ read line
++ module_status_built fuse4bsd 0.3.9~pre1.20080208
++ local ret=1 directory ka k a state 'oifs= 	
' IFS=
++ for directory in '"$dkms_tree/$1/$2/"${3:-[0-9].*}/${4:-*}'
++ IFS=' 	
'
++ ka='[0-9].*/*'
++ k='[0-9].*'
++ a='*'
++ is_module_built fuse4bsd 0.3.9~pre1.20080208 '[0-9].*' '*'
++ [[ -n fuse4bsd ]]
++ [[ -n 0.3.9~pre1.20080208 ]]
++ [[ -n [0-9].* ]]
++ [[ -n * ]]
++ local 'd=/var/lib/dkms/fuse4bsd/0.3.9~pre1.20080208/[0-9].*/*' m=
++ [[ -d /var/lib/dkms/fuse4bsd/0.3.9~pre1.20080208/[0-9].*/*/module ]]
++ return 1
++ continue
++ IFS=' 	
'
++ return 1
+ [[ remove != status ]]
+ [[ ! -n '' ]]
+ [[ ! -n true ]]
+ [[ ! -n '' ]]
++ rpm -qf /lib/modules/
++ grep -v 'not owned by any package'
++ grep kernel
++ head -n 1
+ kernelver_rpm=
++ rpm -q --queryformat '%{ARCH}' ''
+ arch[0]=
++ uname -m
+ arch[0]=x86_64
+ [[ x86_64 = x86_64 ]]
+ grep -q Intel /proc/cpuinfo
+ ls /lib/modules//build/configs
+ grep -q ia32e
+ (( 1 == 1 && 0 > 1 ))
+ multi_arch=
+ local i=0
+ (( i=0 ))
+ (( 0 < 1 ))
+ [[ x86_64 != x86_64 ]]
+ (( i++ ))
+ (( 1 < 1 ))
+ case "$action_to_run" in
+ check_root
+ (( UID == 0 ))
+ return
+ remove_module
+ [[ -n fuse4bsd ]]
+ [[ -n 0.3.9~pre1.20080208 ]]
+ [[ -n '' ]]
+ [[ -d /var/lib/dkms/fuse4bsd/0.3.9~pre1.20080208 ]]
+ local i
+ (( i=0 ))
+ (( i < 0 ))
+ find /var/lib/dkms/fuse4bsd/0.3.9~pre1.20080208/10.0-0-amd64 /var/lib/dkms/fuse4bsd/0.3.9~pre1.20080208/build /var/lib/dkms/fuse4bsd/0.3.9~pre1.20080208/source -maxdepth 0 -type d
+ egrep -qv '(build|tarball|driver_disk|rpm|deb|source)$'
++ ls /var/lib/dkms/fuse4bsd
++ wc -w
++ awk '{print $1}'
+ (( 2 == 0 ))

Message #20 received at 688904@bugs.debian.org (full text, mbox, reply):

From: Christoph Egger <christoph@debian.org>

To: 688904@bugs.debian.org

Subject: dkms remove

Date: Mon, 01 Oct 2012 15:23:41 -0700

Hi!

I'm not really a bash developer but it seems the regexes in
module_status_built are not expanded properly:

> ++ module_status_built fuse4bsd 0.3.9~pre1.20080208
> ++ local ret=1 directory ka k a state 'oifs= 	
> ' IFS=
> ++ for directory in '"$dkms_tree/$1/$2/"${3:-[0-9].*}/${4:-*}'
> ++ IFS=' 	
> '
> ++ ka='[0-9].*/*'
> ++ k='[0-9].*'
> ++ a='*'
> ++ is_module_built fuse4bsd 0.3.9~pre1.20080208 '[0-9].*' '*'
> ++ [[ -n fuse4bsd ]]
> ++ [[ -n 0.3.9~pre1.20080208 ]]
> ++ [[ -n [0-9].* ]]
> ++ [[ -n * ]]
> ++ local 'd=/var/lib/dkms/fuse4bsd/0.3.9~pre1.20080208/[0-9].*/*' m=
> ++ [[ -d /var/lib/dkms/fuse4bsd/0.3.9~pre1.20080208/[0-9].*/*/module ]]
> ++ return 1
> ++ continue
> ++ IFS=' 	
> '
> ++ return 1

Note: there is a directory named 10.0-0-amd64 there

Is this some kind of advanced bash glob that only matches a single digit
before the .?

Regards

    Christoph

Message #25 received at 688904@bugs.debian.org (full text, mbox, reply):

From: Christoph Egger <christoph@debian.org>

To: 688904@bugs.debian.org

Cc: control@bugs.debian.org

Subject: dkms remove on kernels with major > 9

Date: Mon, 01 Oct 2012 15:31:22 -0700

tag 688904 +patch
kthxbye

Indeed it seems to be the problem indeed.

In line 1833 replacing

 for directory in "$dkms_tree/$1/$2/"${3:-[0-9].*}/${4:-*}; do

with

  for directory in "$dkms_tree/$1/$2/"${3:-[0-9]?.*}/${4:-*}; do

did the trick for me though I admit *this* way of fixing the glob is a
bit ugly -- needs someone with more bash clue to work out something
better but should be easy enough now ;-)

Regards

    Christoph

Message #32 received at 688904@bugs.debian.org (full text, mbox, reply):

From: Christoph Egger <christoph@debian.org>

To: 688904@bugs.debian.org

Subject: Re: dkms remove on kernels with major > 9

Date: Mon, 01 Oct 2012 15:47:32 -0700

slightly better:

--- a/dkms	2012-10-01 15:44:50.000000000 -0700
+++ b/dkms	2012-10-01 15:44:25.981366657 -0700
@@ -19,6 +19,7 @@
 #    Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 #
 
+shopt -s extglob
 
 # All of the variables we will accept from dkms.conf.
 # Does not include directives
@@ -1827,7 +1831,7 @@
 # we do not get word splitting where it would be inconvienent.
 module_status_built() {
     local ret=1 directory ka k a state oifs="$IFS" IFS=''
-    for directory in "$dkms_tree/$1/$2/"${3:-[0-9].*}/${4:-*}; do
+    for directory in "$dkms_tree/$1/$2/"${3:-+([0-9]).*}/${4:-*}; do
 	IFS="$oifs"
 	ka="${directory#$dkms_tree/$1/$2/}"
 	k="${ka%/*}"

Message #39 received at 688904@bugs.debian.org (full text, mbox, reply):

From: Christoph Egger <christoph@debian.org>

To: 688904@bugs.debian.org

Subject: dkms: diff for NMU version 2.2.0.3-1.2

Date: Fri, 5 Oct 2012 20:57:36 -0700

[Message part 1 (text/plain, inline)]

tags 688904 + pending
thanks

Dear maintainer,

I've prepared an NMU for dkms (versioned as 2.2.0.3-1.2) and
uploaded it to DELAYED/3. Please feel free to tell me if I
should delay it longer.

Regards

    Christoph

nb: I'll ask release for an unblock as soonas it hits unstable

[dkms-2.2.0.3-1.2-nmu.diff (text/x-diff, attachment)]

Message #46 received at 688904@bugs.debian.org (full text, mbox, reply):

From: Michael Gilbert <mgilbert@debian.org>

To: Christoph Egger <christoph@debian.org>, 688904@bugs.debian.org

Subject: Re: [Pkg-dkms-maint] Bug#688904: dkms: diff for NMU version 2.2.0.3-1.2

Date: Sun, 7 Oct 2012 15:07:58 -0400

control: severity -1 important

On Fri, Oct 5, 2012 at 11:57 PM, Christoph Egger wrote:
> tags 688904 + pending
> thanks
>
> Dear maintainer,
>
> I've prepared an NMU for dkms (versioned as 2.2.0.3-1.2) and
> uploaded it to DELAYED/3. Please feel free to tell me if I
> should delay it longer.

This is not release-critical since kfreebsd 10 is not part of the
wheezy release.

Best wishes,
Mike

Message #53 received at 688904@bugs.debian.org (full text, mbox, reply):

From: Christoph Egger <christoph@debian.org>

To: Michael Gilbert <mgilbert@debian.org>

Cc: 688904@bugs.debian.org

Subject: Re: [Pkg-dkms-maint] Bug#688904: dkms: diff for NMU version 2.2.0.3-1.2

Date: Sun, 07 Oct 2012 13:45:08 -0700

Michael Gilbert <mgilbert@debian.org> writes:
> On Fri, Oct 5, 2012 at 11:57 PM, Christoph Egger wrote:
>> tags 688904 + pending
>> thanks
>>
>> Dear maintainer,
>>
>> I've prepared an NMU for dkms (versioned as 2.2.0.3-1.2) and
>> uploaded it to DELAYED/3. Please feel free to tell me if I
>> should delay it longer.
>
> This is not release-critical since kfreebsd 10 is not part of the
> wheezy release.

Might still affect partial upgrades to jessi so not sure on how critical
it actually is though I agree it's not clearly RC.

Regards

    Christoph

Message #58 received at 688904-close@bugs.debian.org (full text, mbox, reply):

From: Christoph Egger <christoph@debian.org>

To: 688904-close@bugs.debian.org

Subject: Bug#688904: fixed in dkms 2.2.0.3-1.2

Date: Tue, 09 Oct 2012 04:17:30 +0000

Source: dkms
Source-Version: 2.2.0.3-1.2

We believe that the bug you reported is fixed in the latest version of
dkms, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 688904@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Christoph Egger <christoph@debian.org> (supplier of updated dkms package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Fri, 05 Oct 2012 20:52:34 -0700
Source: dkms
Binary: dkms
Architecture: source all
Version: 2.2.0.3-1.2
Distribution: unstable
Urgency: low
Maintainer: Dynamic Kernel Modules Support Team <pkg-dkms-maint@lists.alioth.debian.org>
Changed-By: Christoph Egger <christoph@debian.org>
Description: 
 dkms       - Dynamic Kernel Module Support Framework
Closes: 688904
Changes: 
 dkms (2.2.0.3-1.2) unstable; urgency=low
 .
   * Non-maintainer upload.
   * Don't fail if kernel major version number is not single-digit (like on
     kfreebsd-10) (Closes: 688904)
Checksums-Sha1: 
 7339cf55d15531d78a9889142ebe586cae0f3e46 1978 dkms_2.2.0.3-1.2.dsc
 9a444e8de50ca3e81322c63a8b19b5082ef83937 15472 dkms_2.2.0.3-1.2.debian.tar.gz
 3fb938a79e01f0e184170c99f5258d80feb11c30 77412 dkms_2.2.0.3-1.2_all.deb
Checksums-Sha256: 
 9f29fe949e76f142209fa2e28ff263e9f8cdf0b87cc7e2c76fe1caf7fc20e400 1978 dkms_2.2.0.3-1.2.dsc
 0548351e619e55b920840111555566b277a84ac4f1fc253946ab278c06eee72d 15472 dkms_2.2.0.3-1.2.debian.tar.gz
 b009da081853fff0c46cfbe252e881955467d9a1aff8885df5e388c42ab9c3e4 77412 dkms_2.2.0.3-1.2_all.deb
Files: 
 a522c4eb0b5279afda4a81b06f3bc89d 1978 kernel optional dkms_2.2.0.3-1.2.dsc
 cbf12004a17262c775e8241183cce24c 15472 kernel optional dkms_2.2.0.3-1.2.debian.tar.gz
 01b3282c8bc4bb444c45304953463035 77412 kernel optional dkms_2.2.0.3-1.2_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/kFreeBSD)

iQIcBAEBAgAGBQJQb6vDAAoJEKv/7bJACMb5aZIP/RPquQTWNCAnBAFhJwlQfWGq
9IixyGiUijk6IG/WERwWkDRP/3Vi8uKAYTNNCkc7jygC41wX3gb0SOfkOEuQl3ZK
iT8n/dwVF5Ww+k7nU2QiIyDQCjhuEsp8bbtxHP+LFZkoWMW2IF4xoG54joGWGXFN
jNWob0covfOyWfK/5ugit7749hevH2iOBzdlN6BFzSY+Xu6S+jTFsBK6oFhZBvos
Nj7etXrL3VYUn06t7QEZneCkS/aVKQfC8L8vVeHLj0xKusWBRDxM3fqUVXMFCNXR
AK88q+dYf3hywSrf36FXQuigW7hH+2r/5Pd8QMkEI+0zJHD0dfkcTQTS4P386DF/
8LHz0owJtHUVNEUmUjdWKHrK84wOjnkfUbhrn0M8IDRwepIag1mVkXs7fS7tPLsa
3w6jvsqTd7Edhh0cHzX3TqjQM2ptMiAmCtoZAR7Fdl7EBOtOVoSb9pH8M16WAt1Z
Eh0P0SdJs2T53UV+s2at+y76eqYApnEXmwPYQ6dZrahgQCTrCMdVJfbe0b6PMRkV
ZyiS8Cj1npqMPMNrxJxFrFeShu+YnwKGpxQ1ZKmMzDxi3BLFMqk0tgaM81p7MrZU
1LNIOIx5Izh9xIoV/yXk72sirXrXqD4oOzXK5b91GpuEukYEzhVpYAiXBjefbSh6
+SI1GW9TRPLMVmZv89A/
=TWRI
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.