On Sun, Jul 06, 2008 at 04:11:53PM +0200, Jonathan Worthington wrote: > Patrick R. Michaud wrote: > >Would it hurt for us to temporarily remove :instanceof from > >src/parser/actions.pm (at least for subs) until your patch is > >ready? Otherwise "module Foo;" doesn't really work -- e.g., for mod_perl6. > > If you're ready for two spectests that currently pass to start failing, > you can do that. On the other hand, my patch causes the exact same > failures and but replaces :instanceof with something useful, so if we > can put up with that regression for a few days I may as well just put my > patch in today as it is anyway. (Trying to get some hacking break > today...RSI is going to bite me next week otherwise. Thus why I'd rather > not try and fix it all up today.)
I don't mind having a couple of regression failures for a day or two. We could also "todo/skip" the newly failing spectests until the full patch is available, to avoid confusion for others who may be running spectest_regression. Pm
