Re: [Ltib] Ltibutils.pm: test for zlib installation

[Stuart Hughes]

Hi Mike,

I can't remember whether or not that was deliberate.  I think it was.
The idea is that it's looking for a symlink and the file it points to,
so there should be 2 (at least).

Regards, Stuart

On 21/06/13 00:06, Mike Goins wrote:
> On Thu, Jun 20, 2013 at 3:29 PM, Stuart Hughes <address@hidden> wrote:
>> Hi Devin,
>>
>> The idea is to check all the possible locations, not just
>> '/usr/lib/x86_64-linux-gnu/libz.so*' because the location varies from
>> distribution to distribution.
>>
>> Regards, Stuart
> 
> 
> I think the OP implied:
> 
> Index: bin/Ltibutils.pm
> ===================================================================
> RCS file: /sources/ltib/ltib/bin/Ltibutils.pm,v
> retrieving revision 1.38
> diff -u -r1.38 Ltibutils.pm
> --- bin/Ltibutils.pm 7 Aug 2011 15:05:53 -0000 1.38
> +++ bin/Ltibutils.pm 20 Jun 2013 23:04:22 -0000
> @@ -591,7 +591,7 @@
>                                     glob('/lib/i386-linux-gnu/libz.so*'),
>                                     
> glob('/usr/lib/x86_64-linux-gnu/libz.so*'),
>                                     glob('/lib/x86_64-linux-gnu/libz.so*'));
> -                                   @f > 1 ? 1 : 0 },
> +                                   @f > 0 ? 1 : 0 },
>      'zlib-devel' => sub { -f '/usr/include/zlib.h' },
>  };
> 
> 
> 
>> On 19/06/13 20:53, Sale, Devin M wrote:
>>> Hello,
>>>
>>> In Ltibutils.pm there's a routine to check for the existence of libz.so* in 
>>> a list of paths:
>>>
>>>     zlib         => sub { my @f = (glob('/usr/lib/libz.so*'),
>>>                                    glob('/lib/libz.so*'),
>>>                                    glob('/lib64/libz.so*'),
>>>                                    glob('/usr/lib/i386-linux-gnu/libz.so*'),
>>>                                    glob('/usr/lib32/libz.so*'),
>>>                                    
>>> glob('/usr/lib/x86_64-linux-gnu/libz.so*') ); @f > 1 ? 1 : 0 },
>>>
>>> Shouldn't the check be
>>>                                    
>>> glob('/usr/lib/x86_64-linux-gnu/libz.so*') ); @f > 0 ? 1 : 0 },
>>>
>>> Since only one instance of libz is required.
>>>
>>> Devin
>>> ****************************************************************************************
>>>  Note: If the reader of this message is not the intended recipient, or an 
>>> employee or agent responsible for delivering this message to the intended 
>>> recipient, you are hereby notified that any dissemination, distribution or 
>>> copying of this communication is strictly prohibited. If you have received 
>>> this communication in error, please notify us immediately by replying to 
>>> the message and deleting it from your computer. Thank you. 
>>> ****************************************************************************************
>>>
>>>
>>> _______________________________________________
>>> LTIB home page: http://ltib.org
>>>
>>> Ltib mailing list
>>> address@hidden
>>> https://lists.nongnu.org/mailman/listinfo/ltib
>>>
>>
>> _______________________________________________
>> LTIB home page: http://ltib.org
>>
>> Ltib mailing list
>> address@hidden
>> https://lists.nongnu.org/mailman/listinfo/ltib
> 
> _______________________________________________
> LTIB home page: http://ltib.org
> 
> Ltib mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/ltib
>