{"thread":{"id":"34748","subject":"[PATCH] Documentation/fast-import: clarify summary for `feature` command","startedAt":"2013-08-23T08:29:20Z","lastAt":"2013-08-25T05:57:37Z","messageCount":2,"participants":["Matthieu Moy","Jonathan Nieder"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"225714","messageId":"1377246560-19847-1-git-send-email-Matthieu.Moy@imag.fr","threadId":"34748","inReplyTo":null,"subject":"[PATCH] Documentation/fast-import: clarify summary for `feature` command","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@imag.fr","sentAt":"2013-08-23T08:29:20Z","receivedAt":"2013-08-23T08:29:20Z","isPatch":true,"sender":{"key":"git@matthieu-moy.fr","avatar":"https://avatars.githubusercontent.com/u/14709?v=4"},"body":"In most cases, \"feature <foo>\" does not just require that the feature\nexists, but also changes the behavior to enable it.\n\nCases where the feature is only requested like cat-blob, notes or ls are\nclearly documented below.\n\nSigned-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>\n---\n Documentation/git-fast-import.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt\nindex bf1a02a..4a9cc74 100644\n--- a/Documentation/git-fast-import.txt\n+++ b/Documentation/git-fast-import.txt\n@@ -361,8 +361,8 @@ and control the current import process.  More detailed discussion\n \t`--cat-blob-fd` or `stdout` if unspecified.\n \n `feature`::\n-\tRequire that fast-import supports the specified feature, or\n-\tabort if it does not.\n+\tEnable the specified feature. This requires that fast-import\n+\tsupports the specified feature, and aborts if it does not.\n \n `option`::\n \tSpecify any of the options listed under OPTIONS that do not\n-- \n1.8.4.rc4.4.g57d5d3d\n"},{"id":"225834","messageId":"20130825055737.GO2882@elie.Belkin","threadId":"34748","inReplyTo":"1377246560-19847-1-git-send-email-Matthieu.Moy@imag.fr","subject":"Re: [PATCH] Documentation/fast-import: clarify summary for `feature` command","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2013-08-25T05:57:37Z","receivedAt":"2013-08-25T05:57:37Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Matthieu Moy wrote:\n\n> In most cases, \"feature <foo>\" does not just require that the feature\n> exists, but also changes the behavior to enable it.\n\nFor what it's worth,\n\nReviewed-by: Jonathan Nieder <jrnieder@gmail.com>\n"}]}