[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
RE: (ITS#3460) Adding LDAP statistics script to contrib directory
> Do you mind if I do some clean up on it?
>
> Mainly Perl::Critic and Perl::Tidy. There's a few tests missing and
> assumptions been made with opening files and system commands etc.
You might also highlight the fact that it seems to require at least 5.8
(due to the "qw()" lists).
I also don't like hard-wiring "/usr/bin/perl", as the system Perls are
rarely up to date. My favourite technique is:
#! /usr/bin/env perl -w
which finds Perl from $PATH.
Or just hard-wire "/usr/local/bin/perl" and let the sysadmin sort it
out...
--
Dave Horsfall DTM VK2KFU daveh@ci.com.au Ph: +61 2 9552-5509 (d) -5500 (sw)
Corinthian Eng'ng P/L, Ste 54 Jones Bay Whf, 26-32 Pirrama Rd, Pyrmont 2009, AU