On 6/1/07, Dan McGee <dpmcgee@gmail.com> wrote:
On 5/14/07, Nathan Jones <nathanj@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@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. -Dan