| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> |
| Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: New "-b slim" option in 2019b zic: should we turn that on? |
| Date: | 2019-10-05 22:33:13 |
| Message-ID: | 23096.1570314793@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> writes:
> I've just run into an issue with this (commit a1207910968). The makefile
> now assumes that zic has this switch. But I was attempting to get around
> an issue on msys2 by using its zic, (ZIC=/usr/bin/zic configure ...). It
> crashes on the floor because it doesn't know about "-b slim". I think we
> probably need a way to turn this off.
I had contemplated injecting the -b switch via
# any custom options you might want to pass to zic while installing data files
-ZIC_OPTIONS =
+ZIC_OPTIONS = -b slim
which would allow overriding it by defining the ZIC_OPTIONS macro.
Does that seem appropriate? I didn't do it because I worried about
interference with existing uses of ZIC_OPTIONS ... but who knows
whether there are any.
BTW, building with old versions of zic is not guaranteed to work anyway.
They do tend to wait a year or two before they start to use new zic
features in the timezone data files, but they don't wait indefinitely.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alvaro Herrera | 2019-10-05 22:43:33 | parallel restore sometimes fails for FKs to partitioned tables |
| Previous Message | Andrew Dunstan | 2019-10-05 21:43:49 | Re: New "-b slim" option in 2019b zic: should we turn that on? |