I'm tidying up the files as requested.  I've run a make test, which passes.<br>I haven't found any DTrace patches for the 1.9 series of ruby.  The SFE build of 1.9.0 doesn't have any applied.<br>I'd suggest having two ruby packages in the repo: ruby19 with latest code and ruby18 with dtrace.<br>
<br>Any thoughts?<br><br><div class="gmail_quote">On Sat, Nov 19, 2011 at 10:35 PM, Alasdair Lumsden <span dir="ltr"><<a href="mailto:alasdairrr@gmail.com">alasdairrr@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi Colin,<br>
<br>
Great stuff! Only minor nitpicks:<br>
<br>
On the Makefile, you've got multiple configure options per CONFIGURE_OPTION, and the indentation of the macros is a bit iffy. But if you like I can tidy these up when we commit, saving you the bother.<br>
<br>
Did you have any luck finding out about the DTrace patches? Also did you give make test a go?<br>
<br>
Cheers,<br>
<br>
Alasdair<div class="im"><br>
<br>
On 11/19/11 09:54 PM, Colin Ellis wrote:<br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">
<a href="https://bitbucket.org/cellis_oidev/oi-build/changeset/4c13f4869651" target="_blank">https://bitbucket.org/cellis_<u></u>oidev/oi-build/changeset/<u></u>4c13f4869651</a><br>
<br>
On Tue, Nov 15, 2011 at 9:32 AM, Guido Berhoerster <<a href="mailto:gber@openindiana.org" target="_blank">gber@openindiana.org</a><br></div>
<mailto:<a href="mailto:gber@openindiana.org" target="_blank">gber@openindiana.org</a>>> wrote:<br>
<br>
    * Colin Ellis <<a href="mailto:panamayacht@gmail.com" target="_blank">panamayacht@gmail.com</a> <mailto:<a href="mailto:panamayacht@gmail.com" target="_blank">panamayacht@gmail.com</a>><u></u>><div class="im">
<br>
    [2011-11-12 19:40]:<br>
     > Fixes are here:<br>
     ><br>
     > <a href="https://bitbucket.org/cellis_oidev/oi-build/changeset/5bb1edf68a1d" target="_blank">https://bitbucket.org/cellis_<u></u>oidev/oi-build/changeset/<u></u>5bb1edf68a1d</a><br>
<br>
    Have you looked through the ruby 1.8 patches and checked which<br>
    are still applicable? At least the DTrace changes should probably<br>
    be ported over.<br>
    Have you checked the built ruby interpeter? Running "make test"<br>
    on the source tree after building should be an easy way to catch<br>
    any issues.<br>
    --<br>
    Guido Berhoerster<br>
<br>
    ______________________________<u></u>_________________<br>
    oi-dev mailing list<br></div>
    <a href="mailto:oi-dev@openindiana.org" target="_blank">oi-dev@openindiana.org</a> <mailto:<a href="mailto:oi-dev@openindiana.org" target="_blank">oi-dev@openindiana.org</a><u></u>><br>
    <a href="http://openindiana.org/mailman/listinfo/oi-dev" target="_blank">http://openindiana.org/<u></u>mailman/listinfo/oi-dev</a><div class="im"><br>
<br>
<br>
<br>
<br>
______________________________<u></u>_________________<br>
oi-dev mailing list<br>
<a href="mailto:oi-dev@openindiana.org" target="_blank">oi-dev@openindiana.org</a><br>
<a href="http://openindiana.org/mailman/listinfo/oi-dev" target="_blank">http://openindiana.org/<u></u>mailman/listinfo/oi-dev</a><br>
</div></blockquote><div><div></div><div class="h5">
<br>
<br>
______________________________<u></u>_________________<br>
oi-dev mailing list<br>
<a href="mailto:oi-dev@openindiana.org" target="_blank">oi-dev@openindiana.org</a><br>
<a href="http://openindiana.org/mailman/listinfo/oi-dev" target="_blank">http://openindiana.org/<u></u>mailman/listinfo/oi-dev</a><br>
</div></div></blockquote></div><br>