Re: [U-Boot] [PATCH] Fix issues introduced by 16fc32f: unify {CONFIG_, }ENV_IS_EMBEDDED
On Tuesday 25 August 2009 04:46:07 Albin Tonnerre wrote: Commit 16fc32f introduced an error in include/environment.h, which makes u-boot fail to compile due to a missing #endif. This also fixes a merge conflict remaining in include/configs/EB+MCF-EV123.h which was caused by the same commit this is only an issue for the next branch ... my local git tree has the right number of #endif's, so i guess i must have sent out the wrong patch guess this needs to get squashed into the relevant change rather than added as a new commit. thanks for the update. -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] Fix issues introduced by 16fc32f: unify {CONFIG_, }ENV_IS_EMBEDDED
On Tue, Aug 25, 2009 at 06:16:24AM -0400, Mike Frysinger wrote : On Tuesday 25 August 2009 04:46:07 Albin Tonnerre wrote: Commit 16fc32f introduced an error in include/environment.h, which makes u-boot fail to compile due to a missing #endif. This also fixes a merge conflict remaining in include/configs/EB+MCF-EV123.h which was caused by the same commit this is only an issue for the next branch ... my local git tree has the right number of #endif's, so i guess i must have sent out the wrong patch guess this needs to get squashed into the relevant change rather than added as a new commit. thanks for the update. How exactly would you do that? The commit actually got pushed, and AFAIK next doesn't get rebased Regards, -- Albin Tonnerre, Free Electrons Kernel, drivers and embedded Linux development, consulting, training and support. http://free-electrons.com signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] Fix issues introduced by 16fc32f: unify {CONFIG_, }ENV_IS_EMBEDDED
Dear Albin Tonnerre, In message 20090825102110.ge10...@pc-ras4041.res.insa you wrote: guess this needs to get squashed into the relevant change rather than added as a new commit. thanks for the update. How exactly would you do that? The commit actually got pushed, and AFAIK next doesn't get rebased It gets rebased when there is a need to do so. This seems to be such a case. 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: w...@denx.de 1 1 was a race-horse, 2 2 was 1 2. When 1 1 1 1 race, 2 2 1 1 2. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] Fix issues introduced by 16fc32f: unify {CONFIG_, }ENV_IS_EMBEDDED
Dear Albin Tonnerre, In message 1251189967-11702-1-git-send-email-albin.tonne...@free-electrons.com you wrote: Commit 16fc32f introduced an error in include/environment.h, which makes u-boot fail to compile due to a missing #endif. This also fixes a merge conflict remaining in include/configs/EB+MCF-EV123.h which was caused by the same commit Signed-off-by: Albin Tonnerre albin.tonne...@free-electrons.com --- include/configs/EB+MCF-EV123.h |7 --- include/environment.h |1 + 2 files changed, 1 insertions(+), 7 deletions(-) Applied to next branch (and rebased next to clean up the mess). Thanks for catching / fixing this. 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: w...@denx.de You don't have to worry about me. I might have been born yesterday... but I stayed up all night. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
