[pacman-dev] [PATCH] Add -s --sign and -v --verify to repo-add usage message.

[Dan McGee]

On Fri, Mar 25, 2011 at 6:16 PM, Ray Kohler <ataraxia937 at gmail.com> wrote:
> On Fri, Mar 25, 2011 at 6:54 PM, Ray Kohler <ataraxia937 at gmail.com> wrote:
>> On Fri, Mar 25, 2011 at 6:45 PM, Dan McGee <dpmcgee at gmail.com> wrote:
>>> On Fri, Mar 25, 2011 at 5:38 PM, Allan McRae <allan at archlinux.org> wrote:
>>>> On 26/03/11 08:16, Dan McGee wrote:
>>>>>
>>>>> On Fri, Mar 25, 2011 at 5:14 PM, Ray Kohler<ataraxia937 at gmail.com>  wrote:
>>>>>>
>>>>>> There were not added to the end of the list as that would mean
>>>>>> splitting the single gettext string that contains -f and the
>>>>>> "see the man page" message, and I wanted to do as little string
>>>>>> damage as possible.
>>>>>
>>>>> Do string damage, it is a major release. But more importantly,
>>>>> repo-remove --help is going to show these and we might want to try and
>>>>> avoid that- I think the reason they weren't added here.
>>>>>
>>>>
>>>> I think these options are still relevant to repo-remove.
>>>
>>> -d and -f? If you remove an entry, the whole darn thing goes,
>>> regardless of whether there are deltas or files ...files in there.
>>
>> That's what checking the code bears out to me as well, but I think
>> Allan's point is that -s and -v actually do apply to repo-remove.
>
> How's this for usage output?
>
> ataraxia at asenath:~/code/pacman/scripts $ ./repo-add -h
> repo-add (pacman) 3.5.1-73-g72775-dirty
>
> Usage: repo-add [-d] [-f] [-q] [-s] [-v] <path-to-db> <package|delta> ...
> repo-add will update a package database by reading a package file.
> Multiple packages to add can be specified on the command line.
>
> Options:
>  -d, --delta   generate and add delta for package update
>  -f, --files   update database's file list
>  -q, --quiet   minimize output
>  -s, --sign    sign database with GnuPG after update
>  -v, --verify  verify database's signature before update
>
> See repo-add(8) for more details and descriptions of the available options.
>
> Example:  repo-add /path/to/repo.db.tar.gz pacman-3.0.0.pkg.tar.gz
>
> ataraxia at asenath:~/code/pacman/scripts $ ./repo-remove -h
> repo-remove (pacman) 3.5.1-73-g72775-dirty
>
> Usage: repo-remove [-q] [-s] [-v] <path-to-db> <packagename> ...
>
> repo-remove will update a package database by removing the package name
> specified on the command line from the given repo database. Multiple
> packages to remove can be specified on the command line.
>
> Options:
>  -q, --quiet   minimize output
>  -s, --sign    sign database with GnuPG after update
>  -v, --verify  verify database's signature before update
>
> See repo-remove(8) for more details and descriptions of the available options.
>
> Example:  repo-remove /path/to/repo.db.tar.gz kernel26

At quick glance, looks great to me.

The manpage might be the next thing to tackle- find a good way to
delineate the two sets of options, as it isn't that clear right now.
-v is also missing it appears.

-Dan