Re: r11190 again! 0.2808_04 will not install in Strawberry Perl 5.10.0.2...
[email protected] (Eric Wilhelm)
| Newsgroups | perl.module.build |
|---|---|
| Message-ID | <[email protected]> |
# from Curtis Jewell
# on Monday 15 September 2008 11:13:
> DB<7> p $cmd
>'C:\strawberry\perl\bin\perl.exe' -MConfig=myconfig -e print -e
> myconfig
>
>(At this point, the first parameter needs to be double-quoted, not
>single quoted, and that's when we hit this error.)
>
> ...
>
> DB<7> p $quoted[0]
>"C:\strawberry\perl\bin\perl.exe"
>
>Line 344 is causing it, does this help? I was able to force the
>installation in the debugger (of course, bunches of tests fail) but
>figure you should know.
Well, r11190 is turning out to be a real pain.
"Better shell quoting and testing thereof"
IMO, the backticks are going to be trouble no matter what. Should we
fight the quoting, or use a list context and a pipe?
Would a simple piped-open to $^X work everywhere, or do we have to do
something more heavyweight?
I would lean toward something like IPC::Cmd (which, IIRC CPANPLUS
already depends on - and is core starting with 5.9.5.), but that means
potentially needing to bundle it or etc. Alternatively, we can
conditionally use it wherever we have it (which means anyone who hits
backtick quoting bugs can simply install IPC::Cmd and move on.)
--Eric
--
"...our schools have been scientifically designed to
prevent overeducation from happening."
--William Troy Harris
---------------------------------------------------
http://scratchcomputing.com
---------------------------------------------------