On Nov 14, 2007 9:29 AM, Dan McGee <dpmcgee@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