From: Junio C Hamano <gitster@pobox•com>
To: Jacob Keller <jacob.e.keller@intel•com>
Cc: git@vger•kernel.org, Johannes Sixt <j6t@kdbg•org>,
Johannes Schindelin <johannes.schindelin@gmx•de>,
Jacob Keller <jacob.keller@gmail•com>
Subject: Re: [PATCH v3 3/5] name-rev: add support to exclude refs by pattern match
Date: Wed, 18 Jan 2017 12:11:27 -0800 [thread overview]
Message-ID: <xmqqtw8w86xc.fsf@gitster.mtv.corp.google.com> (raw)
In-Reply-To: <20170118000930.5431-4-jacob.e.keller@intel.com> (Jacob Keller's message of "Tue, 17 Jan 2017 16:09:28 -0800")
Jacob Keller <jacob.e.keller@intel•com> writes:
> From: Jacob Keller <jacob.keller@gmail•com>
>
> Extend name-rev further to support matching refs by adding `--exclude`
> patterns. These patterns will limit the scope of refs by excluding any
> ref that matches at least one exclude pattern. Checking the exclude refs
> shall happen first, before checking the include --refs patterns.
I do not think we should have this "exclude first and then include"
written down here, as it is an irrelevant implementation detail.
The desired end result is that only refs that match at least one
include and none of the exclude survive. You could implement it by
first checking with include and then further narrowing that set by
filtering those that match exclude (I am not saying that "include
first then exclude" is better---I am saying that it is far less
important than "at least one include and none of the exclude" to
mention the order of application).
> +--exclude=<pattern>::
> + Do not use any ref whose name matches a given shell pattern. The
> + pattern can be one of branch name, tag name or fully qualified ref
> + name. If given multiple times, exclude refs that match any of the given
> + shell patterns. Use `--no-exclude` to clear the list of exclude
> + patterns.
Perhaps insert
When used together with --refs, only those that match at least
one of the --refs patterns and none of the --exclude patterns
are used.
before "Use `--no-exclude` to clear"?
next prev parent reply other threads:[~2017-01-18 20:16 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-18 0:09 [PATCH v3 0/5] extend git-describe pattern matching Jacob Keller
2017-01-18 0:09 ` [PATCH v3 1/5] doc: add documentation for OPT_STRING_LIST Jacob Keller
2017-01-18 19:45 ` Junio C Hamano
2017-01-18 20:08 ` Philip Oakley
2017-01-18 20:58 ` Junio C Hamano
2017-01-19 16:55 ` Johannes Schindelin
2017-01-18 21:10 ` Jacob Keller
2017-01-19 17:58 ` Junio C Hamano
2017-01-18 0:09 ` [PATCH v3 2/5] name-rev: extend --refs to accept multiple patterns Jacob Keller
2017-01-18 20:04 ` Junio C Hamano
2017-01-18 21:12 ` Jacob Keller
2017-01-18 22:42 ` Junio C Hamano
2017-01-18 0:09 ` [PATCH v3 3/5] name-rev: add support to exclude refs by pattern match Jacob Keller
2017-01-18 20:11 ` Junio C Hamano [this message]
2017-01-18 21:13 ` Jacob Keller
2017-01-18 21:56 ` Junio C Hamano
2017-01-18 22:31 ` Jacob Keller
2017-01-18 0:09 ` [PATCH v3 4/5] describe: teach --match to accept multiple patterns Jacob Keller
2017-01-18 0:09 ` [PATCH v3 5/5] describe: teach describe negative pattern matches Jacob Keller
2017-01-18 20:17 ` Junio C Hamano
2017-01-18 20:18 ` [PATCH v3 0/5] extend git-describe pattern matching Junio C Hamano
2017-01-18 21:06 ` Jacob Keller
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=xmqqtw8w86xc.fsf@gitster.mtv.corp.google.com \
--to=gitster@pobox$(echo .)com \
--cc=git@vger$(echo .)kernel.org \
--cc=j6t@kdbg$(echo .)org \
--cc=jacob.e.keller@intel$(echo .)com \
--cc=jacob.keller@gmail$(echo .)com \
--cc=johannes.schindelin@gmx$(echo .)de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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