# Code Review: bugfixes-10

**URL:** <https://www.ruby-forum.com/t/code-review-bugfixes-10/151478>\
**Category:** IronRuby\
**Created:** [October 22, 2008, 1:14am UTC](https://www.ruby-forum.com/t/code-review-bugfixes-10/151478 "2008-10-22T01:14:52Z")\
**Posts on this page:** 2\
**Page:** 1

<div class="post-metadata">

**Author:** ![John\_LSSIRONRUBY](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/j/4491bb/32.png) [@John\_LSSIRONRUBY](https://www.ruby-forum.com/u/John_LSSIRONRUBY)\
**Post date:** [October 22, 2008, 1:14am UTC](https://www.ruby-forum.com/t/code-review-bugfixes-10/151478/1 "2008-10-22T01:14:52Z")

</div>

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

---

<div class="post-metadata">

**Author:** ![John\_LSSIRONRUBY](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/j/4491bb/32.png) [@John\_LSSIRONRUBY](https://www.ruby-forum.com/u/John_LSSIRONRUBY)\
**Post date:** [October 22, 2008, 1:21am UTC](https://www.ruby-forum.com/t/code-review-bugfixes-10/151478/2 "2008-10-22T01:21:18Z")

</div>

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.