Dan McGee wrote:
VerboseTargetInfoList? Ha. Figure out something shorter along those lines and its good with me.
+1 from me if anyone wants to whip this up. It would also keep us down to one option instead of having to worry about combinations of options. Be sure to update the manpage while you are at it. I'll hold off on merging the other patch, so feel free to make it against the current master.
You'll have to figure out the column sizes somehow, not sure how that magic should work. And looking at Justin's original patch, make sure the "New" string is gettext-ed (and you don't make any assumptions about it only being 3 chars). His refactoring was also nice, although this will need to be a slightly different code layout.
-Dan
_______________________________________________ pacman-dev mailing list pacman-dev@archlinux.org http://archlinux.org/mailman/listinfo/pacman-dev
First of all, attached is an update of the original patch with a possible bug correction. Initially I tried to take depends and such into account by checking if the data was NULL in the sync package. Turns out there is a much easier way, check the sync type for PM_SYNC_TYPE_UPGRADE (duh!), if it isn't an upgrade then assume it is new. This cleans up the code a bit, and I believe the other way might have had some instances where a non-upgrade would get through and garbage would be printed. I will try to take a stab at creating the columned output, but like Dan, I do not know exactly how I will get it to work, if someone else has a great idea, feel free to beat me to the implementation of this new output scheme.