Update virinst via wheezy-proposed-updates
Hi,
the tools from virtinst will fail to do anything useful with a bad
locale sitting. The patch is simple and cherry-picked from upstream.
Would this be a case to be fixed via w-p-u? Sid already has a newer
upstream version. Debdiff is attached.
Cheers,
-- Guido
-- System Information:
Debian Release: 7.0
APT prefers testing
APT policy: (990, 'testing'), (50, 'unstable'), (1, 'experimental')
Architecture: i386 (i686)
Kernel: Linux 3.2.0-4-686-pae (SMP w/2 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
diff -Nru virtinst-0.600.1/debian/changelog virtinst-0.600.1/debian/changelog
--- virtinst-0.600.1/debian/changelog 2012-04-16 14:24:13.0 +0200
+++ virtinst-0.600.1/debian/changelog 2013-01-15 17:06:42.0 +0100
@@ -1,3 +1,10 @@
+virtinst (0.600.1-3) wheezy-proposed-updates; urgency=low
+
+ * [3723835] Don't fail if we can't set locale (debian bug #697864)
+Thanks to Cole Robinson (Closes: #697864)
+
+ -- Guido Günther Tue, 15 Jan 2013 17:06:27 +0100
+
virtinst (0.600.1-2) unstable; urgency=low
* [5e59401] Fix location of Debian daily builds.
diff -Nru
virtinst-0.600.1/debian/patches/0005-Don-t-fail-if-we-can-t-set-locale-debian-bug-697864.patch
virtinst-0.600.1/debian/patches/0005-Don-t-fail-if-we-can-t-set-locale-debian-bug-697864.patch
---
virtinst-0.600.1/debian/patches/0005-Don-t-fail-if-we-can-t-set-locale-debian-bug-697864.patch
1970-01-01 01:00:00.0 +0100
+++
virtinst-0.600.1/debian/patches/0005-Don-t-fail-if-we-can-t-set-locale-debian-bug-697864.patch
2013-01-15 16:20:37.0 +0100
@@ -0,0 +1,27 @@
+From: Cole Robinson
+Date: Mon, 14 Jan 2013 15:06:14 -0500
+Subject: Don't fail if we can't set locale (debian bug #697864)
+
+---
+ virtinst/cli.py |8 +++-
+ 1 file changed, 7 insertions(+), 1 deletion(-)
+
+diff --git a/virtinst/cli.py b/virtinst/cli.py
+index 80e1fcf..e739b12 100644
+--- a/virtinst/cli.py
b/virtinst/cli.py
+@@ -144,7 +144,13 @@ def setupParser(usage=None):
+ return parser
+
+ def setupGettext():
+-locale.setlocale(locale.LC_ALL, '')
++try:
++locale.setlocale(locale.LC_ALL, '')
++except locale.Error, e:
++print >> sys.stderr, "warning: failed to set locale, defaulting to C"
++os.environ['LC_ALL'] = 'C'
++locale.setlocale(locale.LC_ALL, 'C')
++
+ gettext.bindtextdomain("virtinst")
+ gettext.install("virtinst")
+
diff -Nru virtinst-0.600.1/debian/patches/series
virtinst-0.600.1/debian/patches/series
--- virtinst-0.600.1/debian/patches/series 2012-04-16 14:23:13.0
+0200
+++ virtinst-0.600.1/debian/patches/series 2013-01-15 16:20:37.0
+0100
@@ -2,3 +2,4 @@
0002-Fix-path-to-pygrub.patch
0003-Fix-path-to-keyboard-configuration.patch
0004-Fix-location-of-Debian-daily-builds.patch
+0005-Don-t-fail-if-we-can-t-set-locale-debian-bug-697864.patch
Re: Update virinst via wheezy-proposed-updates
On Tue, 2013-01-15 at 17:18 +0100, Guido Günther wrote: > the tools from virtinst will fail to do anything useful with a bad > locale sitting. The patch is simple and cherry-picked from upstream. > Would this be a case to be fixed via w-p-u? Sid already has a newer > upstream version. Debdiff is attached. According to the version information for the bug in question (#697864), it's not fixed in unstable yet - is that correct? Regards, Adam -- To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/1358542389.12995.19.ca...@jacala.jungle.funky-badger.org
Re: Update virinst via wheezy-proposed-updates
On Fri, Jan 18, 2013 at 08:53:09PM +, Adam D. Barratt wrote: > On Tue, 2013-01-15 at 17:18 +0100, Guido Günther wrote: > > the tools from virtinst will fail to do anything useful with a bad > > locale sitting. The patch is simple and cherry-picked from upstream. > > Would this be a case to be fixed via w-p-u? Sid already has a newer > > upstream version. Debdiff is attached. > > According to the version information for the bug in question (#697864), > it's not fixed in unstable yet - is that correct? Doh - fixed in sid now. Sorry for the omission. -- Guido -- To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/20130119125143.ga4...@bogon.sigxcpu.org
Re: Update virinst via wheezy-proposed-updates
On 19.01.2013 12:51, Guido Günther wrote: On Fri, Jan 18, 2013 at 08:53:09PM +, Adam D. Barratt wrote: On Tue, 2013-01-15 at 17:18 +0100, Guido Günther wrote: > the tools from virtinst will fail to do anything useful with a bad > locale sitting. The patch is simple and cherry-picked from upstream. > Would this be a case to be fixed via w-p-u? Sid already has a newer > upstream version. Debdiff is attached. According to the version information for the bug in question (#697864), it's not fixed in unstable yet - is that correct? Doh - fixed in sid now. Sorry for the omission. Thanks. +virtinst (0.600.1-3) wheezy-proposed-updates; urgency=low + + * [3723835] Don't fail if we can't set locale (debian bug #697864) +Thanks to Cole Robinson (Closes: #697864) We'd generally prefer "0.600.1-2+deb7u1" as a version number here, although -3 would work as long as there's never been such a version in Debian. Please go ahead; thanks. Regards, Adam -- To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/6d1913599571ac865e4fa16286e84...@mail.adsl.funky-badger.org
Re: Update virinst via wheezy-proposed-updates
On Fri, Jan 25, 2013 at 09:01:51AM +, Adam D. Barratt wrote: > On 19.01.2013 12:51, Guido Günther wrote: > >On Fri, Jan 18, 2013 at 08:53:09PM +, Adam D. Barratt wrote: > >>On Tue, 2013-01-15 at 17:18 +0100, Guido Günther wrote: > >>> the tools from virtinst will fail to do anything useful with a bad > >>> locale sitting. The patch is simple and cherry-picked from > >>upstream. > >>> Would this be a case to be fixed via w-p-u? Sid already has a > >>newer > >>> upstream version. Debdiff is attached. > >> > >>According to the version information for the bug in question > >>(#697864), > >>it's not fixed in unstable yet - is that correct? > > > >Doh - fixed in sid now. Sorry for the omission. > > Thanks. > > +virtinst (0.600.1-3) wheezy-proposed-updates; urgency=low > + > + * [3723835] Don't fail if we can't set locale (debian bug #697864) > +Thanks to Cole Robinson (Closes: #697864) > > We'd generally prefer "0.600.1-2+deb7u1" as a version number here, > although -3 would work as long as there's never been such a version > in Debian. No there wasn't. > > Please go ahead; thanks. Uploaded. Thanks, -- Guido > > Regards, > > Adam > -- To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/20130125204548.ga12...@bogon.sigxcpu.org
Re: Update virinst via wheezy-proposed-updates
On Fri, 2013-01-25 at 21:45 +0100, Guido Günther wrote: > On Fri, Jan 25, 2013 at 09:01:51AM +, Adam D. Barratt wrote: > > On 19.01.2013 12:51, Guido Günther wrote: > > >On Fri, Jan 18, 2013 at 08:53:09PM +, Adam D. Barratt wrote: > > >>On Tue, 2013-01-15 at 17:18 +0100, Guido Günther wrote: > > >>> the tools from virtinst will fail to do anything useful with a bad > > >>> locale sitting. The patch is simple and cherry-picked from > > >>upstream. [...] > > Please go ahead; thanks. > > Uploaded. Thanks, Unblocked. Regards, Adam -- To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/1359148328.28246.10.ca...@jacala.jungle.funky-badger.org
