Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Compress-LZO CPAN distribution.

Report information
The Basics
Id: 46544
Status: open
Priority: 0/
Queue: Compress-LZO

People
Owner: Nobody in particular
Requestors: iwoodhead [...] cpan.org
Cc: kmx [...] cpan.org
AdminCc:

Bug Information
Severity: Important
Broken in: 1.08
Fixed in: (no value)



Subject: Old liblzo version in Makefile.PL breaks installation.
The Makefile.PL for Compress::LZO refers to the older liblzo.so instead of liblzo2.so of the current library. This can be fixed with one char edit. The writemakefile currently: WriteMakefile( NAME => 'Compress::LZO', VERSION_FROM => 'LZO.pm', LIBS => [ "$LZO_LIB -llzo" ], INC => "$LZO_INCLUDE", 'dist' => {COMPRESS=>'gzip', SUFFIX=>'gz'}, ); The LIBS line should read: LIBS => [ "$LZO_LIB -llzo2" ], After this change make test succeeds.
Hi, I just want to support IWOODHEAD's proposal. Please consider applying the enclosed simple patch that fixes -lzo2 issue + sets $LZO_INCLUDE based on $Config{'usrinc'} value. Thanks. -- kmx
diff -r -u Compress-LZO-1.08/Makefile.PL Compress-LZO-1.08_patched/Makefile.PL --- Compress-LZO-1.08/Makefile.PL 1998-08-23 13:14:42.000000000 +0200 +++ Compress-LZO-1.08_patched/Makefile.PL 2009-11-28 11:39:54.943474300 +0100 @@ -5,6 +5,7 @@ # Change the path on the next line to point to the directory where # <lzoconf.h> and <lzo1x.h> can be found. +$LZO_INCLUDE = '-I'.$Config{'usrinc'}.'/lzo'; #$LZO_INCLUDE = '-I/usr/include/lzo'; #$LZO_INCLUDE = '-I/usr/local/include'; #$LZO_INCLUDE = '-I/usr/local/include/lzo'; @@ -21,7 +22,7 @@ WriteMakefile( NAME => 'Compress::LZO', VERSION_FROM => 'LZO.pm', - LIBS => [ "$LZO_LIB -llzo" ], + LIBS => [ "$LZO_LIB -llzo2" ], INC => "$LZO_INCLUDE", 'dist' => {COMPRESS=>'gzip', SUFFIX=>'gz'}, );
RT-Send-CC: cpan [...] oberhumer.com
Hi,

can I do something to help you releasing updated version with the patch I have proposed in my previous bug report?

--
kmx

On 2011-05-05 09:53:30, KMX wrote: Show quoted text
> Hi, > > can I do something to help you releasing updated version with the > patch I have > proposed in my previous bug report?
Hi kmx, maybe adopt the module? There's no release since 2002, so it looks abandoned by the author. Regards, Slaven