On 01/02/17 at 07:20pm, [email protected] wrote: > On 2017-Jan-02, Andrew Gregory wrote with possible deletions: > > On 01/02/17 at 05:10pm, Stefan Klinger wrote: > > > I've noticed that exactly one `Exec` directive is allowed in > > > alpm-hooks(5). Instead, multiple occurrences are silently > > > ignored. > > > I think raising an error would be best > > > > Multiple "assignments" are not going away. Triggers can have multiple > > Targets and forcing people to cram them all on one line is a terrible > > idea. This is also how multi-value options in pacman's configuration > > are parsed. > > Ok, if that actually is usual behavior in ini-style files, so be it. > Could we print a warning if information is lost by reassigning Exec? > I have updated my proposal [1] accordingly. > > > > ____________________ > > > [1] https://github.com/s5k6/pacman/tree/fix-hook-exec
As I said before, I have no real opinion one way or the other about issuing a warning/error on overwriting single-value options, so I'll leave that to Allan. If you want your patch to actually be reviewed, though, you need to submit it to this list with `git send-email`. apg
