[pacman-dev] [PATCH] Add VerboseUpgradeInfo Option

Xavier shiningxc at gmail.com
Mon Mar 10 20:50:25 EDT 2008


On Mon, Mar 10, 2008 at 07:38:30PM -0500, Dan McGee wrote:
> 
> Wowzers! This is kind of cool on one hand, but pretty darn overkill
> for at least my typical use pattern. This would be awesome for
> something like a curses frontend though, I would say that. Anyone want
> to start developing? :)
> 
> If I was to critique it, I would agree that the column order should go
> old version, new version, size. But to be honest, I'm not too big on
> this patch so I probably wouldn't take it, unless there are a lot of
> people that have been looking for something like this. The addition of
> 140 lines of code doesn't exactly make me smile with joy, as I already
> think display_targets() is a rather gross function.
> 

That's exactly what I was thinking. The output looks cool, but adding 140
lines to that part I already find messy isn't cool.
If it was up to me, I would maybe just do one column based output, because I
find the oldver and size informations useful.




More information about the pacman-dev mailing list