ingydotnet / io-all-pm

All in One Perl IO
http://search.cpan.org/dist/IO-All/
38 stars 20 forks source link

t/pipe.t fails if the perl binary can't be found as 'perl' #35

Closed Hugmeir closed 10 years ago

Hugmeir commented 10 years ago

This line:

https://github.com/ingydotnet/io-all-pm/blob/65c8640738137c59df9d0e5e66bc0d60c9bf40d3/t/pipe.t#L7

Assumes that the perl binary can be found as 'perl'. This isn't always true; a simple example is a perl built without -Uversiononly, which will leave a binary named something like perl5.x.y.

Solution is trivial: like the following test, just use $^X instead of 'perl'

shlomif commented 10 years ago

Hi Hugemeir, this should be fixed in this commit: https://github.com/ingydotnet/io-all-pm/commit/89bf1e776e3cc1cc668d0c8540f5fccbe1d00fcb - please check and let us know if it works for you.

frioux commented 10 years ago

@shlomif clearly that fixes it. I'm making a release now to include it.