[pacman-dev] [PATCH] Change -z|--showsize flag to ShowSize pacman.conf option

Dan McGee dpmcgee at gmail.com
Fri Jun 1 11:07:59 EDT 2007

On 6/1/07, Dan McGee <dpmcgee at gmail.com> wrote:
> On 5/14/07, Nathan Jones <nathanj at insightbb.com> wrote:
> > Also, some duplicate printf lines related to the ShowSize option were
> > cleaned up.
> >
> > Dan, I created this diff against your tree not the official one, if it
> > makes a difference when applying.
> >
> > Signed-off-by: Nathan Jones <nathanj at insightbb.com>
> Well this got lost in the email box a while back. Applying now and
> checking, it'll eventually get pushed to the main tree.
> -Dan

OK, pushed to my working branch and all looks good to go. While doing
this, I noticed how awful our pacman.conf parsing really is. We
*silently* fail on any unrecognized option (ex. ShowSize with a pacman
that doesn't support it), and that makes it terribly hard for users to
debug without using the --debug flag. Even then it isn't easy.

Aaron and I both want to move this config file parsing to the front
end; any objections to this? It will be quite a change, but we can
make it much more robust while moving it. Is anyone familiar with how
config file parsing is done by other programs? It seems like a very
manual process currently, and a malformed config file brings pacman
down. I also think we should try to continue when we encounter an
unrecognized option.


More information about the pacman-dev mailing list