Allison has pointed out one error in the part of the patch referring to lib/Parrot/Configure.pm. This comment, which describes what happens *some* of the time when a particular config step DWIMs, should have been deleted.

    # When successful, a Parrot configuration step returns itself,
    # i.e., returns its own object.

That's because the point of this patch is specifically to *not* return the step object.

I will fix the patch.  Still welcome comments on all the rest of it.

kid51

Reply via email to