Package: dkms
Version: 2.2.0.3-1.2
Followup-For: Bug #659672

Dear Maintainer,

this bug still exists. I independently developed a different patch.

have_one_kernel() wrongly returns 1, because the last executed command
is [[ $all ]], which returns 1 because $all == "". It should return 0.

This can be achieved by either applying Nobuhiro's patch or mine,
which changes [[ $all ]] && to [[ -z $all ]] ||

-- System Information:
Debian Release: jessie/sid
  APT prefers stable-updates
  APT policy: (990, 'stable-updates'), (990, 'testing'), (990, 'stable'), (900, 
'testing-updates'), (90, 'unstable'), (9, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.15.7+radeonbios-dirty (SMP w/8 CPU cores; PREEMPT)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages dkms depends on:
ii  build-essential    11.6
ii  coreutils          8.21-1.2
ii  dpkg-dev           1.17.10
ii  gcc                4:4.9.1-1
ii  make               4.0-8
ii  module-init-tools  18-1
ii  patch              2.7.1-5

Versions of packages dkms recommends:
ii  fakeroot                                          1.20.1-1
ii  menu                                              2.1.47
ii  sudo                                              1.8.9p5-1

dkms suggests no packages.

-- no debconf information
--- a/dkms	2012-10-06 03:55:00.000000000 +0000
+++ b/dkms	2014-07-31 09:34:04.394548511 +0000
@@ -166,7 +166,7 @@
     (( ${#kernelver[@]} > 1 )) && \
 	die 4 $"The action $1 does not support multiple kernel version" \
 	$"parameters on the command line."
-    [[ $all ]] && die 5 $"The action $1 does not support the --all" \
+    [[ -z $all ]] || die 5 $"The action $1 does not support the --all" \
 	$"parameter."
 }
 

Reply via email to