all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Damien Wyart <damien.wyart@gmail.com>
To: Stephen Berman <stephen.berman@gmx.net>
Cc: Bastien <bzg@gnu.org>, Andreas Schwab <schwab@suse.de>,
	Richard Stallman <rms@gnu.org>,
	emacs-devel@gnu.org
Subject: Re: When was a change installed?
Date: Wed, 19 Aug 2015 12:16:32 +0200	[thread overview]
Message-ID: <20150819101632.GA32499@brouette> (raw)
In-Reply-To: <877forlhww.fsf@gmx.net>

> > > $ git grep lax-whitespace */ChangeLog*
> > > without the single quotes works for me.

> > That's not the same, as the glob is expanded by the shell before git
> > can see it.

* Stephen Berman <stephen.berman@gmx.net> [2015-08-19 11:57]:
> The error message by git, which I also get on invoking
>   git grep lax-whitespace '*/ChangeLog.*'
> says "Use '--' to separate paths from revisions, like this: 'git
> <command> [<revision>...]  -- [<file>...]'", and indeed, when I run
>   git grep lax-whitespace -- '*/ChangeLog.*'
> it outputs 26 matching lines.  Invoking
>   git grep lax-whitespace */ChangeLog*
> outputs only 25 of thos lines, all from lisp/; missing is
>   "doc/emacs/ChangeLog.1:	(Replace): Document replace-lax-whitespace."
> which the other output contains.

> So you're right there's a difference, but apparently the '--' is necessary.

I guess Andreas is using git 2.5.0, where '--' is not necessary anymore
in this case.

Here is the corresponding item from the release notes:

,----
|  * A heuristic we use to catch mistyped paths on the command line
|   "git <cmd> <revs> <pathspec>" is to make sure that all the non-rev
|    parameters in the later part of the command line are names of the
|    files in the working tree, but that means "git grep $str -- \*.c"
|    must always be disambiguated with "--", because nobody sane will
|    create a file whose name literally is asterisk-dot-see.  Loosen the
|    heuristic to declare that with a wildcard string the user likely
|    meant to give us a pathspec.
`----
                      
-- 
Damien



  reply	other threads:[~2015-08-19 10:16 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-17  5:04 bug#21277: 25.0.50; char-fold search is a screw, and I can't see how to disable Richard Stallman
2015-08-17  7:23 ` Andreas Schwab
2015-08-18  3:41   ` Richard Stallman
2015-08-18 12:30     ` Tassilo Horn
2015-08-18 14:24       ` Drew Adams
2015-08-18 21:58       ` Artur Malabarba
2015-08-18 21:58         ` Artur Malabarba
2015-08-19 18:14         ` Richard Stallman
2015-08-19 22:17       ` Juri Linkov
2015-08-18  3:41   ` Richard Stallman
2015-08-19 22:08     ` Juri Linkov
2015-08-20 16:56       ` Richard Stallman
2015-08-18  3:42   ` When was a change installed? Richard Stallman
2015-08-18 14:10     ` Alan Mackenzie
2015-08-19  6:38       ` Steinar Bang
2015-08-19 18:16         ` Richard Stallman
2015-08-18 14:18     ` Andreas Schwab
2015-08-19  1:19       ` Richard Stallman
2015-08-19  4:28         ` David Kastrup
2015-08-19 18:18           ` Richard Stallman
2015-08-20  2:37             ` Stefan Monnier
2015-08-19  1:19       ` Richard Stallman
2015-08-19  7:52         ` Andreas Schwab
2015-08-19 18:18           ` Richard Stallman
2015-08-19  9:22         ` Bastien
2015-08-19  9:25           ` Andreas Schwab
2015-08-19  9:57             ` Stephen Berman
2015-08-19 10:16               ` Damien Wyart [this message]
2015-08-18 14:28     ` Tassilo Horn
2015-08-19  1:24       ` Richard Stallman
2015-08-17 19:02 ` bug#21277: 25.0.50; char-fold search is a screw, and I can't see how to disable Mark Oteiza
2015-08-18 22:29 ` Artur Malabarba
2015-08-19 22:02 ` Juri Linkov
2015-08-20  9:36   ` Artur Malabarba

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=20150819101632.GA32499@brouette \
    --to=damien.wyart@gmail.com \
    --cc=bzg@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=rms@gnu.org \
    --cc=schwab@suse.de \
    --cc=stephen.berman@gmx.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

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.