diff options
author | Eli Schwartz <eschwartz@archlinux.org> | 2020-05-10 06:32:39 +0200 |
---|---|---|
committer | Allan McRae <allan@archlinux.org> | 2020-05-11 04:19:40 +0200 |
commit | 8e769ddb8a59a9fbacf4614283d2fb519f022386 (patch) | |
tree | 02f0b5ddc6740d4b88df7709dff5d4939f33758b /src/common | |
parent | 7423b166047ff168ef337818958bf86ed1d42a0a (diff) | |
download | pacman-8e769ddb8a59a9fbacf4614283d2fb519f022386.tar.gz pacman-8e769ddb8a59a9fbacf4614283d2fb519f022386.tar.xz |
Log invalid conf settings as an error
This is not a warning, _parse_options() returns failure without even
parsing further lines and the attempted pacman/pacman-conf program
execution immediately aborts. Warnings are for when e.g. later on if we
don't recognize a setting at all, we skip over it and have enough
confidence in this to continue executing the program.
The current implementation results in pacman-conf aborting with:
warning: config file /etc/pacman.conf, line 60: invalid value for 'ParallelDownloads' : '2.5'
error parsing '/etc/pacman.conf'
or pacman -Syu aborting with the entirely more cryptic:
warning: config file /etc/pacman.conf, line 59: invalid value for 'ParallelDownloads' : '2.5'
and this isn't just a problem for the newly added ParallelDownloads
setting, either, you could get the same problem if you specified a
broken XferCommand, but that's harder as it's more accepting of input
and you probably don't hit this except with unbalanced quotes.
Signed-off-by: Eli Schwartz <eschwartz@archlinux.org>
Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'src/common')
0 files changed, 0 insertions, 0 deletions