Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Inline CPAN distribution.

Report information
The Basics
Id: 4150
Status: resolved
Priority: 0/
Queue: Inline

People
Owner: Nobody in particular
Requestors: corion [...] cpan.org
Cc: TONYC [...] cpan.org
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: (no value)



Subject: Inline.pm fails (tests) when the build directory contains spaces
The tests fail due to manual building of a string to pass to the shell, for example when the .cpan directory is located in C:\Dokumente und Einstellungen\Max\.cpan (as is the default under Windows 2000). Here's the patch to fix my problems with Inline. The problems extend to Inline/C.pm, with which I'm still struggling, and also into ExtUtils::MakeMaker, which I'll also submit patches for. If I move the build directory to a directory without a space in it, everything works fine. --- Inline.pm.old 2002-10-28 08:25:19.000000000 +0100 +++ Inline.pm 2003-10-17 22:35:00.000000000 +0200 @@ -740,11 +740,15 @@ sub create_config_file { my($v,$d,$f) = File::Spec->splitpath($inline); $f = "" if $f eq 'Inline.pm'; $inline = File::Spec->catpath($v,$d,$f); - my $INC = "-I$inline -I" . - join(" -I", grep {(-d File::Spec->catdir($_,"Inline") or - -d File::Spec->catdir($_,"auto","Inline") - )} @INC); - system "$perl $INC -MInline=_CONFIG_ -e1 $dir" + #my $INC = "-I$inline -I" . + # join(" -I", grep {(-d File::Spec->catdir($_,"Inline") or + # -d File::Spec->catdir($_,"auto","Inline") + # )} @INC); + #system "$perl $INC -MInline=_CONFIG_ -e1 $dir" + my @INC = map { "-I$_" } + ($inline, + grep {(-d File::Spec->catdir($_,"Inline") or -d File::Spec-> catdir($_,"auto","Inline"))} @INC); + system $perl, @INC, "-MInline=_CONFIG_", "-e1", "$dir" and croak M20_config_creation_failed($dir); return; } @@ -929,7 +933,7 @@ sub write_inl_file { File::Spec->catfile($o->{API}{install_lib},"auto",$o->{API}{modpname}, "$o->{API}{modfname}.inl"); open INL, "> $inl" - or croak "Can't create Inline validation file $inl"; + or croak "Can't create Inline validation file $inl: $!"; my $apiversion = $Config{apiversion} || $Config{xs_apiversion}; print INL Inline::denter->new() ->indent(*md5, $o->{INLINE}{md5},
RT-Send-CC: sisyphus [...] cpan.org
This has, I believe, been addressed in Inline-0.45 (patch applied), though it's untested by the test suite. (There are, as noted, other aspects to be dealt with before the problem is fixed.) If there's a problem with the way this has been dealt with, please feel free to re-open the ticket (or submit a new report). To further discuss the issue (but without re-opening the ticket), consider posting to the Inline mailing list, or send a private email to sisyphus at cpan dot org. Thanks for the report !! Cheers, Rob