Return-Path: Delivered-To: apmail-httpd-test-dev-archive@www.apache.org Received: (qmail 34979 invoked from network); 15 Apr 2004 21:14:29 -0000 Received: from daedalus.apache.org (HELO mail.apache.org) (208.185.179.12) by minotaur-2.apache.org with SMTP; 15 Apr 2004 21:14:29 -0000 Received: (qmail 30370 invoked by uid 500); 15 Apr 2004 21:14:16 -0000 Delivered-To: apmail-httpd-test-dev-archive@httpd.apache.org Received: (qmail 30337 invoked by uid 500); 15 Apr 2004 21:14:16 -0000 Mailing-List: contact test-dev-help@httpd.apache.org; run by ezmlm Precedence: bulk Reply-To: test-dev@httpd.apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list test-dev@httpd.apache.org Received: (qmail 30320 invoked from network); 15 Apr 2004 21:14:15 -0000 Received: from unknown (HELO Boron.MeepZor.Com) (204.146.167.214) by daedalus.apache.org with SMTP; 15 Apr 2004 21:14:15 -0000 Received: from Golux.Com (dsl093-240-231.ral1.dsl.speakeasy.net [66.93.240.231]) (authenticated) by Boron.MeepZor.Com (8.11.6/8.11.6) with ESMTP id i3FLEM614175; Thu, 15 Apr 2004 17:14:22 -0400 Message-ID: <407EFB0A.1070603@Golux.Com> Date: Thu, 15 Apr 2004 17:13:46 -0400 From: Rodent of Unusual Size Organization: The Apache Software Foundation User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.6) Gecko/20040113 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Apache test suite Subject: qw() and embedded spaces in filname paths Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N X-Spam-Rating: minotaur-2.apache.org 1.6.2 0/1000/N okey, bug found. noticed on windows because spaces in filenames are more common there, but it happens anywhere. here's the deal. this command: perl Makefile.PL -httpd "C:/A B/apache.exe" -apxs "C:/A B/apxs" results in the following being put into t/TEST %Apache::testConfig::Argv = qw(httpd C:\A B\apache.exe apxs C:\A B\apxs) so, duh, the spaces lose their significance because qw() is treating the whitespace as delimiters. here is a *really* rough patch that should fix this. it needs to be cleaned up by some perl guru, and probably applied to other files that do this as well (like TestMM.pm). comments? Index: Apache-Test/lib/Apache/TestRun.pm =================================================================== RCS file: /home/cvs/httpd-test/perl-framework/Apache-Test/lib/Apache/TestRun.pm,v retrieving revision 1.163 diff -u -r1.163 TestRun.pm --- Apache-Test/lib/Apache/TestRun.pm 5 Apr 2004 04:35:21 -0000 1.163 +++ Apache-Test/lib/Apache/TestRun.pm 15 Apr 2004 21:11:36 -0000 @@ -1221,8 +1221,19 @@ $body .= Apache::TestConfig->modperl_2_inc_fixup; - if (@Apache::TestMM::Argv) { - $body .= "\n\%Apache::TestConfig::Argv = qw(@Apache::TestMM::Argv);\n"; + if (my @args = @Apache::TestMM::Argv) { + $body .= "\n\%Apache::TestConfig::Argv = ("; + my ($k, $v); + while (@args) { + $k = shift @args; + $k =~ s/'/\\'/g; + $body .= "'$k' => "; + $v = shift @args; + $v =~ s/'/\\'/g; + $body .= "'$v'"; + $body .= ', ' if @args; + } + $body .= ");\n"; } my $header = Apache::TestConfig->perlscript_header; -- #ken P-)} Ken Coar, Sanagendamgagwedweinini http://Ken.Coar.Org/ Author, developer, opinionist http://Apache-Server.Com/ "Millennium hand and shrimp!"