# Code Review: compilefix

**URL:** <https://www.ruby-forum.com/t/code-review-compilefix/158506>\
**Category:** IronRuby\
**Created:** [January 24, 2009, 12:42am UTC](https://www.ruby-forum.com/t/code-review-compilefix/158506 "2009-01-24T00:42:10Z")\
**Posts on this page:** 3\
**Page:** 1

<div class="post-metadata">

**Author:** ![raker\_t](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/r/ecae2f/32.png) [@raker\_t](https://www.ruby-forum.com/u/raker_t)\
**Post date:** [January 24, 2009, 12:42am UTC](https://www.ruby-forum.com/t/code-review-compilefix/158506/1 "2009-01-24T00:42:10Z")

</div>

tfpt review “/shelveset:compilefix;REDMOND\jdeville”  
Comment :  
This shelveset removes the project transforms from the Ruby rakefiles.  
This means that there is no longer a need to set MERLIN\_ROOT for  
external contributors.

---

<div class="post-metadata">

**Author:** ![raker\_t](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/r/ecae2f/32.png) [@raker\_t](https://www.ruby-forum.com/u/raker_t)\
**Post date:** [January 28, 2009, 1:14am UTC](https://www.ruby-forum.com/t/code-review-compilefix/158506/2 "2009-01-28T01:14:10Z")

</div>

Ah deleting code, I love it. And goodbye IronRuby.source\_context … =)

Looks good.  
~js

---

<div class="post-metadata">

**Author:** ![raker\_t](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/r/ecae2f/32.png) [@raker\_t](https://www.ruby-forum.com/u/raker_t)\
**Post date:** [January 28, 2009, 1:18am UTC](https://www.ruby-forum.com/t/code-review-compilefix/158506/3 "2009-01-28T01:18:16Z")

</div>

Sweet, going in as direct commit, followed by a push for external 😉

JD