Re: [U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-09 Thread Mike Frysinger
On Sunday, November 07, 2010 10:11:14 Sebastien Carlier wrote: > On 11/07/2010 03:16 PM, Peter Tyser wrote: > > I'm guessing lots of boards will have this same issue. I imagine its > > due to include/config_cmd_defaults.h, so maybe if you fix the issue in > > that one place all the compile issues

Re: [U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-07 Thread Wolfgang Denk
Dear Peter Tyser, In message <1289168695.3511.11.ca...@ptyser-laptop> you wrote: > > > > new 100% $(AR)-free version is available here: > > > > Please post your patch on the mailing list for review. > > For what its worth, Sebastien's patch is significantly larger than the > 100K mailing list li

Re: [U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-07 Thread Peter Tyser
Hi Wolfgang, > In message <4cd56618.4010...@gmail.com> you wrote: > > > > My previous patch missed some places that create library archives. A > > new 100% $(AR)-free version is available here: > > Please post your patch on the mailing list for review. For what its worth, Sebastien's patch is

Re: [U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-07 Thread Wolfgang Denk
Dear Sebastien Carlier, In message <4cd56618.4010...@gmail.com> you wrote: > > My previous patch missed some places that create library archives. A > new 100% $(AR)-free version is available here: Please post your patch on the mailing list for review. Best regards, Wolfgang Denk -- DENX Sof

Re: [U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-07 Thread Andreas Bießmann
Dear Sebastien Carlier, Am 07.11.2010 um 17:18 schrieb Sebastien Carlier: > Dear Andreas, > > On 11/07/2010 04:30 PM, Andreas Bießmann wrote: >> I guess the boards are broken before your library changes too. > > The boards in question do actually build with library archives. The > final binary

Re: [U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-07 Thread Sebastien Carlier
Dear Andreas, On 11/07/2010 04:30 PM, Andreas Bießmann wrote: > I guess the boards are broken before your library changes too. The boards in question do actually build with library archives. The final binary does not make use of the NFS stuff, therefore the linker does not need to resolve the un

Re: [U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-07 Thread Andreas Bießmann
Dear Sebastien Carlier, Am 07.11.2010 um 16:11 schrieb Sebastien Carlier: > Dear Peter, > > On 11/07/2010 03:16 PM, Peter Tyser wrote: >> - You shouldn't be making changes to stuff like CONFIG_CMD_NFS in this >> patch. Its unrelated, and should be dealt with in another patch. eg >> your patche

Re: [U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-07 Thread Sebastien Carlier
Dear Peter, On 11/07/2010 03:16 PM, Peter Tyser wrote: > You shouldn't need to send the patch using "git send-email". The patch > is greater than U-Boot's mailing list limit (100k) and posting the patch > on a website is perfectly acceptable. Also, it shouldn't be necessary > to split the patch

Re: [U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-07 Thread Peter Tyser
On Sat, 2010-11-06 at 18:21 +0100, Albert ARIBAUD wrote: > Le 06/11/2010 15:28, Sebastien Carlier a écrit : > > Hello all, > > > > My previous patch missed some places that create library archives. A > > new 100% $(AR)-free version is available here: > > > > http://io.oiioiio.com/~sebc/0001-

Re: [U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-06 Thread Albert ARIBAUD
Le 06/11/2010 15:28, Sebastien Carlier a écrit : > Hello all, > > My previous patch missed some places that create library archives. A > new 100% $(AR)-free version is available here: > > http://io.oiioiio.com/~sebc/0001-Use-partial-linking-v2.patch > > I have tested this patch with MAKEALL

[U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-06 Thread Sebastien Carlier
Hello all, My previous patch missed some places that create library archives. A new 100% $(AR)-free version is available here: http://io.oiioiio.com/~sebc/0001-Use-partial-linking-v2.patch I have tested this patch with MAKEALL -A arm and checked that it does not break any build on this a