[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#172439: Bug#156386: patch to fix this bug 172439 - nope



At Sat, 21 Dec 2002 21:55:05 +0900,
GOTO Masanori wrote:
> At Sat, 21 Dec 2002 14:56:34 +0900,
> Masato Taruishi wrote:
> > At 20 Dec 2002 16:05:06 +0100,
> > Nagy Peter.1 wrote:
> > 
> > > Actually, your patch doesn't fix bug #172439. The problem is in the
> > > postinst script. If /etc/locale.gen exists, bug XXX GENERATED XXX is not
> > > present in it, postint won't touch /etc/locale.gen.
> > 
> > Oh, sorry. I misunderstood that these reports are the same bugs. 
> 
> Agh, Jeff remove the Closes: from changelog...
>
> > > I think this whole locale.postinst magic is quite confusing and should
> > > be removed.
> > 
> > I don't think so. I guess it's better config simply
> > overwrote /etc/locale.gen no-matter-what or prompt in config
> > that XXGENERATEDXX is not found or something like that.
> 
> "Prompting in config" is good idea. locale.gen is in /etc/,
> so overwriting /etc/locale.gen is not fine.

I discussed with Masato Taruishi on IRC, and he suggested and taught
what was the problem (thanks Masato!). I summarize this issue:

* This bug is caused by not only locales but also localeconf.
* locales makes /etc/locale.gen with XXGENERATEDXX.
* localeconf also makes /etc/locale.gen with another way
  (### BEGIN DEBCONF SECTION FOR localeconf).
* Thus if you used localeconf, then you use locales debconf,
  you can't generate /etc/locale.gen properly.

To resolve this bug, some idea is come up.

(1) localeconf simply put XXGENERATEDXX into /etc/locale.gen, or 
    something to do.
(2) locales simply overwrites /etc/locale.gen if XXGENERATEDXX is not
    present. 
(3) locales.config grep-s XXGENERATEDXX is present in /etc/locales.gen.
    If there is no XXGENERATEDXX text, locales warns.

I think (1) is the best solution.  This bug is the conflict between
locales and localeconf. /etc/locale.gen is originally locales package
file, but localeconf overwrites unsafe method.
I would like to reassign this bug to localeconf. Is this OK?

Regards,
-- gotom



Reply to: