[Quantal,SRU,pull-request] Misc linux-backports-modules cleanup

From: Leann Ogasawara <leann.ogasawara@canonical.com>

On 10/19/2012 05:40 AM, Tim Gardner wrote:
> On 10/18/2012 03:00 PM, Leann Ogasawara wrote:
>> BugLink: http://bugs.launchpad.net/bugs/1068125
>> BugLink: http://bugs.launchpad.net/bugs/1068283
>>
>> == Quantal SRU Justification ==
>> The following pull request performs some misc cleanup for the
>> linux-backports-modules package for Quantal.
>>
>> The first patch makes the compat-wireless firmware udev helper file
>> names version and ABI specific.  We allow multiple upstream versions of
>> the compat-wireless stack to be packaged and installed.  There currently
>> exists a potential for conflicts with some of the compat-wireless
>> firmware udev helper file names due to the files only being ABI
>> specific, but not version specific, with respect to their naming
>> convention.  Fix this by making these both version and ABI specific.
>>
>
> I am not convinced this is necessary. All subsequent CW packages
> _must_ conflict, therefore you can never get into a state of clashing
> udev firmware helpers. Refer to the complex conflict relationships in
> Lucid LBM.

Hrm, seems we need our Conflicts: set up for Precise LBM too.  But
that's beyond the scope of this patch set.  I'll send a follow on patch
for that.

> Its possible that you could get into trouble if you force installed a
> kernel and CW package from another release that had the same ABI. To
> avoid that conflict you'd have to distinguish the udev helpers names
> by release name _and_ ABI. I'm not sure that scenario is worth the
> complexity.

While I agree it's not a common scenario to encounter, it seemed a
harmless enough change to at least prevent a potential conflict and
issue.  We already make the udev helper file names ABI and flavor
specific, so why not take that extra precaution and tack on the release
version as well.

>
>> The second patch removes all udebs and udeb creation logic from LBM.
>> LBM is an elective install and not part of any default server or netboot
>> installation.  It is therefore unnecessary to provide an
>> updates-modules.udeb (it was empty anyways) nor have anything to do with
>> udeb creation in general for that matter.
>>
>
> Shouldn't this patch also remove debian/d-i ?

Yep, I totally missed the obvious there.  We no longer have a need for
the kernel-versions.in file that remains in d-i.  The following patch
removes d-i/kernel-versions.in and cleans up any misc references to it
as well.  Once that's removed, git takes care of the rest for getting
rid of d-i.  I've pushed this to my pull-request branch.  Let me know if
you prefer I just squash it with the original patch.

Thanks,
Leann

From baa3fce92747794552eca1806ed73727baf32142 Mon Sep 17 00:00:00 2001
From: Leann Ogasawara <leann.ogasawara@canonical.com>
Date: Fri, 19 Oct 2012 06:31:08 -0700
Subject: [PATCH] UBUNTU: Remove debian/d-i

BugLink: http://bugs.launchpad.net/bugs/1068125

LBM is an elective install and not part of any default server or netboot
installation.  We should remove debian/d-i as it is unnecessary.

Signed-off-by: Leann Ogasawara <leann.ogasawara@canonical.com>
---
 debian/d-i/kernel-versions.in |    4 ----
 debian/rules                  |    5 ++---
 2 files changed, 2 insertions(+), 7 deletions(-)
 delete mode 100644 debian/d-i/kernel-versions.in

[Quantal,SRU,pull-request] Misc linux-backports-modules cleanup

Commit Message

Comments

Patch