[pacman-dev] [PATCH 4/4] CLI args: update --help and manpage

[Dan McGee]

On Sun, Oct 10, 2010 at 8:30 PM, Allan McRae <allan at 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 at 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