[Buildroot] [PATCH] add lzop dependency

[Buildroot] [PATCH] add lzop dependency

[Yegor Yefremov]

lzop will be required to compress Linux kernel via lzo compression method.

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

Index: b/toolchain/dependencies/dependencies.sh
===================================================================
--- a/toolchain/dependencies/dependencies.sh	2010-08-26 16:37:08.000000000 +0200
+++ b/toolchain/dependencies/dependencies.sh	2010-09-27 09:52:29.000000000 +0200
@@ -137,7 +137,7 @@
 fi;
 
 # Check that a few mandatory programs are installed
-for prog in awk bison flex msgfmt makeinfo patch gzip bzip2 perl tar wget cpio python svn ; do
+for prog in awk bison flex msgfmt makeinfo patch gzip bzip2 perl tar wget cpio python svn lzop ; do
     if ! which $prog > /dev/null ; then
 	/bin/echo -e "\nYou must install '$prog' on your build machine";
 	if test $prog = "makeinfo" ; then

[Buildroot] [PATCH] add lzop dependency

[Peter Korsgaard]

>>>>> "Yegor" == Yegor Yefremov <yegor_sub1@visionsystems.de> writes:

 Yegor> lzop will be required to compress Linux kernel via lzo compression method.
 Yegor> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

Lzop isn't that common and it's only needed for a fairly specific setup,
wouldn't it make more sense to build it if needed using the host
infrastructure we have instead?

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH] add lzop dependency

[Yegor Yefremov]

>  Yegor> lzop will be required to compress Linux kernel via lzo compression method.
>  Yegor> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
>
> Lzop isn't that common and it's only needed for a fairly specific setup,
> wouldn't it make more sense to build it if needed using the host
> infrastructure we have instead?

Yes, it makes sense. I'll rewrite the patch. What do you think about letting user to specify kernel compression:
Kernel->Compression method->gzip
                                                    ->lzma
                                                    ->lzo

Then I could combine these two issues.

Regards,
Yegor

[Buildroot] [PATCH] add lzop dependency

[Peter Korsgaard]

>>>>> "Yegor" == Yegor Yefremov <yegor_sub1@visionsystems.de> writes:

Hi,

 >> Lzop isn't that common and it's only needed for a fairly specific setup,
 >> wouldn't it make more sense to build it if needed using the host
 >> infrastructure we have instead?

 Yegor> Yes, it makes sense. I'll rewrite the patch. What do you think
 Yegor> about letting user to specify kernel compression:

That makes sense on ARM when zImage is selected, but might not for other
archs. From a quick look in the kernel sources it seems to be available
on:

arm, blackfin, mips, s390, sh, x86

So atleast ppc, avr32 and sparc are missing.

-- 
Bye, Peter Korsgaard