Skip to contentSkip to conversation

Please pull u-boot-arm

GIT PULL Peter Pearse 2 participants 2 messages Apr 11, 2008

lore
Share & tools
lore
#0 lore raw
Quoted 82 lines

-----Original Message-----
From: Peter Pearse [mailto:peter.pearse at arm.com]
Sent: 11 April 2008 09:47
To: 'Wolfgang Denk'
Subject: RE: [U-Boot-Users] [GIT PULL] Please pull u-boot-arm

Quoted 9 lines

-----Original Message-----
From: u-boot-users-bounces at lists.sourceforge.net [mailto:u-boot-users-bounces at lists.sourceforge.net] On Behalf Of Wolfgang Denk
Sent: 10 April 2008 11:24
To: Peter Pearse; Guennadi Liakhovetski
Cc: u-boot-users at lists.sourceforge.net
Subject: Re: [U-Boot-Users] [GIT PULL] Please pull u-boot-arm

Quoted 6 lines

Dear Peter,

Quoted 3 lines · in message <[email protected]> you wrote:

These are the patches:

--- snip ---

Quoted 10 lines

I would expect, that the state of the current repository represents what we get after applying Sascha's and Guennadi's patches (in the correct versions and order). Howver, the current tree looks totally different to me.

Can you please check with Guennadi what happened, how this can be cleaned up, and especially how this can be prevented in the future?

Note that there is not only not the expected state of the

source code,

Quoted 14 lines

but there is also corruption of meta-information again; for example:

	-> git-show --pretty=fuller
7a837b7310166ae8fc8b8d66d7ef01b60a80f9d6
	commit 7a837b7310166ae8fc8b8d66d7ef01b60a80f9d6
	Author:     Guennadi Liakhovetski <[[email protected]]>
-------------------------------------------^^^^^^^^^^^^
	AuthorDate: Sun Mar 30 11:32:30 2008 +0100
	Commit:     Peter Pearse <[email protected]>
	CommitDate: Sun Mar 30 11:32:30 2008 +0100

	    Support for the MX31ADS evaluation board from Freescale
	...
Quoted 1 line

None of Guennadis messages on the mailing list ever used

something

Quoted 1 line

like "[lg at denx.de]" - this must be something that happend

on your end

Quoted 3 lines

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.

On the code my patch corrections e.g. for line length might differ from those applied by someone else.

Guennadis mail address corruption occurred because I missed an "on behalf of" in the patch mail.

I'm looking at Manus's mail to see where we differ...

Regards

Peter

#1 lore raw

Peter, please, do NOT drop addressees from CC.

Quoted 5 lines · Peter Pearse
Quoted 5 lines

I find I have to manually edit the mails I get from my (company mandated) mail client.

My process is

Quoted 8 lines
Quoted 8 lines

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.

On the code my patch corrections e.g. for line length might differ from those applied by someone else.

To which of the above "corrections" does this hunk belong:

board/mx31ads/Makefile +5 -1
index dfadd96..c854e05 100644
@@ -1,18 +1,14 @@
 #
+# (C) Copyright 2000-2008
 # Copyright (C) 2008, Guennadi Liakhovetski <[email protected]>
 #
+# See file CREDITS for list of people who contributed to this
+# project.
+#
 # This program is free software; you can redistribute it and/or
 # modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundatio; either version 2 of
-# published by the Free Software Foundation; either version 2 of
 

Especially the last line?

Thanks
Guennadi
---
Guennadi Liakhovetski