Re: [PATCH] check-ignore: correct documentation about output
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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] check-ignore: correct documentation about output
[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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] check-ignore: correct documentation about output
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 >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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] check-ignore: correct documentation about output
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> --- > 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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] check-ignore: correct documentation about output
+cc Jeff On Mon, Nov 16, 2015 at 6:13 AM, Dennis Kaarsemakerwrote: > Ping. Junio is on vacation, so Jeff is our interim maintainer, and it seems he isn't up to full power as Junio. :) -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html