Message ID | 4DDDE3AF.2000906@samsung.com |
---|---|
State | Superseded |
Headers | show |
Hi Minkyu, Le 26/05/2011 07:22, Minkyu Kang a écrit : > Dear Albert Aribaud, Please make sure the 'at free dot fr' address of mine is not used any more -- there are good chances I won't see U-Boot mail on it anyway. > The following changes since commit 0cecc52c233e5f607f826a4e5c97f92c7a7d6dc4: > > Merge branch 'master' of git://git.denx.de/u-boot-arm into next (2011-05-23 17:47:29 +0900) > > are available in the git repository at: > > git://git.denx.de/u-boot-samsung master Rather than a merge, could you please do a rebase (onto f14a522a (Beagleboard: fixed typo in typecast) as indicated in http://www.denx.de/wiki/view/U-Boot/CustodianGitTrees#BEFORE_Requesting_a_Pull)? In addition with making the commit tree simpler, it will ensure that the "since" commit mentioned in your request is actually one that exists in the pulling repo. Amicalement,
Dear Albert ARIBAUD, On 26 May 2011 14:49, Albert ARIBAUD <albert.u.boot@aribaud.net> wrote: > Hi Minkyu, > > Le 26/05/2011 07:22, Minkyu Kang a écrit : >> Dear Albert Aribaud, > > Please make sure the 'at free dot fr' address of mine is not used any > more -- there are good chances I won't see U-Boot mail on it anyway. OK. > >> The following changes since commit 0cecc52c233e5f607f826a4e5c97f92c7a7d6dc4: >> >> Merge branch 'master' of git://git.denx.de/u-boot-arm into next (2011-05-23 17:47:29 +0900) >> >> are available in the git repository at: >> >> git://git.denx.de/u-boot-samsung master > > Rather than a merge, could you please do a rebase (onto f14a522a > (Beagleboard: fixed typo in typecast) as indicated in > http://www.denx.de/wiki/view/U-Boot/CustodianGitTrees#BEFORE_Requesting_a_Pull)? I already did a rebase. Any problem? Please see the summary. (http://git.denx.de/?p=u-boot/u-boot-samsung.git;a=summary) > > In addition with making the commit tree simpler, it will ensure that the > "since" commit mentioned in your request is actually one that exists in > the pulling repo. > > Amicalement, > -- > Albert. > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot > Thanks Minkyu Kang
Hi Minkyu, Le 26/05/2011 08:24, Minkyu Kang a écrit : >> Rather than a merge, could you please do a rebase (onto f14a522a >> (Beagleboard: fixed typo in typecast) as indicated in >> http://www.denx.de/wiki/view/U-Boot/CustodianGitTrees#BEFORE_Requesting_a_Pull)? > > I already did a rebase. > Any problem? The problem is, it is not a rebase. What you did apparently is merge the u-boot-arm master branch to yours, yielding the commit that your pull request is based upon. What the Wiki suggests is that you rebase your branch onto the target one before doing the pull request, so that it is based upon the tip of the target branch. Amicalement,
Dear Albert ARIBAUD, On 26 May 2011 17:13, Albert ARIBAUD <albert.u.boot@aribaud.net> wrote: > Hi Minkyu, > > Le 26/05/2011 08:24, Minkyu Kang a écrit : > >>> Rather than a merge, could you please do a rebase (onto f14a522a >>> (Beagleboard: fixed typo in typecast) as indicated in >>> >>> http://www.denx.de/wiki/view/U-Boot/CustodianGitTrees#BEFORE_Requesting_a_Pull)? >> >> I already did a rebase. >> Any problem? > > The problem is, it is not a rebase. What you did apparently is merge the > u-boot-arm master branch to yours, yielding the commit that your pull > request is based upon. What the Wiki suggests is that you rebase your branch > onto the target one before doing the pull request, so that it is based upon > the tip of the target branch. > OK. I see. I'll send pull request again. Thanks Minkyu Kang