FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > ArchLinux > ArchLinux Pacman Development

 
 
LinkBack Thread Tools
 
Old 06-20-2011, 06:15 PM
Dave Reisner
 
Default repo-add.8.txt: document valid DB file extensions

Signed-off-by: Dave Reisner <d@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'. The file does not need to
+exist, but all parent directories must exist.
+

Common Options
--------------
--
1.7.5.4
 
Old 06-20-2011, 10:30 PM
Eric Bélanger
 
Default repo-add.8.txt: document valid DB file extensions

On Mon, Jun 20, 2011 at 2:15 PM, Dave Reisner <d@falconindy.com> wrote:
> Signed-off-by: Dave Reisner <d@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.



The file does not need to
> +exist, but all parent directories must exist.
> +
>
> *Common Options
> *--------------
> --
> 1.7.5.4
>
>
>
 
Old 06-21-2011, 04:38 PM
Dan McGee
 
Default repo-add.8.txt: document valid DB file extensions

On Mon, Jun 20, 2011 at 6:08 PM, Dave Reisner <d@falconindy.com> wrote:
> 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@falconindy.com> wrote:
>> > Signed-off-by: Dave Reisner <d@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).

The current text on your working branch and my comments:

+A package database is a tar file, optionally compressed, with one of the
+extensions: ``.tar', ``.tar.gz', ``.tar.bz2', or ``.tar.xz'. The file does
+not need to exist, but all parent directories must exist.
+

A package database is a tar file, optionally compressed. The valid
extensions are...

Drop "valid extensions" so it matches the text I know I'd go searching
for in the error message. I'm lazy reading manpages.

This might also get adjusted a bit depending on whether we enforce
'.db' and '.files' suffixes (before the tar suffix bits).

-Dan
 
Old 06-23-2011, 01:00 AM
Dave Reisner
 
Default repo-add.8.txt: document valid DB file extensions

Signed-off-by: Dave Reisner <d@falconindy.com>
---
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..36855af 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 tar file, optionally compressed. Valid extensions are:
+``.db.tar', ``.db.tar.gz', ``.db.tar.bz2', or ``.db.tar.xz'. The file does
+not need to exist, but all parent directories must exist.
+

Common Options
--------------
--
1.7.5.4
 
Old 06-24-2011, 04:10 AM
Dan McGee
 
Default repo-add.8.txt: document valid DB file extensions

On Wed, Jun 22, 2011 at 8:00 PM, Dave Reisner <d@falconindy.com> wrote:
> Signed-off-by: Dave Reisner <d@falconindy.com>
> ---
> *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..36855af 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 tar file, optionally compressed. Valid extensions are:
> +``.db.tar', ``.db.tar.gz', ``.db.tar.bz2', or ``.db.tar.xz'. The file does
> +not need to exist, but all parent directories must exist.
> +

.files needs to be included somewhere in this.

>
> *Common Options
> *--------------
> --
> 1.7.5.4
>
>
>
 

Thread Tools




All times are GMT. The time now is 03:50 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright ©2007 - 2008, www.linux-archive.org