Two comments: * The change from ironruby to ruby in the ExecutionContext might break a lang test. You can check with rake mspec:lang. * Should we default the .NET namespaces to do a default require, without the file, but allow that require to be overridden with a file?
Other than that it looks good. -----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of John Lam (IRONRUBY) Sent: Tuesday, October 21, 2008 4:14 PM To: IronRuby External Code Reviewers Cc: ironruby-core@rubyforge.org Subject: [Ironruby-core] Code Review: bugfixes-10 tfpt review "/shelveset:bugfixes-10;REDMOND\jflam" Comment : Fixes find bug, adds thunks for system and system.drawing assemblies and forces ironruby to use MRI gem path _______________________________________________ Ironruby-core mailing list Ironruby-core@rubyforge.org http://rubyforge.org/mailman/listinfo/ironruby-core
