Git development
 help / color / mirror / Atom feed
From: Matthieu Moy <Matthieu.Moy@imag.fr>
To: Pieter de Bie <pdebie@ai.rug.nl>
Cc: Johannes Schindelin <Johannes.Schindelin@gmx.de>, git@vger.kernel.org
Subject: Re: [PATCH] Documentation/git-gc.txt: change --aggressive description
Date: Wed, 14 May 2008 09:48:35 +0200	[thread overview]
Message-ID: <vpqd4npz5po.fsf@bauges.imag.fr> (raw)
In-Reply-To: <FCA89971-FDBB-4E44-82CE-C0AA854A4667@ai.rug.nl> (Pieter de Bie's message of "Wed\, 14 May 2008 08\:48\:19 +0200")

Pieter de Bie <pdebie@ai.rug.nl> writes:

> How about dropping the last sentence ("Using --aggressive only makes
> sense if you have a badly packed repository, such as created by git- 
> fast-import.") then?

I think informing the user that git-fast-import may require
--aggressive is good. Actually, I like the sentence as it is (the
"such as" makes it clear that it's not the /only/ case where it can be
required).

My 2cts,

-- 
Matthieu

  reply	other threads:[~2008-05-14  7:51 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-13  9:53 [PATCH] Documentation/git-gc.txt: change --aggressive description Pieter de Bie
2008-05-13 10:57 ` Teemu Likonen
2008-05-13 11:08   ` Teemu Likonen
2008-05-13 11:53     ` Johannes Schindelin
2008-05-13 15:03       ` Teemu Likonen
2008-05-13 11:13   ` Pieter de Bie
2008-05-13 11:52 ` Johannes Schindelin
2008-05-14  6:48   ` Pieter de Bie
2008-05-14  7:48     ` Matthieu Moy [this message]
2008-05-24 19:03     ` Pieter de Bie

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=vpqd4npz5po.fsf@bauges.imag.fr \
    --to=matthieu.moy@imag.fr \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    --cc=pdebie@ai.rug.nl \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox