FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Ubuntu > Ubuntu Kernel Team

 
 
LinkBack Thread Tools
 
Old 03-25-2009, 12:48 PM
Bryan Wu
 
Default UBUNTU: Add 3 missing files to prerm remove file list

Andy Whitcroft wrote:
> On Wed, Mar 25, 2009 at 09:19:44PM +0800, bryan.wu@canonical.com wrote:
>> From: Bryan Wu <bryan.wu@canonical.com>
>>
>> Bug: #345623
>>
>> Since commit 94cba110c064e49b437fe482dbbcb6f7b3a652d4, a remove file list was
>
> Actually that commit was the original fold down of the debian directory
> for Jaunty so its not obviously useful to have that commit listed. I
> would just drop this first sentence it adds nothing.
>
>> introduced. While purging some old kernels, Matt noticed warnings from dpkg
>
> I wouldn't quote peoples name generally. I would just make it factual.
>
> When purging old kernel, we see warnings from dpkg indicating that ...
>
>> that it was not removing /lib/modules/xxx. "modules.alias.bin modules.dep.bin
>> modules.symbols.bin" 3 files are left behind.
>>
>> This patch fixes this by adding these 3 files to the prerm remove file list.
>>
>> Signed-off-by: Bryan Wu <bryan.wu@canonical.com>
>> ---
>> debian/control-scripts/prerm | 1 +
>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/debian/control-scripts/prerm b/debian/control-scripts/prerm
>> index 47fbd21..4c0ad60 100644
>> --- a/debian/control-scripts/prerm
>> +++ b/debian/control-scripts/prerm
>> @@ -284,6 +284,7 @@ sub success () {
>> modules.ieee1394map modules.pnpbiosmap
>> modules.alias modules.ccwmap modules.inputmap
>> modules.symbols modules.ofmap modules.seriomap
>> + modules.alias.bin modules.dep.bin modules.symbols.bin
>> };
>>
>> foreach my $extra_file (@files_to_remove) {
>
> The patch looks technically correct.
>
> Acked-by: Andy Whitcroft <apw@canonical.com>
>
> -apw
>

Totally understood. Need I resend this patch?

--
Bryan Wu <bryan.wu@canonical.com>
Kernel Developer +86.138-1617-6545 Mobile
Ubuntu Kernel Team
Canonical Ltd. www.canonical.com
Ubuntu - Linux for human beings | www.ubuntu.com

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 02-11-2010, 10:33 AM
Bryan Wu
 
Default UBUNTU: Add 3 missing files to prerm remove file list

BugLink: http://bugs.launchpad.net/bugs/345623
BugLink: http://bugs.launchpad.net/bugs/415832

While purging some old kernels, Matt noticed warnings from dpkg
that it was not removing /lib/modules/xxx. "modules.alias.bin modules.dep.bin
modules.symbols.bin" 3 files are left behind. The new files are the result
of changes to the modules-tools-init package, e.g., modprobe and depmod now
create and use a binary cache file.

This patch fixes this by adding these 3 files to the prerm remove file list.

Signed-off-by: Bryan Wu <bryan.wu@canonical.com>
Acked-by: Andy Whitcroft <apw@canonical.com>
Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
Signed-off-by: Andy Whitcroft <apw@canonical.com>
Acked-by: Tim Gardner <tim.gardner@canonical.com>
Acked-by: Stefan Bader <stefan.bader@canonical.com>

(cherry-picked from 8b453930465c15f490ea44c2987b0a1bfe71ed66 for fsl-imx51)
Signed-off-by: Bryan Wu <bryan.wu@canonical.com>
---
debian.fsl-imx51/control-scripts/prerm | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/debian.fsl-imx51/control-scripts/prerm b/debian.fsl-imx51/control-scripts/prerm
index 3d1fe38..68633bf 100644
--- a/debian.fsl-imx51/control-scripts/prerm
+++ b/debian.fsl-imx51/control-scripts/prerm
@@ -284,6 +284,7 @@ sub success () {
modules.ieee1394map modules.pnpbiosmap
modules.alias modules.ccwmap modules.inputmap
modules.symbols modules.ofmap modules.seriomap
+ modules.alias.bin modules.dep.bin modules.symbols.bin
};

foreach my $extra_file (@files_to_remove) {
--
1.6.3.3


--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 

Thread Tools




All times are GMT. The time now is 02:35 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org