Git development
 help / color / mirror / Atom feed
From: Petr Baudis <pasky@suse.cz>
To: Junio C Hamano <junkio@cox.net>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] commit: detect misspelled pathspec while making a partial commit.
Date: Tue, 14 Feb 2006 22:25:52 +0100	[thread overview]
Message-ID: <20060214212552.GH31278@pasky.or.cz> (raw)
In-Reply-To: <7vfymlr7n8.fsf@assigned-by-dhcp.cox.net>

Dear diary, on Tue, Feb 14, 2006 at 09:46:03PM CET, I got a letter
where Junio C Hamano <junkio@cox.net> said that...
> When you say "git commit Documentaiton" to make partial commit
> for the files only in that directory, we did not detect that as
> a misspelled pathname and attempted to commit index without
> change.  If nothing matched, there is no harm done, but if the
> index gets modified otherwise by having another valid pathspec
> or after an explicit update-index, a user will not notice
> without paying attention to the "git status" preview.
> 
> This introduces --error-unmatch option to ls-files, and uses it
> to detect this common user error.
> 
> Signed-off-by: Junio C Hamano <junkio@cox.net>
> 
> ---
> 
>  * Pasky, does this address the issue you mentioned earlier on
>    the #git channnel?

Yes, this is what I meant. I actually have no practical experience with
this since I'm not a git-commit user, I merely expressed horror ;) on
what you mentioned during the night. BUT it turns out that this affects
Cogito as well, albeit in a milder version - only when you pass multiple
explicit paths to cg-commit and some of them are misspelled, they are
ignored quietly (unless all of them are misspelled).

So, thanks for the fix on behalf of Cogito users as well. ;-)

-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
Of the 3 great composers Mozart tells us what it's like to be human,
Beethoven tells us what it's like to be Beethoven and Bach tells us
what it's like to be the universe.  -- Douglas Adams

      reply	other threads:[~2006-02-14 21:25 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-14 20:46 [PATCH] commit: detect misspelled pathspec while making a partial commit Junio C Hamano
2006-02-14 21:25 ` Petr Baudis [this message]

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=20060214212552.GH31278@pasky.or.cz \
    --to=pasky@suse.cz \
    --cc=git@vger.kernel.org \
    --cc=junkio@cox.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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox