[ruby-core:46121] Re: [ruby-trunk - Feature #2565] adding hooks for better tracing

On Tue, Jul 03, 2012 at 01:31:53AM +0900, vo.x (Vit Ondruch) wrote:
> 
> Issue #2565 has been updated by vo.x (Vit Ondruch).
> 
> 
> >  AFAIK, the only open issue (that Vit raised [ruby-core:44980]) is that
> >  we shouldn't check in the dummy probe header file.  I agree with Vit, so
> >  I want to update my patch before merge. :-)
> 
> Well, I proposed two options, either have both versions in SCM or keep it out of the SCM. I'm still in favor of the latter, however I realized later that there is one major disadvantage. If we want to avoid the need of Ruby during build from tarball, as it is possible now, then the release manager has to have available either DTrace or SystemTap on his/her system. Not sure if that might be fulfilled :/ If not, then both files should be pre-generated by its maintainer and stored in SCM.
> 
> In other words, we should think also about associated work-flow and release management.

I don't think the header file should be generated when the tarball is
being built, but when the user installs Ruby (this is how PostgreSQL
does their dtrace headers).

I'm working on a sed script to convert the .d file to a .h file in the
case that the user doesn't have dtrace on their system.  Then the
release manager doesn't need to have Ruby or DTrace on their system when
making the tar.

How does that sound?

-- 
Aaron Patterson
http://tenderlovemaking.com/