Re: Trivial patch to git-svn

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Adam Roben <aroben@xxxxxxxxx> wrote:
> Eric Wong wrote:
> >David Reiss <dreiss@xxxxxxxxxxxx> wrote:
> >  
> >>From d9f93dc1c503c5f42b27485b2c35b38e8c9bca44 Mon Sep 17 00:00:00 2001
> >>From: David Reiss <dreiss@dreiss-vmware.(none)>
> >>Date: Tue, 13 Nov 2007 13:47:26 -0800
> >>Subject: [PATCH] Fix a typo and add a comma in an error message in 
> >>git-svn
> >>
> >>Signed-off-by: David Reiss <dreiss@xxxxxxxxxxxx>
> >>---
> >> git-svn.perl |    2 +-
> >> 1 files changed, 1 insertions(+), 1 deletions(-)
> >>
> >>diff --git a/git-svn.perl b/git-svn.perl
> >>index e3e00fd..981cdd6 100755
> >>--- a/git-svn.perl
> >>+++ b/git-svn.perl
> >>@@ -391,7 +391,7 @@ sub cmd_set_tree {
> >> sub cmd_dcommit {
> >> 	my $head = shift;
> >> 	git_cmd_try { command_oneline(qw/diff-index --quiet HEAD/) }
> >>-		'Cannot dcommit with a dirty index.  Commit your changes 
> >>first'
> >>+		'Cannot dcommit with a dirty index.  Commit your changes 
> >>first, '
> >> 		. "or stash them with `git stash'.\n";
> >> 	$head ||= 'HEAD';
> >> 	my @refs;
> >>    
> >
> >I am not an English expert, but both seem acceptable to me.  Anybody else
> >agree/disagree?
> >  
> 
> I don't think the comma matters very much, but having a space between 
> "first" and "or" is pretty important. :-)

Good eyes, thanks Adam.

Acked-by: Eric Wong <normalperson@xxxxxxxx>

Pushed out to http://git.bogomips.org/git-svn.git

Along with several other changes
(http://article.gmane.org/gmane.comp.version-control.git/65314)

-- 
Eric Wong
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux