Re: [PATCH v4] git-contacts: allow inputting patch via stdin

[Junio C Hamano <gitster@pobox.com>]

Brigham Campbell <me@brighamcampbell.com> writes:

> Make git-contacts accept patch contents via stdin for better
> interoperability with other utilities. Read from stdin when the user
> passes `-` at least once:
>
> $ git contacts - <patch

OK.

> Even before this patch, git-contacts parses files first, then git
> rev-lists second, regardless of their order in argv. If we instead want
> git-contacts to parse arguments in the same order that they're passed,
> that's a change which should occur in a separate patch.

Fair enough.

We would want an update to contrib/contacts/git-contacts.adoc as
well.  Here is my attempt.  I realize that the new text implies that
we read only one patch from the standard input, but I do not know
offhand whether scan_patches() successfully reads multiple patches
concatenated together into a single stream.  If it does, we may want
to say "read patch file(s) from" instead.

The original uses <rev> in <angle brackets> to clarify that it is a
placeholder, so the added text does the same for <patch>.  The
original text does not say what the command expects of the "patch
files" it mentions, which may be worth improving and when it happens
we would say <patch> in the description to refer to one argument on
the command line.  But I left it outside the scope of this message.



 contrib/contacts/git-contacts.adoc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git c/contrib/contacts/git-contacts.adoc w/contrib/contacts/git-contacts.adoc
index dd914d1261..6473e878e0 100644
--- c/contrib/contacts/git-contacts.adoc
+++ w/contrib/contacts/git-contacts.adoc
@@ -24,7 +24,8 @@ Input consists of one or more patch files or revision arguments.  A revision
 argument can be a range or a single `<rev>` which is interpreted as
 `<rev>..HEAD`, thus the same revision arguments are accepted as for
 linkgit:git-format-patch[1]. Patch files and revision arguments can be combined
-in the same invocation.
+in the same invocation.  A single dash `'-'` character in place of a <patch>
+tells the command to read a patch from the standard input.  
 
 This command can be useful for determining the list of people with whom to
 discuss proposed changes, or for finding the list of recipients to Cc: when