* [PATCH] check-ignore: correct documentation about output
@ 2015-11-08 20:10 Dennis Kaarsemaker
2015-11-16 14:13 ` Dennis Kaarsemaker
2015-11-20 11:31 ` Jeff King
0 siblings, 2 replies; 8+ messages in thread
From: Dennis Kaarsemaker @ 2015-11-08 20:10 UTC (permalink / raw)
To: git
By default git check-ignore shows only the filenames that will be
ignored, not the pattern that causes their exclusion.
Signed-off-by: Dennis Kaarsemaker <dennis@kaarsemaker.net>
---
Documentation/git-check-ignore.txt | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/Documentation/git-check-ignore.txt b/Documentation/git-check-ignore.txt
index 59531ab..0a628ac 100644
--- a/Documentation/git-check-ignore.txt
+++ b/Documentation/git-check-ignore.txt
@@ -16,10 +16,9 @@ DESCRIPTION
-----------
For each pathname given via the command-line or from a file via
-`--stdin`, show the pattern from .gitignore (or other input files to
-the exclude mechanism) that decides if the pathname is excluded or
-included. Later patterns within a file take precedence over earlier
-ones.
+`--stdin`, check whether the file is excluded by .gitignore (or other
+input files to the exclude mechanism) and output the path if it is
+excluded.
By default, tracked files are not shown at all since they are not
subject to exclude rules; but see `--no-index'.
--
2.6.3-495-gf0a7f49
--
Dennis Kaarsemaker <dennis@kaarsemaker.net>
http://twitter.com/seveas
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] check-ignore: correct documentation about output
2015-11-08 20:10 [PATCH] check-ignore: correct documentation about output Dennis Kaarsemaker
@ 2015-11-16 14:13 ` Dennis Kaarsemaker
2015-11-17 0:22 ` Stefan Beller
2015-11-20 11:31 ` Jeff King
1 sibling, 1 reply; 8+ messages in thread
From: Dennis Kaarsemaker @ 2015-11-16 14:13 UTC (permalink / raw)
To: git
Ping.
On zo, 2015-11-08 at 21:10 +0100, Dennis Kaarsemaker wrote:
> By default git check-ignore shows only the filenames that will be
> ignored, not the pattern that causes their exclusion.
>
> Signed-off-by: Dennis Kaarsemaker <dennis@kaarsemaker.net>
> ---
> Documentation/git-check-ignore.txt | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/git-check-ignore.txt b/Documentation/git
> -check-ignore.txt
> index 59531ab..0a628ac 100644
> --- a/Documentation/git-check-ignore.txt
> +++ b/Documentation/git-check-ignore.txt
> @@ -16,10 +16,9 @@ DESCRIPTION
> -----------
>
> For each pathname given via the command-line or from a file via
> -`--stdin`, show the pattern from .gitignore (or other input files to
> -the exclude mechanism) that decides if the pathname is excluded or
> -included. Later patterns within a file take precedence over earlier
> -ones.
> +`--stdin`, check whether the file is excluded by .gitignore (or
> other
> +input files to the exclude mechanism) and output the path if it is
> +excluded.
>
> By default, tracked files are not shown at all since they are not
> subject to exclude rules; but see `--no-index'.
> --
> 2.6.3-495-gf0a7f49
>
>
--
Dennis Kaarsemaker
http://www.kaarsemaker.net
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] check-ignore: correct documentation about output
2015-11-16 14:13 ` Dennis Kaarsemaker
@ 2015-11-17 0:22 ` Stefan Beller
2015-11-17 22:37 ` Jeff King
0 siblings, 1 reply; 8+ messages in thread
From: Stefan Beller @ 2015-11-17 0:22 UTC (permalink / raw)
To: Dennis Kaarsemaker, Jeff King; +Cc: git
+cc Jeff
On Mon, Nov 16, 2015 at 6:13 AM, Dennis Kaarsemaker
<dennis@kaarsemaker.net> wrote:
> Ping.
Junio is on vacation, so Jeff is our interim maintainer,
and it seems he isn't up to full power as Junio. :)
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] check-ignore: correct documentation about output
2015-11-17 0:22 ` Stefan Beller
@ 2015-11-17 22:37 ` Jeff King
0 siblings, 0 replies; 8+ messages in thread
From: Jeff King @ 2015-11-17 22:37 UTC (permalink / raw)
To: Stefan Beller; +Cc: Dennis Kaarsemaker, git
On Mon, Nov 16, 2015 at 04:22:16PM -0800, Stefan Beller wrote:
> +cc Jeff
>
> On Mon, Nov 16, 2015 at 6:13 AM, Dennis Kaarsemaker
> <dennis@kaarsemaker.net> wrote:
> > Ping.
>
> Junio is on vacation, so Jeff is our interim maintainer,
> and it seems he isn't up to full power as Junio. :)
Yes, I'm sorry, I've been doing a terrible job so far, as I haven't
pushed out anything. I'm making it my priority over the next day or so.
-Peff
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] check-ignore: correct documentation about output
2015-11-08 20:10 [PATCH] check-ignore: correct documentation about output Dennis Kaarsemaker
2015-11-16 14:13 ` Dennis Kaarsemaker
@ 2015-11-20 11:31 ` Jeff King
2015-11-20 20:29 ` Dennis Kaarsemaker
1 sibling, 1 reply; 8+ messages in thread
From: Jeff King @ 2015-11-20 11:31 UTC (permalink / raw)
To: Dennis Kaarsemaker; +Cc: git
On Sun, Nov 08, 2015 at 09:10:39PM +0100, Dennis Kaarsemaker wrote:
> By default git check-ignore shows only the filenames that will be
> ignored, not the pattern that causes their exclusion.
Yeah, I think the described behavior is really about the "-v" output.
> --- a/Documentation/git-check-ignore.txt
> +++ b/Documentation/git-check-ignore.txt
> @@ -16,10 +16,9 @@ DESCRIPTION
> -----------
>
> For each pathname given via the command-line or from a file via
> -`--stdin`, show the pattern from .gitignore (or other input files to
> -the exclude mechanism) that decides if the pathname is excluded or
> -included. Later patterns within a file take precedence over earlier
> -ones.
> +`--stdin`, check whether the file is excluded by .gitignore (or other
> +input files to the exclude mechanism) and output the path if it is
> +excluded.
This just drops the mention of patterns entirely. Is the description of
"-v" sufficient to cover this (especially I am thinking of the
last-pattern-wins behavior)?
-Peff
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] check-ignore: correct documentation about output
2015-11-20 11:31 ` Jeff King
@ 2015-11-20 20:29 ` Dennis Kaarsemaker
2015-11-20 20:30 ` [PATCH v2] " Dennis Kaarsemaker
0 siblings, 1 reply; 8+ messages in thread
From: Dennis Kaarsemaker @ 2015-11-20 20:29 UTC (permalink / raw)
To: Jeff King, git
[Peff, sorry for the duplicate mail. It seems I can't brain today...]
On vr, 2015-11-20 at 06:31 -0500, Jeff King wrote:
> On Sun, Nov 08, 2015 at 09:10:39PM +0100, Dennis Kaarsemaker wrote:
>
> > By default git check-ignore shows only the filenames that will be
> > ignored, not the pattern that causes their exclusion.
>
> Yeah, I think the described behavior is really about the "-v" output.
Yup.
> > --- a/Documentation/git-check-ignore.txt
> > +++ b/Documentation/git-check-ignore.txt
> > @@ -16,10 +16,9 @@ DESCRIPTION
> > -----------
> >
> > For each pathname given via the command-line or from a file via
> > -`--stdin`, show the pattern from .gitignore (or other input files
> > to
> > -the exclude mechanism) that decides if the pathname is excluded or
> > -included. Later patterns within a file take precedence over
> > earlier
> > -ones.
> > +`--stdin`, check whether the file is excluded by .gitignore (or
> > other
> > +input files to the exclude mechanism) and output the path if it is
> > +excluded.
>
> This just drops the mention of patterns entirely. Is the description
> of "-v" sufficient to cover this (especially I am thinking of the
> last-pattern-wins behavior)?
The -v should then also describe precendence rules between files. Or
maybe just link to gitignore(5).
--
Dennis Kaarsemaker
www.kaarsemaker.net
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2] check-ignore: correct documentation about output
2015-11-20 20:29 ` Dennis Kaarsemaker
@ 2015-11-20 20:30 ` Dennis Kaarsemaker
2015-11-24 22:13 ` Jeff King
0 siblings, 1 reply; 8+ messages in thread
From: Dennis Kaarsemaker @ 2015-11-20 20:30 UTC (permalink / raw)
To: git
By default git check-ignore shows only the filenames that will be
ignored, not the pattern that causes their exclusion. Instead of moving
the partial exclude pattern precendence information to the -v option
where it belongs, link to gitignore(5) which describes this more
thoroughly.
Signed-off-by: Dennis Kaarsemaker <dennis@kaarsemaker.net>
---
Documentation/git-check-ignore.txt | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/Documentation/git-check-ignore.txt b/Documentation/git-check-ignore.txt
index 59531ab..e94367a 100644
--- a/Documentation/git-check-ignore.txt
+++ b/Documentation/git-check-ignore.txt
@@ -16,10 +16,9 @@ DESCRIPTION
-----------
For each pathname given via the command-line or from a file via
-`--stdin`, show the pattern from .gitignore (or other input files to
-the exclude mechanism) that decides if the pathname is excluded or
-included. Later patterns within a file take precedence over earlier
-ones.
+`--stdin`, check whether the file is excluded by .gitignore (or other
+input files to the exclude mechanism) and output the path if it is
+excluded.
By default, tracked files are not shown at all since they are not
subject to exclude rules; but see `--no-index'.
@@ -32,7 +31,8 @@ OPTIONS
-v, --verbose::
Also output details about the matching pattern (if any)
- for each given pathname.
+ for each given pathname. For precedence rules within and
+ between exclude sources, see linkgit:gitignore[5].
--stdin::
Read pathnames from the standard input, one per line,
--
2.6.3-495-gf0a7f49
--
Dennis Kaarsemaker <dennis@kaarsemaker.net>
http://twitter.com/seveas
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2] check-ignore: correct documentation about output
2015-11-20 20:30 ` [PATCH v2] " Dennis Kaarsemaker
@ 2015-11-24 22:13 ` Jeff King
0 siblings, 0 replies; 8+ messages in thread
From: Jeff King @ 2015-11-24 22:13 UTC (permalink / raw)
To: Dennis Kaarsemaker; +Cc: git
On Fri, Nov 20, 2015 at 09:30:48PM +0100, Dennis Kaarsemaker wrote:
> By default git check-ignore shows only the filenames that will be
> ignored, not the pattern that causes their exclusion. Instead of moving
> the partial exclude pattern precendence information to the -v option
> where it belongs, link to gitignore(5) which describes this more
> thoroughly.
>
> Signed-off-by: Dennis Kaarsemaker <dennis@kaarsemaker.net>
> ---
> Documentation/git-check-ignore.txt | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
Thanks, your update looks good to me.
-Peff
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2015-11-24 22:13 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-08 20:10 [PATCH] check-ignore: correct documentation about output Dennis Kaarsemaker
2015-11-16 14:13 ` Dennis Kaarsemaker
2015-11-17 0:22 ` Stefan Beller
2015-11-17 22:37 ` Jeff King
2015-11-20 11:31 ` Jeff King
2015-11-20 20:29 ` Dennis Kaarsemaker
2015-11-20 20:30 ` [PATCH v2] " Dennis Kaarsemaker
2015-11-24 22:13 ` Jeff King
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.