git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
* small misspellings fixes
@ 2013-06-18 23:04 Veres Lajos
  2013-06-19  4:15 ` Ramkumar Ramachandra
  0 siblings, 1 reply; 5+ messages in thread
From: Veres Lajos @ 2013-06-18 23:04 UTC (permalink / raw)
  To: git

Hi,

I am trying to convert this pull request:
https://github.com/git/git/pull/42
to a proper patch email
(Sorry If I miss something about the process.)

I found a few trivial misspellings.

Signed-off-by: Veres Lajos <vlajos@gmail.com>
---
diff --git a/git-p4.py b/git-p4.py
index 911bbce..88fcf23 100755
--- a/git-p4.py
+++ b/git-p4.py
@@ -3168,7 +3168,7 @@ class P4Rebase(Command):
         if os.system("git update-index --refresh") != 0:
             die("Some files in your working directory are modified and different than what is in your index. You can use git update-index <filename> to bring the index up-to-date or stash away all your changes with git stash.");
         if len(read_pipe("git diff-index HEAD --")) > 0:
-            die("You have uncommited changes. Please commit them before rebasing or stash them away with git stash.");
+            die("You have uncommitted changes. Please commit them before rebasing or stash them away with git stash.");

         [upstream, settings] = findUpstreamBranchPoint()
         if len(upstream) == 0:
diff --git a/git-svn.perl b/git-svn.perl
index d070de0..4e8275f 100755
--- a/git-svn.perl
+++ b/git-svn.perl
@@ -1246,7 +1246,7 @@ sub cmd_rebase {
 		return;
 	}
 	if (command(qw/diff-index HEAD --/)) {
-		print STDERR "Cannot rebase with uncommited changes:\n";
+		print STDERR "Cannot rebase with uncommitted changes:\n";
 		command_noisy('status');
 		exit 1;
 	}
diff --git a/t/t7600-merge.sh b/t/t7600-merge.sh
index 2f70433..460d8eb 100755
--- a/t/t7600-merge.sh
+++ b/t/t7600-merge.sh
@@ -316,7 +316,7 @@ test_expect_success 'merge c1 with c2 (squash)' '

 test_debug 'git log --graph --decorate --oneline --all'

-test_expect_success 'unsuccesful merge of c1 with c2 (squash, ff-only)' '
+test_expect_success 'unsuccessful merge of c1 with c2 (squash, ff-only)' '
 	git reset --hard c1 &&
 	test_must_fail git merge --squash --ff-only c2
 '

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: small misspellings fixes
  2013-06-18 23:04 Veres Lajos
@ 2013-06-19  4:15 ` Ramkumar Ramachandra
  0 siblings, 0 replies; 5+ messages in thread
From: Ramkumar Ramachandra @ 2013-06-19  4:15 UTC (permalink / raw)
  To: Veres Lajos; +Cc: git

Veres Lajos wrote:
> Subject: small misspelling fixes

Good.

> I am trying to convert this pull request:
> https://github.com/git/git/pull/42
> to a proper patch email
> (Sorry If I miss something about the process.)

This does not belong in the commit message: you don't want this to be
shown when someone is reading 'git log' output, do you?  Put such
comments under the "--" line after the signoff, so it'll be ignored by
git am.

