On Fri, Mar 25, 2011 at 6:16 PM, Ray Kohler <[email protected]> wrote: > On Fri, Mar 25, 2011 at 6:54 PM, Ray Kohler <[email protected]> wrote: >> On Fri, Mar 25, 2011 at 6:45 PM, Dan McGee <[email protected]> wrote: >>> On Fri, Mar 25, 2011 at 5:38 PM, Allan McRae <[email protected]> wrote: >>>> On 26/03/11 08:16, Dan McGee wrote: >>>>> >>>>> On Fri, Mar 25, 2011 at 5:14 PM, Ray Kohler<[email protected]> 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@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@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
