[pacman-dev] [PATCH] repo-add.8.txt: document valid DB file extensions

[Dave Reisner]

On Mon, Jun 20, 2011 at 06:30:29PM -0400, Eric Bélanger wrote:
> On Mon, Jun 20, 2011 at 2:15 PM, Dave Reisner <d at falconindy.com> wrote:
> > Signed-off-by: Dave Reisner <d at falconindy.com>
> > ---
> > This isn't documented anywhere except in the code, and every few months when I
> > end up having to test something that requires my own repo, it takes me 10
> > minutes to figure out wtf a valid file extension is.
> >
> > d
> >
> >  doc/repo-add.8.txt |    4 ++++
> >  1 files changed, 4 insertions(+), 0 deletions(-)
> >
> > diff --git a/doc/repo-add.8.txt b/doc/repo-add.8.txt
> > index 0196882..c4f42a2 100644
> > --- a/doc/repo-add.8.txt
> > +++ b/doc/repo-add.8.txt
> > @@ -29,6 +29,10 @@ command line.
> >  delta specified on the command line. Multiple packages and/or delta to remove
> >  can be specified on the command line.
> >
> > +A package database is a path a compressed tar file with one of the extensions:
> > +``.tar'', ``.tar.gz'', ``.tar.bz2'', or ``.tar.xz''.
> 
> I am not a native English speaker but this sentence looks awkward.  It
> seems like a word (or verb?) is missing in the "a path a compressed
> tar file" part.
> 

A herp a derp.

I'm missing a 'to', perhaps. A database is a path to a tar file...
Should revise this to note that compression is optional (in addition to
the .tar extension).

thanks,
d

> 
> 
>  The file does not need to
> > +exist, but all parent directories must exist.
> > +
> >
> >  Common Options
> >  --------------
> > --
> > 1.7.5.4
> >
> >
> >
>