Re: parameter append syntax (was: milter_header_checks, pcre, chroot)

2022-06-26 Thread raf
On Sun, Jun 26, 2022 at 07:45:47AM -0400, Wietse Venema wrote: > raf: > > Also, is .= the best notation? Would += be better? > > https://marc.info/?l=postfix-users=164779562215790=2 > > Wietse Of course. cheers, raf

Re: parameter append syntax (was: milter_header_checks, pcre, chroot)

2022-06-26 Thread Wietse Venema
raf: > Also, is .= the best notation? Would += be better? https://marc.info/?l=postfix-users=164779562215790=2 Wietse

Re: parameter append syntax (was: milter_header_checks, pcre, chroot)

2022-06-25 Thread raf
On Sat, Jun 25, 2022 at 09:08:30PM -0400, Wietse Venema wrote: > raf: > > If .= can reliably distinguish between being applied to > > a list or scalar parameter, maybe it could automatically > > include a leading space when adding to lists. > > Unfortunately, the main.cf parser does not know

Re: parameter append syntax (was: milter_header_checks, pcre, chroot)

2022-06-25 Thread Wietse Venema
raf: > If .= can reliably distinguish between being applied to > a list or scalar parameter, maybe it could automatically > include a leading space when adding to lists. Unfortunately, the main.cf parser does not know if a parameter value is used as a list such as export_environment = TZ

Re: parameter append syntax (was: milter_header_checks, pcre, chroot)

2022-06-25 Thread Viktor Dukhovni
On Sat, Jun 25, 2022 at 11:07:20AM -0400, Wietse Venema wrote: > It can be fixed by requiring a comma when appending to a list: > > export_environment .= , { NAME = value } Indeed something like this would have to be the main.cf syntax, unless there are instead two related append operators:

Re: parameter append syntax (was: milter_header_checks, pcre, chroot)

2022-06-25 Thread raf
On Sat, Jun 25, 2022 at 11:07:20AM -0400, Wietse Venema wrote: > Wietse: > >I'm looking into adding "name .= value" support but this is tricky > >because it has to work not only in main.cf but also in master.cf > >(-o name.=value). > > That idea originated in the context op adding an 'extra'

parameter append syntax (was: milter_header_checks, pcre, chroot)

2022-06-25 Thread Wietse Venema
Wietse: >I'm looking into adding "name .= value" support but this is tricky >because it has to work not only in main.cf but also in master.cf >(-o name.=value). That idea originated in the context op adding an 'extra' to the default value of export_environment, or addind map that I forgot to