threads / discuss / 10856

Trivial patch to git-svn

Subject: Trivial patch to git-svn

## tl;dr

4 messages between Nov 13, 2007 and Nov 17, 2007.

replies: 3people: 3as markdown or json

David Reiss· Nov 13, 2007, 21:56 UTC · lore
 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@facebook.com>
---
  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;
-- 
1.5.3.4
Eric Wong· Nov 17, 2007, 21:12 UTC · re: David Reiss · lore

Re: Trivial patch to git-svn

David Reiss <dreiss@facebook.com> wrote:
Show 25 quoted lines
> 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@facebook.com>
> ---
>  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?

-- 
Eric Wong
Adam Roben· Nov 17, 2007, 21:26 UTC · re: Eric Wong · lore

Re: Trivial patch to git-svn

Eric Wong wrote:
Show 32 quoted lines
> David Reiss <dreiss@facebook.com> 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@facebook.com>
>> ---
>>  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. :-)

-Adam
Eric Wong· Nov 17, 2007, 21:54 UTC · re: Adam Roben · lore

Re: Trivial patch to git-svn

Adam Roben <aroben@apple.com> wrote:
Show 37 quoted lines
> Eric Wong wrote:
> >David Reiss <dreiss@facebook.com> 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@facebook.com>
> >>---
> >> 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@yhbt.net>
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

← back to recent threads