| From: | Hugh Ranalli <hugh(at)whtc(dot)ca> |
|---|---|
| To: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
| Cc: | thomas(dot)munro(at)enterprisedb(dot)com, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Daniel Verite <daniel(at)manitou-mail(dot)org>, pgsql-bugs(at)lists(dot)postgresql(dot)org |
| Subject: | Re: BUG #15548: Unaccent does not remove combining diacritical characters |
| Date: | 2019-01-02 17:58:03 |
| Message-ID: | CAAhbUMOG_S4DLQSDw3Jbqf8vipuvCAfxvH7T8+W2H0jFA-2K-g@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs pgsql-hackers |
On Wed, 2 Jan 2019 at 12:41, Peter Eisentraut <
peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:
> On 20/12/2018 23:39, Hugh Ranalli wrote:
> > 01 - Updates generate_unaccent_rules.py to be Python 2 and 3 compatible.
>
> My opinion is that we should just convert the whole thing to Python 3
> and be done. This script is only run rarely, on a developer's machine,
> so it's not unreasonable to expect Python 3 to be available.
>
Well, this is definitely an edge case, but I am actually running the
patched script from a complex application installer running a
custom-compiled version of Python 2.7. The installer runs under the same
Python instance as the application. I certainly could invoke Python 3 to
run this script, it's just a little more work, so I'm happy to go with the
team's decision. Just let me know.
Hugh
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2019-01-02 19:22:29 | Re: Is temporary functions feature official/supported? Found some issues with it. |
| Previous Message | Peter Eisentraut | 2019-01-02 17:41:40 | Re: BUG #15548: Unaccent does not remove combining diacritical characters |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Mark | 2019-01-02 18:04:19 | Re: Query planner / Analyse statistics bad estimate rows=1 with maximum statistics 10000 on PostgreSQL 10.2 |
| Previous Message | Peter Eisentraut | 2019-01-02 17:41:40 | Re: BUG #15548: Unaccent does not remove combining diacritical characters |