Fix for missing parrot phony.exe under cygwin per RT #113992 #74

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@ronaldxs
ronaldxs commented Jul 4, 2012

No description provided.

@pmichaud
Collaborator
pmichaud commented Jul 4, 2012

The pull request puts the fix in the wrong place, unfortunately. First, I think the fix belongs in Parrot, not Rakudo -- see updated comments in RT #113992.

Even if it turns out that Rakudo needs to supply a workaround, this is going to be the wrong place to do it. The NQP::Configure module is intended to be shared by NQP, Rakudo, and other languages to be built on top of NQP, so doing repairs of Parrot's tarball hopefully doesn't belong in this (shared) library. I'd be more likely to accept it as a fix in Star's Configure.pl (but I still think it's a Parrot bug that should be fixed in the Parrot tarball).

Pm

@pmichaud pmichaud closed this Jul 4, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment