Skip to contentSkip to conversation

add lzop dependency

PATCH New Yegor Yefremov 2 participants 4 messages Sep 27, 2010 – Sep 28, 2010

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 add lzop dependency

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

toolchain/dependencies/dependencies.sh +1 -1
Index: b/toolchain/dependencies/dependencies.sh
===================================================================
@@ -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
#1 lore raw

PATCH 1/1 add lzop dependency

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Yegor" == Yegor Yefremov <[email protected]> writes:

Yegor> lzop will be required to compress Linux kernel via lzo compression method. Yegor> Signed-off-by: Yegor Yefremov <[email protected]>

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?

Signature · 1 line
Bye, Peter Korsgaard
#2 lore raw

PATCH 1/1 add lzop dependency

Quoted 8 lines

Yegor> lzop will be required to compress Linux kernel via lzo compression method. Yegor> Signed-off-by: Yegor Yefremov <[email protected]>

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

#3 lore raw

PATCH 1/1 add lzop dependency

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Yegor" == Yegor Yefremov <[email protected]> writes:

Hi,

Quoted 3 lines
Quoted 3 lines

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.

Signature · 1 line
Bye, Peter Korsgaard