Skip Menu |

This queue is for tickets about the Test-Harness CPAN distribution.

Report information
The Basics
Id: 49417
Status: resolved
Priority: 0/
Queue: Test-Harness
People
Owner: Nobody in particular
Requestors: kmx [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 3.17
Fixed in: (no value)

History Show all quoted text
  Fri Sep 04 00:01:03 2009 kmx [...] cpan.org - Ticket created
Subject: [PATCH] Handling spaces in path to perl executable
Hi, I would like to propose the following patch to lib\Test\Harness.pm: - chomp( @inc = `$perl -e "print join qq[\\n], \@INC, q[]"` ); + chomp( @inc = `"$perl" -e "print join qq[\\n], \@INC, q[]"` ); It handles a situation when perl executable path (in $perl variable) contains spaces. I have tested the patch with Test-Harness-3.17 (on Win32/strawberry perl). Thanks in advance for considering the proposed patch. -- kmx
  Sun Jan 17 16:11:46 2010 andy [...] hexten.net - Correspondence added
On Fri Sep 04 00:01:03 2009, KMX wrote: Show quoted text
> Hi, > > I would like to propose the following patch to lib\Test\Harness.pm: > - chomp( @inc = `$perl -e "print join qq[\\n], \@INC, q[]"` ); > + chomp( @inc = `"$perl" -e "print join qq[\\n], \@INC, q[]"` ); > > It handles a situation when perl executable path (in $perl variable) > contains spaces. > > I have tested the patch with Test-Harness-3.17 (on Win32/strawberry perl). > > Thanks in advance for considering the proposed patch. > > -- > kmx >
Applied, thanks!
  Sun Jan 17 16:11:47 2010 The RT System itself - Status changed from 'new' to 'open'
  Sun Jan 17 16:11:48 2010 andy [...] hexten.net - Status changed from 'open' to 'resolved'