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

[Luigi Mantellini]

Ok... Now I understand.

We have two choices: CPPFLAGS or patch the LzmaDec.c.
I think that you prefer the second choice. right?

best regards,

luigi

On Mon, Jul 27, 2009 at 9:33 PM, Mike Frysinger<vapier at gentoo.org> wrote:
> On Monday 27 July 2009 15:12:11 Luigi Mantellini wrote:
>> On Mon, Jul 27, 2009 at 8:58 PM, Mike Frysinger<vapier at gentoo.org> wrote:
>> >> CFLAGS bugs?
>> >> I tried to comment the -I...blablalb... and the build (crrectly) fails.
>> >> Can you remove the .depend from your source dir?
>> >
>> > i said it doesnt matter.  but if you're going to continue down this line
>> > of reasoning anyways, i just did `git clean -x -d` and got the same
>> > result.
>>
>> Very strange...which compiler are you using?
>> I'm using a gcc4.3.3 (buitl using openwrt trunk)
>
> the gcc doesnt matter (but i am using 4.3.3 too).  i already pointed out the
> issue -- the common .depend code does not use $(CFLAGS), and the lzma code
> isnt using proper paths.
>
> just change the *1* include and drop the -I path hack and be done with it.
>
> --- a/lib_generic/lzma/LzmaDec.c
> +++ b/lib_generic/lzma/LzmaDec.c
> @@ -9 +9 @@
> -#include <string.h>
> +#include <linux/string.h>
> --- a/lib_generic/lzma/Makefile
> +++ b/lib_generic/lzma/Makefile
> @@ -33 +33 @@ SOBJS =
> -CFLAGS += -D_LZMA_PROB32 -I$(TOPDIR)/include/linux
> +CFLAGS += -D_LZMA_PROB32
>
>> this is my .depend (after a git clean -x -d -f and after a make mrproper):
>>
>> LzmaTools.o: LzmaTools.c /mnt/devel/onu/u-boot-onu/include/config.h \
>
> i said LzmaDec.c, not LzmaTools.c
>
> just run:
> $ touch lib_generic/lzma/LzmaDec.c
> $ make depend SUBDIRS=lib_generic/lzma
> for dir in lib_generic/lzma ; do make -C $dir _depend ; done
> 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
> make[1]: Leaving directory `/usr/local/src/u-boot/blackfin/lib_generic/lzma'
> make[1]: Entering directory `/usr/local/src/u-boot/blackfin/lib_generic/lzma'
> make[1]: Nothing to be done for `_depend'.
> make[1]: Leaving directory `/usr/local/src/u-boot/blackfin/lib_generic/lzma'
> -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