Bugzilla – Bug 4307
user reported build-perl-modules.pl bug
Last modified: 2008-12-18 11:11:39 UTC
It looks like a bug was introduced in slimserver's Bin/build-perl-modules.pl script in revision 8759. The script stores the text "lwp", or the full path of curl or wget (from File::Which) in $downloadUsing. The script later compares $downloadUsing to 'curl', which will almost never match since $downloadUsing contains the full path of curl or wget, not just 'curl' or 'wget'. Below is a quick change I used with success on my system, but there are probably more robust solutions. The change that introduced the problem can be seen here: <http://svn.slimdevices.com/trunk/server/Bin/build-perl-modules.pl?rev=8759&r1=8712&r2=8759>< /a> -Michael --- Bin/build-perl-modules.pl.orig 2006-10-04 21:29:15.000000000 -0700 +++ Bin/build-perl-modules.pl 2006-10-04 18:00:41.000000000 -0700 @@ -198,7 +198,7 @@ LWP::Simple::getstore("$SOURCE/$package?view=auto", $package); - } elsif ($downloadUsing eq 'curl') { + } elsif ($downloadUsing =~ /curl$/) { `$downloadUsing --silent -o $package $SOURCE/$package?view=auto`;
Fixed in change 10291