On Sun, Oct 10, 2010 at 8:30 PM, Allan McRae <allan@archlinux.org> wrote:
On 11/10/10 05:06, Xavier Chantry wrote:
The three parts (help, manpage and code) are now organized in the same way and much easier to compare : - specific options - install/upgrade options for -S and -U - transaction options for -S -R and -U - global options
After this re-organization, it was easy to update and sync the three components together. Duplication is also avoided.
Signed-off-by: Xavier Chantry<chantry.xavier@gmail.com>
This had actually been on my TODO list for a while so it is good to see it fixed. Overall, I think these look good. One small point:
./src/pacman/pacman -D --help usage: lt-pacman {-D --database} <options> <package(s)> options: --asdeps mark packages as non-explicitly installed --asexplicit mark packages as explicitly installed -b, --dbpath <path> set an alternate database location -r, --root <path> set an alternate installation root -v, --verbose be verbose --arch <arch> set an alternate architecture --cachedir <dir> set an alternate package cache location --config <path> set an alternate configuration file --debug display debug messages --logfile <path> set an alternate log file --noconfirm do not ask for any confirmation
On my TODO list, I had also had removing --arch and --cachedir from -D and -Q as they are useless options there as far as I can tell. And --cachedir from -U.
Slightly related, I wonder if we can shorten the description of "-k, --dbonly" and "--print" so they do not loop on an 80 width terminal. Or maybe add some line wrapping.
I'm not too thrilled abuot this patch for the fact that we lost alpha ordering by shortopt for sync/upgrade/remove options. This is a step backwards in my opinion. I can deal with the dbpath/root/every single op uses these things being at the bottom (and I think a blank line separating them would be good too), but the list seems very mismanaged for the main ops now. I pulled the other two patches in, but not this one. -Dan