[PATCH 1/1] UBUNTU: Add 3 missing files to prerm remove file list

[Andy Whitcroft]

On Wed, Mar 25, 2009 at 09:19:44PM +0800, bryan.wu at canonical.com wrote:
> From: Bryan Wu <bryan.wu at 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 at 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 at canonical.com>

-apw