Sync dkms 2.1.1.2-2 (main) from Debian testing (main)

Bug #534843 reported by Anders Kaseorg
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
dkms (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Please sync dkms 2.1.1.2-2 (main) from Debian testing (main)

Explanation of the Ubuntu delta and why it can be dropped:

Debian has been updating their dkms package so that it can be synced
into Ubuntu. The current Debian package is identical to the Ubuntu
package plus the following seven commits (modulo debian/changelog):

  * [ff95487] dh_dkms: continue the loop if there's nothing to do on the
    current package. (Closes: #568580)
    http://git.debian.org/?p=collab-maint/dkms.git;a=commitdiff;h=ff95487
    LP bug #534832
  * [73dd83f] dh_dkms: specify that version mangling only happens if
    PACKAGE_VERSION=#MODULE_VERSION# in dkms.conf
    http://git.debian.org/?p=collab-maint/dkms.git;a=commitdiff;h=73dd83f
  * [cdda61c] debian/scripts/postinst-dkms: really make sure the error
    is thrown when nothing is found. (Closes: #568591)
    http://git.debian.org/?p=collab-maint/dkms.git;a=commitdiff;h=cdda61c
    LP bug #534833
  * [2930aa9] Insert the package version directly into postinst at
    creation time
    http://git.debian.org/?p=collab-maint/dkms.git;a=commitdiff;h=2930aa9
  * [9ccbdc7] Skip prerm removal if there are no modules for that
    version
    http://git.debian.org/?p=collab-maint/dkms.git;a=commitdiff;h=9ccbdc7
  * [e8aeb49] Add a comment in debian/patches/series and fix quilt-
    build-dep-but-no-series-file lintian warning
    http://git.debian.org/?p=collab-maint/dkms.git;a=commitdiff;h=e8aeb49
  * [bc89600] Fixed a minor spelling error in dh_dkms man page
    http://git.debian.org/?p=collab-maint/dkms.git;a=commitdiff;h=bc89600

The first and third are critical bug fixes to make dh_dkms work at
all, and the rest would be nice to have; plus it’s useful for Ubuntu
and Debian to share the dkms package.

Changelog entries since current lucid version 2.1.1.2-0ubuntu1:

dkms (2.1.1.2-2) unstable; urgency=low

  * [67fb76a] Do not remove /etc/header_postinst.d/dkms in preinst. Hopefully
    now Ubuntu can sync dkms from Debian

 -- Giuseppe Iuculano <email address hidden> Tue, 23 Feb 2010 10:02:40 +0100

dkms (2.1.1.2-1) unstable; urgency=low

  [ David Paleino ]
  * [ff95487] dh_dkms: continue the loop if there's nothing to do on the
    current package. (Closes: #568580)
  * [73dd83f] dh_dkms: specify that version mangling only happens if
    PACKAGE_VERSION=#MODULE_VERSION# in dkms.conf
  * [cdda61c] debian/scripts/postinst-dkms: really make sure the error
    is thrown when nothing is found. (Closes: #568591)

  [ Frédéric Brière ]
  * [2930aa9] Insert the package version directly into postinst at
    creation time
  * [9ccbdc7] Skip prerm removal if there are no modules for that
    version

  [ Giuseppe Iuculano ]
  * [898ab8c] Imported Upstream version 2.1.1.2
  * [4fff928] Removed 01_shell_error.patch, merged in upstream
  * [c5566c7] Removed 02_header_postinst.patch to avoid delta with
    Ubuntu. Hopefully /etc/kernel/header_postinst.d directory will be
    supported in Debian soon
  * [e8aeb49] Add a comment in debian/patches/series and fix quilt-
    build-dep-but-no-series-file lintian warning
  * [bc89600] Fixed a minor spelling error in dh_dkms man page

 -- Giuseppe Iuculano <email address hidden> Mon, 22 Feb 2010 15:34:12 +0100

Related branches

Revision history for this message
Anders Kaseorg (andersk) wrote :

I guess this probably needs a fakesync because Debian repacked the .orig.tar.gz (its contents are the same).

Revision history for this message
Mario Limonciello (superm1) wrote :

Ack

Changed in dkms (Ubuntu):
status: New → Confirmed
tags: added: kernel-series-unknown
tags: removed: kernel-series-unknown
Revision history for this message
Daniel Holbach (dholbach) wrote :

Unsubscribing sponsors.

Revision history for this message
Anders Kaseorg (andersk) wrote :

broder points out to me that this probably needs a FeatureFreezeException. However, I think the case is pretty straightforward.

Revision history for this message
Anders Kaseorg (andersk) wrote :

(oops, pressed submit too early)

broder points out to me that this probably needs a FeatureFreezeException. However, I think the case is pretty straightforward. The entire diff consists of bug fixes to dh_dkms. I’ve built and tested the new dkms package, and built and tested the new version of openafs that uses dh_dkms against it, at https://launchpad.net/~anders-kaseorg/+archive/openafs ; everything works fine.

Changed in dkms (Ubuntu):
status: Confirmed → Incomplete
Revision history for this message
Anders Kaseorg (andersk) wrote :

Jamie: What is missing from this report?

Revision history for this message
Anders Kaseorg (andersk) wrote :

ubuntu-release: Is a FFe required/granted for this sync?

I don’t think it is required, because the patches are only bugfixes to dh_dkms (which currently doesn’t work at all) and documentation updates. However, ubuntu-archive has requested a decision.

Revision history for this message
Martin Pitt (pitti) wrote :

No FFE required, just bug fixes. Great to see that we are in sync again (at least fakesync...), but maybe there can be a new upstream release soon, too? (/me waves to Mario :) )

Please go ahead with the (fake)sync.

Revision history for this message
Anders Kaseorg (andersk) wrote :

Thanks Martin. Marking confirmed again.

Changed in dkms (Ubuntu):
status: Incomplete → Confirmed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package dkms - 2.1.1.2-2fakesync1

---------------
dkms (2.1.1.2-2fakesync1) lucid; urgency=low

  * Fakesync with debian (LP: #534843)
 -- Mario Limonciello <email address hidden> Sun, 28 Mar 2010 00:45:36 -0500

Changed in dkms (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.