On Wed, 9 Feb 2005 18:41:35 +0100, "Jos I. Boumans" <[EMAIL PROTECTED]> wrote:
> > On Feb 9, 2005, at 6:11 PM, Rafael Garcia-Suarez wrote: > > > Jos I. Boumans wrote: > >> > >> On Jan 25, 2005, at 12:26 PM, Jos I. Boumans wrote: > >> > >>> Although things don't quite work on vms (yet), here is the patch to > >>> integrate IPC::Run 0.79 with core perl. This allows us to at least > >>> test it widely on other systems as well and is a prereq for CPANPLUS > >>> to function. > >> > >> Here's a second attempt at integrating IPC::Run (now at version 0.80), > >> with the most notable change the location of the modules themselves; > >> they now all live directly in lib/, and their test files in > >> lib/Foo/Bar/t (modeled after attribute::handlers). > >> > >> All tests pass on my osx box and the patch applies cleanly against a > >> recent rsync of the perl sources. > > > > Doesn't work here, due to $^X issues : > > > > [EMAIL PROTECTED] t]$ pwd > > /home/rafael/bleadperl/t > > [EMAIL PROTECTED] t]$ ./perl -le 'print $^X' > > /home/rafael/bleadperl/perl > > > > However if I remove all lines > > $^X = '../../../t/' . $^X; > > from the preambles, it works (with make test and make test_harness) > > here. > > As I don't understand the purpose of this line, I assume it's safe to > > leave it out ? > > It's actually not safe to leave it out... it does some tests of running > 'command line' commands, and it uses $^X for it, which is './perl' on > my box, which is wrong since we chdir'd away from that directory. > > What is the proper way to change this then? File::Spec->rel2abs it > before chdir'ing? Why not Cwd and abspath? -- H.Merijn Brand Amsterdam Perl Mongers (http://amsterdam.pm.org/) using Perl 5.6.2, 5.8.0, 5.8.3, & 5.9.2 on HP-UX 10.20, 11.00 & 11.11, AIX 4.3, SuSE 9.0 pro 2.4.21 & Win2k. http://www.cmve.net/~merijn Smoking perl: [email protected], perl QA: http://qa.perl.org reports to: [EMAIL PROTECTED], [email protected]
