Debian Bug report logs - #553665
dkms: debhelper support for DKMS

Toggle useless messages

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

From: Frédéric Brière <fbriere@fbriere.net>

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

Subject: dkms: debhelper support for DKMS

Date: Sun, 01 Nov 2009 15:03:38 -0500

Package: dkms
Version: 2.1.0.1-1
Severity: wishlist

Now that DKMS has been proclaimed as the Way of the Future[tm] for
Debian, it would be nice to have a dh_dkms to take care of the heavy
lifting, at least regarding the postinst/prerm.  (I think #553542
nicely illustrates that need.)

I wouldn't mind lending a hand on the Perl/Debhelper side of things if
need be.


-- System Information:
Debian Release: squeeze/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: i386 (i686)

Kernel: Linux 2.6.26.8 (SMP w/1 CPU core)
Locale: LANG=en_CA.UTF-8, LC_CTYPE=en_CA.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages dkms depends on:
ii  build-essential           11.4           Informational list of build-essent
ii  dpkg-dev                  1.15.4.1       Debian package development tools
ii  gcc                       4:4.3.3-9+nmu1 The GNU C compiler
ii  make                      3.81-6         An utility for Directing compilati
ii  module-init-tools         3.11-1         tools for managing Linux kernel mo

Versions of packages dkms recommends:
ii  fakeroot           1.14.3                Gives a fake root environment
ii  linux-headers-2.6. 2.6.26.8-10.00.Custom Header files related to Linux kern
ii  linux-image-2.6.26 2.6.26.8-10.00.Custom Linux kernel binary image for vers
ii  lsb-release        3.2-23                Linux Standard Base version report
ii  menu               2.1.42                generates programs menu for all me
ii  patch              2.5.9-5               Apply a diff file to an original
ii  sudo               1.7.2p1-1             Provide limited super user privile

dkms suggests no packages.

-- no debconf information

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

From: Mario Limonciello <mario_limonciello@dell.com>

To: Frédéric Brière <fbriere@fbriere.net>, 553665@bugs.debian.org, DKMS maintainers <pkg-dkms-maint@lists.alioth.debian.org>

Subject: Re: [Pkg-dkms-maint] Bug#553665: dkms: debhelper support for DKMS

Date: Mon, 02 Nov 2009 13:11:28 -0600

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

It should be worthwhile to note that there is an upstream "common"
postinst that is available with DKMS 2.1.0 and later.  It would be most
ideal to work off that if at all possible.

http://linux.dell.com/git/?p=dkms.git;a=blob;f=dkms_common.postinst;h=0424b76dc8075a741d190e62a7c237e4ab8c87a8;hb=HEAD

Here's how it's normally used in the DKMS mkdeb template:

http://linux.dell.com/git/?p=dkms.git;a=blob;f=template-dkms-mkdeb/debian/postinst;h=d856c08349812564602ff49335968f6763fc3af8;hb=HEAD

Regards

Frédéric Brière wrote:
> Package: dkms
> Version: 2.1.0.1-1
> Severity: wishlist
>
> Now that DKMS has been proclaimed as the Way of the Future[tm] for
> Debian, it would be nice to have a dh_dkms to take care of the heavy
> lifting, at least regarding the postinst/prerm.  (I think #553542
> nicely illustrates that need.)
>
> I wouldn't mind lending a hand on the Perl/Debhelper side of things if
> need be.
>
>
> -- System Information:
> Debian Release: squeeze/sid
>   APT prefers unstable
>   APT policy: (500, 'unstable')
> Architecture: i386 (i686)
>
> Kernel: Linux 2.6.26.8 (SMP w/1 CPU core)
> Locale: LANG=en_CA.UTF-8, LC_CTYPE=en_CA.UTF-8 (charmap=UTF-8)
> Shell: /bin/sh linked to /bin/dash
>
> Versions of packages dkms depends on:
> ii  build-essential           11.4           Informational list of build-essent
> ii  dpkg-dev                  1.15.4.1       Debian package development tools
> ii  gcc                       4:4.3.3-9+nmu1 The GNU C compiler
> ii  make                      3.81-6         An utility for Directing compilati
> ii  module-init-tools         3.11-1         tools for managing Linux kernel mo
>
> Versions of packages dkms recommends:
> ii  fakeroot           1.14.3                Gives a fake root environment
> ii  linux-headers-2.6. 2.6.26.8-10.00.Custom Header files related to Linux kern
> ii  linux-image-2.6.26 2.6.26.8-10.00.Custom Linux kernel binary image for vers
> ii  lsb-release        3.2-23                Linux Standard Base version report
> ii  menu               2.1.42                generates programs menu for all me
> ii  patch              2.5.9-5               Apply a diff file to an original
> ii  sudo               1.7.2p1-1             Provide limited super user privile
>
> dkms suggests no packages.
>
> -- no debconf information
>
>
>
> _______________________________________________
> Pkg-dkms-maint mailing list
> Pkg-dkms-maint@lists.alioth.debian.org
> http://lists.alioth.debian.org/mailman/listinfo/pkg-dkms-maint
>   

-- 
Mario Limonciello
*Dell | Linux Engineering*
mario_limonciello@dell.com

[signature.asc (application/pgp-signature, attachment)]

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

From: Frédéric Brière <fbriere@fbriere.net>

To: 553665@bugs.debian.org

Subject: Re: [Pkg-dkms-maint] Bug#553665: dkms: debhelper support for DKMS

Date: Tue, 3 Nov 2009 10:01:55 -0500

On Mon, Nov 02, 2009 at 01:11:28PM -0600, Mario Limonciello wrote:
> It should be worthwhile to note that there is an upstream "common"
> postinst that is available with DKMS 2.1.0 and later.  It would be most

