diff mbox

[Maverick,Natty] SRU: virtual image needs another nls module

Message ID 4DBA8EDA.5060407@canonical.com
State New
Headers show

Commit Message

Stefan Bader April 29, 2011, 10:11 a.m. UTC
It turns out that cp850 alone is not enough but also needs iso-8859-1.
This patch would add it (applies to maverick and natty).
For Oneiric we either need same or go for a different approach of just including
everything fs related into virtual.

-Stefan

Comments

John Johansen April 29, 2011, 10:24 a.m. UTC | #1
On 04/29/2011 03:11 AM, Stefan Bader wrote:
> It turns out that cp850 alone is not enough but also needs iso-8859-1.
> This patch would add it (applies to maverick and natty).
> For Oneiric we either need same or go for a different approach of just including
> everything fs related into virtual.
> 
> -Stefan
> 
> From eebc798af8e188d61494889f8280b67a21fd4db8 Mon Sep 17 00:00:00 2001
> From: Stefan Bader <stefan.bader@canonical.com>
> Date: Fri, 29 Apr 2011 11:27:14 +0200
> Subject: [PATCH] UUNTU: Include nls_iso8859-1 for virtual images
>
> BugLink: http://bugs.launchpad.net/bugs/732046
>
> It turns out that including cp437 alone is not sufficient to mount
> a vfat file system. With this additional module, things were reported
> to be working.
>
> Signed-off-by: Stefan Bader <stefan.bader@canonical.com>

Acked-by: John Johansen <john.johansen@canonical.com>
Herton Ronaldo Krzesinski April 29, 2011, 12:48 p.m. UTC | #2
On Fri, Apr 29, 2011 at 12:11:38PM +0200, Stefan Bader wrote:
> It turns out that cp850 alone is not enough but also needs iso-8859-1.
> This patch would add it (applies to maverick and natty).
> For Oneiric we either need same or go for a different approach of just including
> everything fs related into virtual.
> 
> -Stefan

> From eebc798af8e188d61494889f8280b67a21fd4db8 Mon Sep 17 00:00:00 2001
> From: Stefan Bader <stefan.bader@canonical.com>
> Date: Fri, 29 Apr 2011 11:27:14 +0200
> Subject: [PATCH] UUNTU: Include nls_iso8859-1 for virtual images
> 
> BugLink: http://bugs.launchpad.net/bugs/732046
> 
> It turns out that including cp437 alone is not sufficient to mount
> a vfat file system. With this additional module, things were reported
> to be working.
> 
> Signed-off-by: Stefan Bader <stefan.bader@canonical.com>

Acked-by: Herton Ronaldo Krzesinski <herton.krzesinski@canonical.com>
Tim Gardner April 29, 2011, 1:25 p.m. UTC | #3
On 04/29/2011 04:11 AM, Stefan Bader wrote:
> It turns out that cp850 alone is not enough but also needs iso-8859-1.
> This patch would add it (applies to maverick and natty).
> For Oneiric we either need same or go for a different approach of just including
> everything fs related into virtual.
>
> -Stefan
>

applied to Maverick/Natty/Oneiric with spelling correction.
Stefan Bader April 29, 2011, 1:32 p.m. UTC | #4
On 04/29/2011 03:25 PM, Tim Gardner wrote:
> On 04/29/2011 04:11 AM, Stefan Bader wrote:
>> It turns out that cp850 alone is not enough but also needs iso-8859-1.
>> This patch would add it (applies to maverick and natty).
>> For Oneiric we either need same or go for a different approach of just including
>> everything fs related into virtual.
>>
>> -Stefan
>>
> 
> applied to Maverick/Natty/Oneiric with spelling correction.
> 
Thanks (specially for fixing up the speeling) :)
diff mbox

Patch

From eebc798af8e188d61494889f8280b67a21fd4db8 Mon Sep 17 00:00:00 2001
From: Stefan Bader <stefan.bader@canonical.com>
Date: Fri, 29 Apr 2011 11:27:14 +0200
Subject: [PATCH] UUNTU: Include nls_iso8859-1 for virtual images

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

It turns out that including cp437 alone is not sufficient to mount
a vfat file system. With this additional module, things were reported
to be working.

Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
---
 debian.master/control.d/virtual.inclusion-list |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/debian.master/control.d/virtual.inclusion-list b/debian.master/control.d/virtual.inclusion-list
index 70277fe..c4bd08e 100644
--- a/debian.master/control.d/virtual.inclusion-list
+++ b/debian.master/control.d/virtual.inclusion-list
@@ -88,6 +88,7 @@  fs/configfs/*
 fs/exportfs/*
 fs/fat/*
 fs/nls/nls_cp437.ko
+fs/nls/nls_iso8859-1.ko
 fs/isofs/*
 fs/cifs/*
 fs/xfs/*
-- 
1.7.0.4