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:19 PM
 
Default UBUNTU: Add 3 missing files to prerm remove file list

From: Bryan Wu <bryan.wu@canonical.com>

Bug: #345623

Since commit 94cba110c064e49b437fe482dbbcb6f7b3a652d4, a remove file list was
introduced. 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.

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) {
--
1.6.0.4


--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 03-25-2009, 12:32 PM
Andy Whitcroft
 
Default UBUNTU: Add 3 missing files to prerm remove file list

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

--
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 06:00 AM.

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