Re: switch /usr/share/locale to different layout

2013-05-31 Thread Stefan Sperling
On Thu, May 30, 2013 at 10:05:45PM +0200, Stefan Sperling wrote: > Does anyone see problems with this plan? Haven't heard any objections. Updated diff which contains small tweaks and patches an mtree file to no longer create directories for arbitrary locales, some of which we don't even support.

Re: switch /usr/share/locale to different layout

2013-05-31 Thread Stefan Sperling
On Fri, May 31, 2013 at 07:30:20AM -0500, Vladimir Támara Patiño wrote: > I agree with Andres, that a structure should be required for the locale > name, I guess fine starting with C, POSIX, C.encoding, > POSIX.encoding, ll.encoding or ll_CC.encoding, with CC according to > ISO 3166-1 alfa-2, > and

Re: switch /usr/share/locale to different layout

2013-05-31 Thread Vladimir Támara Patiño
On Thu, May 30, 2013 at 10:05:45PM +0200, Stefan Sperling wrote: Does anyone see problems with this plan? Sounds good. I agree with Andres, that a structure should be required for the locale name, I guess fine starting with C, POSIX, C.encoding, POSIX.encoding, ll.encoding or ll_CC.encodin

Re: switch /usr/share/locale to different layout

2013-05-31 Thread Stefan Sperling
On Fri, May 31, 2013 at 01:05:08AM -0430, Andres Perera wrote: > Besides being redundant, it's generally better to bail on open() error > instead of checking if the length is less than PATH_MAX in > anticipation. > > [Unrelated to your patch, but since we are on the subject of path > checks in set

Re: switch /usr/share/locale to different layout

2013-05-30 Thread Andres Perera
On Thu, May 30, 2013 at 11:49 PM, Stefan Sperling wrote: > On Thu, May 30, 2013 at 05:18:48PM -0430, Andres Perera wrote: >> As I mentioned, there's code that expects the prior layout, and that's >> confusing. >> >> on src/lib/libc/locale/setlocale.c, load_locale_sub() : >> >>228 len

Re: switch /usr/share/locale to different layout

2013-05-30 Thread Stefan Sperling
On Thu, May 30, 2013 at 05:18:48PM -0430, Andres Perera wrote: > As I mentioned, there's code that expects the prior layout, and that's > confusing. > > on src/lib/libc/locale/setlocale.c, load_locale_sub() : > >228 len = snprintf(name, sizeof(name), "%s/%s/%s", >229

Re: switch /usr/share/locale to different layout

2013-05-30 Thread Andres Perera
On Thu, May 30, 2013 at 3:35 PM, Stefan Sperling wrote: > I've received several requests for adding new locale names, > both on this list, and off-list, from several people. > > I've been trying to find a way to keep /usr/share/locale reasonably > clean while also allowing people to use their pref

Re: switch /usr/share/locale to different layout

2013-05-30 Thread Antoine Jacoutot
On Thu, May 30, 2013 at 10:05:45PM +0200, Stefan Sperling wrote: > I've received several requests for adding new locale names, > both on this list, and off-list, from several people. > > I've been trying to find a way to keep /usr/share/locale reasonably > clean while also allowing people to use t

switch /usr/share/locale to different layout

2013-05-30 Thread Stefan Sperling
I've received several requests for adding new locale names, both on this list, and off-list, from several people. I've been trying to find a way to keep /usr/share/locale reasonably clean while also allowing people to use their preferred locale names. Currently, the list of supported locale names