# [PATCH] Documentation/fast-import: clarify summary for `feature` command

2 messages from 2013-08-23 to 2013-08-25. Participants: Matthieu Moy, Jonathan Nieder.
Thread: https://gitlist.dev/t/34748

## Matthieu Moy, 2013-08-23 08:29

Subject: [PATCH] Documentation/fast-import: clarify summary for `feature` command
Message-ID: <1377246560-19847-1-git-send-email-Matthieu.Moy@imag.fr>
URL: https://gitlist.dev/e/1377246560-19847-1-git-send-email-Matthieu.Moy%40imag.fr

```
In most cases, "feature <foo>" does not just require that the feature
exists, but also changes the behavior to enable it.

Cases where the feature is only requested like cat-blob, notes or ls are
clearly documented below.

Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
---
 Documentation/git-fast-import.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt
index bf1a02a..4a9cc74 100644
--- a/Documentation/git-fast-import.txt
+++ b/Documentation/git-fast-import.txt
@@ -361,8 +361,8 @@ and control the current import process.  More detailed discussion
 	`--cat-blob-fd` or `stdout` if unspecified.
 
 `feature`::
-	Require that fast-import supports the specified feature, or
-	abort if it does not.
+	Enable the specified feature. This requires that fast-import
+	supports the specified feature, and aborts if it does not.
 
 `option`::
 	Specify any of the options listed under OPTIONS that do not
-- 
1.8.4.rc4.4.g57d5d3d

```

## Jonathan Nieder, 2013-08-25 05:57

Subject: Re: [PATCH] Documentation/fast-import: clarify summary for `feature` command
Message-ID: <20130825055737.GO2882@elie.Belkin>
URL: https://gitlist.dev/e/20130825055737.GO2882%40elie.Belkin
In-Reply-To: <1377246560-19847-1-git-send-email-Matthieu.Moy@imag.fr>

```
Matthieu Moy wrote:

> In most cases, "feature <foo>" does not just require that the feature
> exists, but also changes the behavior to enable it.

For what it's worth,

Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>

```
