Re: New "-b slim" option in 2019b zic: should we turn that on? - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: New "-b slim" option in 2019b zic: should we turn that on?
Date
Msg-id 56a6d366-a3a4-2e57-62a7-6be9eec0e4ea@2ndQuadrant.com
Whole thread Raw
In response to Re: New "-b slim" option in 2019b zic: should we turn that on?  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: New "-b slim" option in 2019b zic: should we turn that on?
List pgsql-hackers
On 10/5/19 6:33 PM, Tom Lane wrote:
> Andrew Dunstan <andrew.dunstan@2ndquadrant.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.


I don't think that's going to work very well with a buildfarm member,
where there's no convenient way to set it.


But it turns out there are bigger problems with what I'm doing, anyway,
so let's leave sleeping dogs lie for now.


cheers


andrew


-- 
Andrew Dunstan                https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services




pgsql-hackers by date:

Previous
From: Noah Misch
Date:
Subject: Re: expressive test macros (was: Report test_atomic_ops() failuresconsistently, via macros)
Next
From: Tom Lane
Date:
Subject: Re: New "-b slim" option in 2019b zic: should we turn that on?