From: "Vít Ondruch" Date: 2012-11-16T23:28:02+09:00 Subject: [ruby-core:49422] Re: [ruby-cvs:44801] tenderlove:r37631 (trunk): * probes.d: add DTrace probe declarations. --20cf303b3c11186bf804ce9d9394 Content-Type: text/plain; charset=ISO-8859-1 2012/11/13 NARUSE, Yui > 2012/11/13 U.Nakamura : > > Hello, > > > > In message "[ruby-core:49282] Re: [ruby-cvs:44801] tenderlove:r37631 > (trunk): * probes.d: add DTrace probe declarations." > > on Nov.13,2012 12:56:23, wrote: > >> This breaks mswin build. > >> I CANNOT accept that ruby distribution depends on the existance of sed. > >> > >> I wait for correction of this problem for 24 hours. > >> If not corrected, I'll revert theis commit. > > > > Now naruse-san and me fixed most of the problems, so I withdrow > > the plan to revert. > > > > But I think there is still a serious problem. > > Now building ruby from snapshot needs a running ruby, I guess. > > It must be solved before preview2 release. > > It also fixed by r37640. > > -- > NARUSE, Yui > > Not completely. If you have no dtrace/systemtap on your system, then you need to have baseruby (testing with r37681): echo executable host ruby is required. use --with-baseruby option.; false ./tool/gen_dummy_probes.rb ./probes.d > dmyprobes.h executable host ruby is required. use --with-baseruby option. make: *** [dmyprobes.h] Error 1 That was the reason why Aaron proposed to use sed instead of Ruby [1]. Btw if you want to use Ruby, there is better version [2] then directly rewriting sed into Ruby regexps as in r37636. Vit [1] http://bugs.ruby-lang.org/issues/2565#note-40 [2] http://bugs.ruby-lang.org/attachments/2684/0001-Generate-probes.h-from-probes.d.patch --20cf303b3c11186bf804ce9d9394 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable

2012/11/13 NARUSE, Yui <naruse@airemix.= jp>
2012/11/13 U.Nakamura <usa@garb= agecollect.jp>:
> Hello,
>
> In message "[ruby-core:49282] Re: [ruby-cvs:44801] tenderlove:r37= 631 (trunk): * probes.d: add DTrace probe declarations."
> =A0 =A0 on Nov.13,2012 12:56:23, <usa@garbagecollect.jp> wrote:
>> This breaks mswin build.
>> I CANNOT accept that ruby distribution depends on the existance of= sed.
>>
>> I wait for correction of this problem for 24 hours.
>> If not corrected, I'll revert theis commit.
>
> Now naruse-san and me fixed most of the problems, so I withdrow
> the plan to revert.
>
> But I think there is still a serious problem.
> Now building ruby from snapshot needs a running ruby, I guess.
> It must be solved before preview2 release.

It also fixed by r37640.

--
NARUSE, Yui =A0<naruse@airemix.jp>



Not completely. If you have no dtr= ace/systemtap on your system, then you need to have baseruby (testing with = r37681):

echo executable host ruby is required.=A0 use --with-baseru= by option.; false ./tool/gen_dummy_probes.rb ./probes.d > dmyprobes.h executable host ruby is required. use --with-baseruby option.
make: *** = [dmyprobes.h] Error 1

That was the reason why Aaron proposed to use = sed instead of Ruby [1].

Btw if you want to use Ruby, there is bette= r version [2] then directly rewriting sed into Ruby regexps as in r37636.

Vit


[1] http://bugs.ruby-lang.org/issues/2565#note-40
[2] http://bugs.ruby-lang.org/attachments/2684/0001-Generate-pr= obes.h-from-probes.d.patch
--20cf303b3c11186bf804ce9d9394--