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?


-- Jos Boumans

        "Bush is a very stupid man. The American people are not stupid,
        they are very clever. I can't understand how such clever people
        came to elect such a stupid president."  -- Saeed Al Sahaf

        CPANPLUS        http://cpanplus.sf.net



Reply via email to