You are viewing a plain text version of this content. The canonical link for it is here.
Posted to apreq-dev@httpd.apache.org by "Philip M. Gollucci" <pg...@p6m7g8.com> on 2005/04/08 09:56:07 UTC
RE: [PATCH[ build/version_check.pl
Uwe Voelker wrote:
>> And a sidenote to libapreq2: If libtool is not installed there is a
>> misleading error message:
>>
>> > ./buildconf
>> removing stale config files
>> checking buildconf prereqs
>> build/version_check.pl failed: no version_string found in ''.
>>
>> Maybe it should output the name of programm it is testing (libtool in
>> this case).
Hi, as promised, patch to output the tool in question.
It fixed a typo using $_ instead of $tool in the string
Index: build/version_check.pl
===================================================================
--- build/version_check.pl (revision 160526)
+++ build/version_check.pl (working copy)
@@ -133,7 +133,7 @@
my @version = split /\./, $version;
$_ = $prereq{$tool}->{test}->();
-die "$0 failed: no version_string found in '$_'.\n" unless /(\d[.\d]+)/;
+die "$0 failed: no version_string found in '$_' for '$tool'.\n" unless
/(\d[.\d]+)/;
my $saw = $1;
my @saw = split /\./, $saw;
$_ = 0 for @saw[@saw .. $#version]; # ensure @saw has enough entries
END
------------------------------------------------------
Philip M. Gollucci (pgollucci@p6m7g8.com) 301.254.5198
Consultant / http://p6m7g8.net/Resume/resume.shtml