[pacman-dev] Potential bug in alpm-hooks; fix provided

Allan McRae allan at archlinux.org
Tue Jan 3 03:58:34 UTC 2017


On 03/01/17 04:27, Andrew Gregory wrote:
> On 01/02/17 at 07:20pm, git at stefan-klinger.de 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`.

I think an error message is too strict and a debug message is not
enough.   So I will accept a warning message being printed.

A


More information about the pacman-dev mailing list