| From: | Peter Eisentraut <peter_e(at)gmx(dot)net> | 
|---|---|
| To: | Jeff Davis <pgsql(at)j-davis(dot)com> | 
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgresql(dot)org | 
| Subject: | Re: (patch) regression diffs on collate.linux.utf8 test | 
| Date: | 2011-10-18 12:16:04 | 
| Message-ID: | 1318940164.8007.2.camel@fsopti579.F-Secure.com | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-hackers | 
On tis, 2011-10-18 at 01:07 -0700, Jeff Davis wrote:
> On Sun, 2011-10-16 at 16:00 -0400, Tom Lane wrote:
> > Jeff Davis <pgsql(at)j-davis(dot)com> writes:
> > > On master, I see a minor test error (at least on my machine) as well as
> > > a diff. Patch attached.
> > 
> > Hmm, yeah, I forgot to fix this regression test when I added that DETAIL
> > line.  However, I don't see the need for fooling with the lc_time value?
> > 
> > 			regards, tom lane
> 
> Here is the diff that I'm seeing on master right now with:
> 
>   make -s check EXTRA_TESTS=collate.linux.utf8
> 
> If I qualify it as "tr_TR.UTF-8" it works. Perhaps I have something
> misconfigured on my system (Ubuntu 11.10)? I just installed:
>   language-pack-de
>   language-pack-tr
>   language-pack-sv
> 
> in an attempt to make the test work, and it works all except for that
> lc_time settng.
I think the language-pack packages have nothing to do with it; they only
supply translations.
Possibly, things are set up so that only UTF-8 locales are installed by
default.  Since the collate.linux.utf8 requires a UTF-8 environment, it
seems reasonable to use the tr_TR.UTF-8 locale for LC_TIME, instead of
requiring an unrelated (ISO-8859-9) locale to be installed.  So I think
the change you propose is reasonable.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2011-10-18 12:24:24 | Re: new compiler warnings | 
| Previous Message | Tom Lane | 2011-10-18 12:13:11 | Re: (patch) regression diffs on collate.linux.utf8 test |