Debian Bug report logs - #659672
dkms: dkms uninstall does not work

Toggle useless messages

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

From: Nobuhiro Iwamatsu <iwamatsu@nigauri.org>

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

Subject: dkms: dkms uninstall does not work

Date: Mon, 13 Feb 2012 08:40:11 +0900

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

Source: dkms
Version: 2.2.0.3-1
Severity: important
Tags: patch

Hi,

'dkms uninstall' does not work.

-----
$ dkms status
hello, 0.0.1, 3.0.0-1-amd64, x86_64: installed
$ sudo dkms uninstall -m hello -v 0.0.1 -k 3.0.0-1-amd64
$ dkms status
hello, 0.0.1, 3.0.0-1-amd64, x86_64: installed
-----

Because no values return in have_one_kernel function, processing
finishes with this function.
I created a patch which revise this problem.

Could you check this patch?

Best regards,
  Nobuhiro

-- 
Nobuhiro Iwamatsu
   iwamatsu at {nigauri.org / debian.org}
   GPG ID: 40AD1FA6

[dkms_have_one_kernel_return_value.patch (text/x-patch, attachment)]

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

From: Wilfried Klaebe <w+reportbug@chaos.in-kiel.de>

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

Subject: Re: dkms: dkms uninstall does not work

Date: Thu, 31 Jul 2014 16:35:35 +0200

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

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

[dkms-fix-have_one_kernel.patch (text/x-diff, attachment)]

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

From: Gaudenz Steinlin <gaudenz@debian.org>

To: 659672@bugs.debian.org

Cc: control@bugs.debian.org

Subject: dkms uninstall does not work, upstream fix

Date: Thu, 14 Aug 2014 11:58:25 +0200

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

tags +fixed-upstream
thanks

Hi

In addition to the two patches already posted to this bug, this is also
fixed upstream in commit eea7213f80ed25e9a5de1a49b7da0e7108c7b7d1 since
end of 2012 (!). Is anyone still taking care of dkms in Debian? Or
should this package be orphaned?

Gaudenz

[Message part 2 (application/pgp-signature, inline)]

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

From: Giuseppe Iuculano <iuculano@debian.org>

To: 659672@bugs.debian.org,control@bugs.debian.org,659672-submitter@bugs.debian.org

Subject: [1fa1a31] Fix for Bug#659672 committed to git

Date: Sat, 04 Oct 2014 17:40:40 +0000

tags 659672 + pending
thanks

Hello,

     The following change has been committed for this bug by
 Giuseppe Iuculano <iuculano@debian.org> on Sat, 4 Oct 2014 19:38:21 +0200.
 The fix will be in the next upload. 
=========================================================================
fixes have_one_kernel for non-zero return code.

Closes: #659672
Closes: #735181
Thanks: Mario Limonciello
=========================================================================

You can check the diff of the fix at:

    ;a=commitdiff;h=1fa1a31

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

From: Giuseppe Iuculano <iuculano@debian.org>

To: 659672-close@bugs.debian.org

Subject: Bug#659672: fixed in dkms 2.2.0.3-2

Date: Sat, 04 Oct 2014 18:18:37 +0000

Source: dkms
Source-Version: 2.2.0.3-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 659672@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Giuseppe Iuculano <iuculano@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@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Sat, 04 Oct 2014 19:51:23 +0200
Source: dkms
Binary: dkms
Architecture: source all
Version: 2.2.0.3-2
Distribution: unstable
Urgency: low
Maintainer: Dynamic Kernel Modules Support Team <pkg-dkms-maint@lists.alioth.debian.org>
Changed-By: Giuseppe Iuculano <iuculano@debian.org>
Description:
 dkms       - Dynamic Kernel Module Support Framework
Closes: 659672 693634 735181 757758 761728 763929
Changes:
 dkms (2.2.0.3-2) unstable; urgency=low
 .
   * [49fccbc] Depends on kmod | kldutils (Closes: #761728)
   * [9064776] Fixed "sed without options" issue.
     Thanks to Cristian Ionescu-Idbohrn (Closes: #757758, 763929)
   * [1fa1a31] fixes have_one_kernel for non-zero return code.
     Thanks to Mario Limonciello (Closes: #659672, #735181)
   * [ba6140d] Added Multi-Arch: foreign.
     Thanks to Ben Hutchings (Closes: #693634)
Checksums-Sha1:
 0f2ca9be46e6dbc7fd923b1689f6b0327a687234 1957 dkms_2.2.0.3-2.dsc
 3505cf358926700f5a92bee9ee8541d9016b1f90 15132 dkms_2.2.0.3-2.debian.tar.xz
 8819f15ee9d70b89e8d833d200e09dbd94f3ab8d 70904 dkms_2.2.0.3-2_all.deb
Checksums-Sha256:
 d6b8d4fc98f017696f74fb78fabf324b36d307b6e6ddb6c96c50e80552c86512 1957 dkms_2.2.0.3-2.dsc
 fae90aa33ac8ec783ef1ee7d44e1dc4e337c5738b6f0ff0684218852a00c2fbb 15132 dkms_2.2.0.3-2.debian.tar.xz
 86e60e515d9f797bfb7baf1bebe41cbedd2b6d9569f3dfff51f55f49af54989f 70904 dkms_2.2.0.3-2_all.deb
Files:
 a86e1e553d28914fd36d16db80c213dd 70904 kernel optional dkms_2.2.0.3-2_all.deb
 fd6b2de9fdeacd304302353c478d8cef 1957 kernel optional dkms_2.2.0.3-2.dsc
 c521f0084dc8a0be0cac6198fabbb0b6 15132 kernel optional dkms_2.2.0.3-2.debian.tar.xz

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBCAAGBQJUMDbBAAoJEI9hzo2UfbET8+QQAKZcejVmTwpYLxq/s5iqf2hu
fbyVsEt/LlmioEALTO1F3KvzHvw4DJq+faRwy1kP8T4twYMqvHWNHHCw9F8UqI2Z
Wi0etxwnqbZjb34YXGcLEcQizVcq2PKcAPVmI2it/O8Yg0zIM65hCiDaH8GsQEqc
laIbGECh6Hdf7eHKjIgjswvn2j1Vl87cqSkOvz+uodL4LonnaRIgABsb6iTcXkmk
npWxHfDCvQxaflOHjKtSlPOReqYW+Zc7CPeTRmDHwCcX35o0WQlyeVm/NsEjC38M
eXqkKJF3CuKO2EdsNsHzkz0dnCekZDK8C0txvg+PTUXysgTQagYVvmdPhyKEs8kt
KteC/GWPtGP+1cRC2MTA4RfRS28tnnGul6saTHhV6aS/tZYp1Akb1r9d3bfgBXpg
3JNVwTWTmvh7UIEF78N0+vANLnH/adQThAAY4kq6QfA2h5WExrIlRi7o6cxR7e7j
kAml7DwKhoIlKQ8HcGqBw2ikihE3Bdp9UoMHKs4S9fgxBhGz0sAGbaDM+pOzyeh6
Hz6a+K4NNo3WGAbnFY2YZHXj+yShVnKG+XFHQ4B0p+r5e+T1tUN1GtFuJ3D+6QpC
leRxjPK1UqwZLxUiEYxNlrNHOhmQyFNJbxvIZyuCSaskOgy6uVAKU2KA+PF0r2x2
9Du0sVckNAd/EATltg+H
=/z47
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.