------- Forwarded Message Date: Fri, 11 Apr 2008 12:13:46 +0200 From: Wolfgang Denk <[EMAIL PROTECTED]> To: "Peter Pearse" <[EMAIL PROTECTED]> cc: [EMAIL PROTECTED] Subject: Re: [U-Boot-Users] [GIT PULL] Please pull u-boot-arm
Dear Peter, in message <[EMAIL PROTECTED]> you wrote: > ... > > None of Guennadis messages on the mailing list ever used > > something like "[EMAIL PROTECTED]" - this must be something that > > happend on your end when processing his patches. Where > > are the brackets coming from? > > Don't you use git-am to apply the patches? > > I find I have to manually edit the mails I get from my (company mandated) > mail client. > > My process is > > a) Save mail > b) Manually edit to remove extraneous/incorrect data and formatting > c) Run mail thru <linux kernel>/scripts/checkpatch.pl until acceptable > d) Run git-am, correcting the patch until applies without output other than > re > whitespace. This sounds terribly inefficient to me. Can't you use any other (free) email acoount for the U-Boot work which doesn't have such restrictions? > On the code my patch corrections e.g. for line length > might differ from those applied by someone else. I'm not sure if I understand you correctly. You mean, you modify the code before you check it in? You must not do this. A patch that was submitted on the mailing list and sigend off by the author has to be checked in as is, without modifications of the code. If you feel that cleanup is needed, you either have to reject the patch and ask the author the resubmit a new, cleaned up version, or you have to fix the code in a second step which then results in a new commit showing you as the author of the changes. But one must not, under no circumstances, modify the code first, and then check in the modified code under the original author's name without any indication that the code was modified by somebody else. I'm not sure if this is what you mean, but if iti s, then please never do that again. It's a strict NO. > I'm looking at Manus's mail to see where we differ... Yes, please. Thanks in advance. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] They're usually so busy thinking about what happens next that the only time they ever find out what is happening now is when they come to look back on it. - Terry Pratchett, _Wyrd Sisters_ ------- End of Forwarded Message ------------------------------------------------------------------------- This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone _______________________________________________ U-Boot-Users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/u-boot-users
