On 18/11/15 06:23, Heiko Schlittermann wrote:
> Jeremy Harris <[email protected]> (Di 17 Nov 2015 15:02:25 CET):
>> On 17/11/15 13:02, Heiko Schlittermann wrote:
>>>     exim -C /dev/null -bP primary_hostname
>>>     exim -C <(echo primary_hostname=foo) -bP primary_hostname
>>>
>>> With the default being tls_advertise_hosts = *, this above line issue
>>> all that ugly warnings about missing tls_* support files.
>>
>> How about suppressing the tls warnings when -bP is used?
> 
> Ok, I've implemented this, using the existing 'checking' variable in exim.c.
> Now I set this in -bP and -brw too.

not -brt ?

and...
> 
> But, ~ exim.c:3490
> 
>     if …
>      (
>      list_options &&
>      (checking || smtp_input || extract_recipients ||
>        filter_test != FTEST_NONE || bi_option)
>      ) ||
>      … then issue "incompatible options"
> 
> Are there security reasons, or may I change this to
> 
>     if …
>      (
>      list_options &&
> -    (checking || smtp_input || extract_recipients ||
> +    (smtp_input || extract_recipients ||
>        filter_test != FTEST_NONE || bi_option)
>      ) ||
>      … then issue "incompatible options"

no, that would not be good; the combination "-be -bP" would
then be allowed, and it makes no sense.

I think you need (list_options || list_config) instead there,
and don't set "checking" for -bP.

-- 
Cheers,
  Jeremy


-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##

Reply via email to