This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] localedef: Add --localedir and --archive-file options (Bug 19130)


On 10/15/2015 07:50 AM, Joseph Myers wrote:
> On Wed, 14 Oct 2015, Carlos O'Donell wrote:
> 
>> (2) Add a `--localedir=PATH` option per BZ #19130.
> 
> It's possible this option name may be confusing.  Cf. bug 14259 discussing 
> the meaning of the configure --localedir option.  (I think that bug is 
> only a documentation issue - install.texi needs to document exactly what 
> the configure option means, without any change to what that option does.)  

It is my opinion that Andreas Schwab was wrong in comment #7:
https://sourceware.org/bugzilla/show_bug.cgi?id=14259#c7

The location of the locale archive, and binary compiled locales is localedir.

The location of the gettext message catalogs is msgcatdir.

The only way to change the location of msgcatdir is to adjust datadir.

Therefore I think the option is correct, and I'll submitted a patch
to fix bug 14259.

> Do we want the localedef option to have a meaning completely different 
> from that configure option, and so possibly add to that confusion?

No. Both options are and do mean the same thing. Unless someone can point
out something I did wrong.

> Also, a new feature like this should have its own NEWS entry, not just a 
> number in the list of fixed bugs (even if by the time of the next release 
> we have a system to generate a list with bug descriptions included).

Agreed. I will post v2 and add a NEWS entry.

Cheers,
Carlos.



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]