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
---------------------------------------------------
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.