patch for git-p4

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

patch for git-p4

From: Andrew Garber <hidden>
Date: 2016-06-15 22:50:20

Hi everyone! This is my first post to the git mailing list.

I was reading the git-p4 source and noticed that some of the lines are
indented using tabs instead of spaces (very bad in Python). Here's a
patch against tag v1.7.4-rc0 (commit
01b97a4cb60723d18b437efdc474503d2a9dd384) of the git source repo.

Thanks!
Andrew Garber

Re: patch for git-p4

From: Erik Faye-Lund <hidden>
Date: 2016-06-15 22:50:20

On Wed, Jan 5, 2011 at 2:45 AM, Andrew Garber [off-list ref] wrote:
Hi everyone! This is my first post to the git mailing list.
Welcome!
I was reading the git-p4 source and noticed that some of the lines are
indented using tabs instead of spaces (very bad in Python). Here's a
patch against tag v1.7.4-rc0 (commit
01b97a4cb60723d18b437efdc474503d2a9dd384) of the git source repo.

Thanks!
Andrew Garber
Please read Documentation/SubmittingPatches for the correct procedure
for submitting patches. The two most important things is:
1) We send patches inline instead of attached (git send-email can do
this for you)
2) You need to add a sign-off

As a side-note, your commit message has some broken grammar in it.
"Some the commits", I'm guessing you omitted a "of" there?

We tend not to write commit messages in past tence. E.g "git-p4:
replace tabs with spaces" (notice that I removed a 'd').
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help