[pacman-dev] [PATCH] Add ShowOldVersion Option

Aaron Griffin aaronmgriffin at gmail.com
Wed Nov 14 11:38:56 EST 2007


On Nov 14, 2007 9:29 AM, Dan McGee <dpmcgee at gmail.com> wrote:
> I guess I'll chime in here since I haven't yet and I'm the one that
> merges your patch anyway. If the code looks OK (haven't looked yet),
> then I will merge this for now because it is optional, so it will not
> affect anyone that doesn't want to see this. We can worry about
> changing the format later.
>
> Personally, I like less clutter, and moving each package to its own
> line when I am installing 60 packages would make my terminal scroll,
> which is not ideal. But thats just me. However, I would not be against
> some option to rectify this. Of course, we are going to reach the
> point soon where the output of this function is getting quite bloated,
> and will need a rewrite in order to keep it clean and well-organized
> with all these new output options.
>
> Aaron, any thoughts?

Heh, my only thoughts were "man, that sure is a lot of info crammed
into a paragraph like that".

But I see what you're saying about this function getting out of
control with ShowSize, and this option, and maybe more later.

I could probably throw some refactoring at display_targets to break it
apart a little /me shrugs




More information about the pacman-dev mailing list