Bug#242006: patch: localize short list of countries

2004-04-19 Thread Joey Hess
Matt Kraai wrote: > I've attached a patch that generates the short translated lists at > runtime. It requires that paste be added to busybox and probably > mishandles names that include commas, but it should still be > smaller than pregenerating the lists. That's a fun hack, but depending on the

Bug#242006: patch: localize short list of countries

2004-04-19 Thread Eugeniy Meshcheryakov
Eugeniy Meshcheryakov wrote: Matt Kraai wrote: I've attached a patch that generates the short translated lists at runtime. It requires that paste be added to busybox and probably mishandles names that include commas, but it should still be smaller than pregenerating the lists. As I see there is n

Bug#242006: patch: localize short list of countries

2004-04-19 Thread Eugeniy Meshcheryakov
Matt Kraai wrote: I've attached a patch that generates the short translated lists at runtime. It requires that paste be added to busybox and probably mishandles names that include commas, but it should still be smaller than pregenerating the lists. As I see there is no commas in templates, they a

Bug#242006: patch: localize short list of countries

2004-04-18 Thread Matt Kraai
On Fri, Apr 16, 2004 at 08:12:39AM +0200, Christian Perrier wrote: > Quoting Eugeniy Meshcheryakov ([EMAIL PROTECTED]): > > Package: countrychooser > > Version: SVN > > Severity: normal > > Followup-For: Bug #242006 > > > >Attached patch adds localized short list support to countrychooser. > >

Bug#242006: patch: localize short list of countries (bugfixes)

2004-04-17 Thread Eugeniy Meshcheryakov
This is the previous patch with the following bugfixes: - fix typo asksedshort->askedshort in debian/postinst (Frans Pop) - escape '$' in debian/postinst (Thiemo Seufer) - remove sed expression 's/$^$/\n/' which does nothing from mkshort Index: countrychooser/debian/postinst ===

Re: Bug#242006: patch: localize short list of countries

2004-04-16 Thread Thiemo Seufer
Frans Pop wrote: [snip] > + if $(echo "$line" | grep -q "^:$1$"); then [snip] > + line=`cat_shortlist $2| grep "$COUNTRYNAME$"` I think such grep expressions need a shell escape in front of the second $ sign. It may work nevertheless in these instances, but... Thiemo -- To UNSU

Re: Bug#242006: patch: localize short list of countries

2004-04-16 Thread Frans Pop
I noticed two small errors in this patch: # Build a short list of supported locales fo should be # Build a short list of supported locales for + asksedshort=0 should be + askedshort=0 They have been fixed in attached ver

Bug#242006: patch: localize short list of countries

2004-04-16 Thread Eugeniy Meshcheryakov
Christian Perrier wrote: This patch did not trigger a lot of reactions.except immediately after its publication. We had a few exchanges on IRC. Joey was concerned about the increased size of countrychooser. Below are the facts: Without the patch: new debian package, version 2.0. size 44080 by

Bug#242006: patch: localize short list of countries

2004-04-15 Thread Christian Perrier
Quoting Eugeniy Meshcheryakov ([EMAIL PROTECTED]): > Package: countrychooser > Version: SVN > Severity: normal > Followup-For: Bug #242006 > >Attached patch adds localized short list support to countrychooser. > Short lists are generated during package build and stored in > /usr/share/shortnam

Bug#242006: patch: localize short list of countries

2004-04-09 Thread Eugeniy Meshcheryakov
Package: countrychooser Version: SVN Severity: normal Followup-For: Bug #242006 Attached patch adds localized short list support to countrychooser. Short lists are generated during package build and stored in /usr/share/shortnames/. -- System Information: Debian Release: testing/unstable APT