Thanks.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* small misspellings fixes
@ 2013-06-19  5:37 Veres Lajos
  2013-06-19  5:51 ` Matthieu Moy
  0 siblings, 1 reply; 5+ messages in thread
From: Veres Lajos @ 2013-06-19  5:37 UTC (permalink / raw)
  To: git

small misspellings fixes

Signed-off-by: Veres Lajos <vlajos@gmail.com>
---
diff --git a/git-p4.py b/git-p4.py
index 911bbce..88fcf23 100755
--- a/git-p4.py
+++ b/git-p4.py
@@ -3168,7 +3168,7 @@ class P4Rebase(Command):
         if os.system("git update-index --refresh") != 0:
             die("Some files in your working directory are modified and different than what is in your index. You can use git update-index <filename> to bring the index up-to-date or stash away all your changes with git stash.");
         if len(read_pipe("git diff-index HEAD --")) > 0:
-            die("You have uncommited changes. Please commit them before rebasing or stash them away with git stash.");
+            die("You have uncommitted changes. Please commit them before rebasing or stash them away with git stash.");

         [upstream, settings] = findUpstreamBranchPoint()
         if len(upstream) == 0:
diff --git a/git-svn.perl b/git-svn.perl
index d070de0..4e8275f 100755
--- a/git-svn.perl
+++ b/git-svn.perl
@@ -1246,7 +1246,7 @@ sub cmd_rebase {
 		return;
 	}
 	if (command(qw/diff-index HEAD --/)) {
-		print STDERR "Cannot rebase with uncommited changes:\n";
+		print STDERR "Cannot rebase with uncommitted changes:\n";
 		command_noisy('status');
 		exit 1;
 	}
diff --git a/t/t7600-merge.sh b/t/t7600-merge.sh
index 2f70433..460d8eb 100755
--- a/t/t7600-merge.sh
+++ b/t/t7600-merge.sh
@@ -316,7 +316,7 @@ test_expect_success 'merge c1 with c2 (squash)' '

 test_debug 'git log --graph --decorate --oneline --all'

-test_expect_success 'unsuccesful merge of c1 with c2 (squash, ff-only)' '
+test_expect_success 'unsuccessful merge of c1 with c2 (squash, ff-only)' '
 	git reset --hard c1 &&
 	test_must_fail git merge --squash --ff-only c2
 '
-- 
Hi,

I am trying to convert this pull request:
https://github.com/git/git/pull/42
to a proper patch email
(Sorry If I miss something about the process.)
-- 
Veres Lajos
vlajos@gmail.com
+36 20 438 5909

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: small misspellings fixes
  2013-06-19  5:37 small misspellings fixes Veres Lajos
@ 2013-06-19  5:51 ` Matthieu Moy
  2013-06-19 18:31   ` Junio C Hamano
  0 siblings, 1 reply; 5+ messages in thread
From: Matthieu Moy @ 2013-06-19  5:51 UTC (permalink / raw)
  To: Veres Lajos; +Cc: git

Veres Lajos <vlajos@gmail.com> writes:

> I am trying to convert this pull request:
> https://github.com/git/git/pull/42
> to a proper patch email

Use "git format-patch" and/or "git send-email" to get the proper
formatting.

> (Sorry If I miss something about the process.)

Yes, your patch doesn't have exactly the right format, and you should
have Cc-ed Junio (see Documentation/SubmittingPatches in Git's source
tree).

Other than that, the content of the patch looks good (and actually, at
least one of the typos is mine :-( ). Can you resend?

Thanks,

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: small misspellings fixes
  2013-06-19  5:51 ` Matthieu Moy
@ 2013-06-19 18:31   ` Junio C Hamano
  0 siblings, 0 replies; 5+ messages in thread
From: Junio C Hamano @ 2013-06-19 18:31 UTC (permalink / raw)
  To: Matthieu Moy; +Cc: Veres Lajos, git

Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> writes:

> Veres Lajos <vlajos@gmail.com> writes:
>
>> I am trying to convert this pull request:
>> https://github.com/git/git/pull/42
>> to a proper patch email
>
> Use "git format-patch" and/or "git send-email" to get the proper
> formatting.
>
>> (Sorry If I miss something about the process.)
>
> Yes, your patch doesn't have exactly the right format, and you should
> have Cc-ed Junio (see Documentation/SubmittingPatches in Git's source
> tree).
>
> Other than that, the content of the patch looks good (and actually, at
> least one of the typos is mine :-( ). Can you resend?

Repeating the subject in the body is unnecessary.

I'll pick it up from here and will fix up.  No need to resend.

Thanks.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-06-19 18:31 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-19  5:37 small misspellings fixes Veres Lajos
2013-06-19  5:51 ` Matthieu Moy
2013-06-19 18:31   ` Junio C Hamano
  -- strict thread matches above, loose matches on Subject: below --
2013-06-18 23:04 Veres Lajos
2013-06-19  4:15 ` Ramkumar Ramachandra

Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).