[U-Boot] [PATCH] Refresh LZMA-lib to v4.65

[Luigi Mantellini]

(sent again)


CFLAGS bugs?
I tried to comment the -I...blablalb... and the build (crrectly) fails.
Can you remove the .depend from your source dir?

ciao


On Mon, Jul 27, 2009 at 8:27 PM, Mike Frysinger<vapier at gentoo.org> wrote:
> On Monday 27 July 2009 14:17:44 Luigi Mantellini wrote:
>> On Mon, Jul 27, 2009 at 7:40 PM, Mike Frysinger wrote:
>> > On Tuesday 21 July 2009 04:45:49 Luigi 'Comio' Mantellini wrote:
>> >> --- /dev/null
>> >> +++ b/lib_generic/lzma/LzmaDec.c
>> >> @@ -0,0 +1,1007 @@
>> >> +/* LzmaDec.c -- LZMA Decoder
>> >> +2008-11-06 : Igor Pavlov : Public domain */
>> >> +
>> >> +#include "LzmaDec.h"
>> >> +
>> >> +#include <string.h>
>> >
>> > there is no string.h in the source tree.  because of this, the .depend
>> > step fails:
>> > make[1]: Entering directory
>> > `/usr/local/src/u-boot/blackfin/lib_generic/lzma' LzmaDec.c:9:20: error:
>> > string.h: No such file or directory
>> >
>> > i know there's a hack in the Makefile:
>> > CFLAGS += -D_LZMA_PROB32 -I$(TOPDIR)/include/linux
>> >
>> > looks like this should be CPPFLAGS, not CFLAGS.  or fix the source and
>> > drop the -I hack.
>>
>> I preferred to not touch the lzma sourcers. Anyway, on my system the
>> hack works fine... can you try to do a "make mrproper && make
>> your_board_config" before to compile?
>
> this is using MAKEALL, so that should be sufficient.  at any rate, if you
> simply read the .depend target you'll see CFLAGS is not used (whether that's a
> bug in .depend is a different discussion).
> -mike
>



-- 
Luigi 'Comio' Mantellini
R&D - Software
Industrie Dial Face S.p.A.
Via Canzo, 4
20068 Peschiera Borromeo (MI), Italy

Tel.: +39 02 5167 2813
Fax: +39 02 5167 2459
web: www.idf-hit.com
mail: luigi.mantellini at idf-hit.com