Yes, /usr/lib/dkms/common.postinst.  I hadn't thought about the version
dependency, though -- thanks for reminding me.  (That's another thing
that dh_dkms would get right.)

BTW, I couldn't help but notice that the postinst you wrote for
virtualbox-ose calls dkms directly.  Was that intentional?

> Here's how it's normally used in the DKMS mkdeb template:

Thanks!  I really should take a closer look at the mkdeb rule.

I'm not sure I understand the $ARCH test, though.  Why does this generic
postinst arbitrarily decide which arches are supported by a module?


Also, could somebody comment on what a prerm should look like?  From
what I understand, it should be something like:

  dkms remove -m $NAME -v $CVERSION --all || true

(virtualbox-ose redirects stdout but not stderr, openafs does neither.)

This should be run on remove/upgrade, at least, and possibly deconfigure
as well, I'm not sure.  virtualbox-ose also does it on purge, which I
believe is redundant, if not wrong (dkms may not be installed at that
point).


-- 
I was attacked by dselect as a small child and have since avoided
debian.
		-- Andrew Morton

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

From: David Paleino <dapal@debian.org>

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

Subject: [19ac85e] Fix for Bug#553665 committed to git

Date: Sat, 16 Jan 2010 22:28:46 +0000

tags 553665 + pending
thanks

Hello,

     The following change has been committed for this bug by
 David Paleino <dapal@debian.org> on Sat, 16 Jan 2010 23:26:24 +0100.
 The fix will be in the next upload. 
=========================================================================
Added DKMS debhelper script (Closes: #553665)

  This script is usable both by calling dh_dkms directly, and by
  passing --with dkms to dh calls in dh7-style debian/rules. Read
  dh_dkms(1) for more information.
=========================================================================

You can check the diff of the fix at:

    http://git.debian.org/?p=collab-maint/dkms.git;a=commitdiff;h=19ac85e

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

From: Giuseppe Iuculano <iuculano@debian.org>

To: 553665-close@bugs.debian.org

Subject: Bug#553665: fixed in dkms 2.1.1.1-1

Date: Wed, 27 Jan 2010 09:27:27 +0000

Source: dkms
Source-Version: 2.1.1.1-1

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:

dkms_2.1.1.1-1.diff.gz
  to main/d/dkms/dkms_2.1.1.1-1.diff.gz
dkms_2.1.1.1-1.dsc
  to main/d/dkms/dkms_2.1.1.1-1.dsc
dkms_2.1.1.1-1_all.deb
  to main/d/dkms/dkms_2.1.1.1-1_all.deb
dkms_2.1.1.1.orig.tar.gz
  to main/d/dkms/dkms_2.1.1.1.orig.tar.gz



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 553665@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@debian.org)


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

Format: 1.8
Date: Wed, 27 Jan 2010 10:02:27 +0100
Source: dkms
Binary: dkms
Architecture: source all
Version: 2.1.1.1-1
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: 553665
Changes: 
 dkms (2.1.1.1-1) unstable; urgency=low
 .
   [ David Paleino ]
   * [19ac85e] Added DKMS debhelper script (Closes: #553665)
 .
   [ Giuseppe Iuculano ]
   * [9d66264] Imported Upstream version 2.1.1.1
   * [0735c11] Removed 01_upstart.patch
   * [ec26539] Merge from Ubuntu: Remove the init script and Upstart job.
     There is no reason that DKMS needs to run on boot; you can build
     modules for non-running kernels just fine at installation time.
   * [48ff9a4] Correct a minor shell error in dkms_autoinstaller
   * [5c76a45] Fixed a minor spelling error in dh_dkms man page
   * [c611461] debian/preinst: Use set -e
Checksums-Sha1: 
 6501196067e0787c4a4d09b163497a353d1236cc 1264 dkms_2.1.1.1-1.dsc
 85c72b915d0987c8c6bf2a9e465b294c5fac32bd 81042 dkms_2.1.1.1.orig.tar.gz
 417f37aaded2a8ab33c0ef764543a41d79641896 11706 dkms_2.1.1.1-1.diff.gz
 ab7038ee42e5c7b535732c35e1df902e505b6405 68758 dkms_2.1.1.1-1_all.deb
Checksums-Sha256: 
 60dea3eacf49d09e26213ce7a835869fd97bd67f70d6a275962d060908857667 1264 dkms_2.1.1.1-1.dsc
 7f4150b3fcf50f92c5f6b172bd41602b8ba3588c41700e9f78eb35e92d97b3be 81042 dkms_2.1.1.1.orig.tar.gz
 0d28b9a579ee10bc4264941501a734deecdef7f4506a3ba4244d174c022bf72b 11706 dkms_2.1.1.1-1.diff.gz
 0bb6f6ba2a315501fc0a0167600a3ed85a7aa6bd62a582cc6a0fead094e6440a 68758 dkms_2.1.1.1-1_all.deb
Files: 
 023e3ecb83d32161067853f5163c7483 1264 kernel optional dkms_2.1.1.1-1.dsc
 bc8c6c5e6d7d16e4c61251c47e134312 81042 kernel optional dkms_2.1.1.1.orig.tar.gz
 f03436a80c2d3f2c6494d65015dacc7a 11706 kernel optional dkms_2.1.1.1-1.diff.gz
 858c64f6c0ac47ce8ab581b99500951c 68758 kernel optional dkms_2.1.1.1-1_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)

iEYEARECAAYFAktgAiUACgkQNxpp46476ar8KACfXayZ/3+0gCwDjx5I11T+oB9I
q8MAn2hpfdpKzSD44x47Y2C4Zc3ZQ8ig
=y4RB
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.