From: Takashi Kokubun Date: 2018-07-09T09:38:35+09:00 Subject: [ruby-core:87884] Re: [Ruby trunk Misc#14902] Replace benchmark/driver.rb with benchmark_driver.gem --===============0008526772== Content-Type: multipart/alternative; boundary="0000000000007c6e7f0570863c4e" --0000000000007c6e7f0570863c4e Content-Type: text/plain; charset="UTF-8" Oh I see. I'll fix so later. Thanks for notification. On Mon, Jul 9, 2018 at 6:14 wrote: > Issue #14902 has been updated by MSP-Greg (Greg L). > > > @k0kubun > > Thank you for your work on this. Somewhat off topic... > > I ran the suite on ruby-loco, and the only change needed was adding the > following line to file_rename.yml, before the `max.times do` line: > > ```ruby > tmp.each { |t| t.close } if RUBY_PLATFORM[/mingw|mswin/] > ``` > > Windows won't allow renaming an open file. Thanks, Greg > > ---------------------------------------- > Misc #14902: Replace benchmark/driver.rb with benchmark_driver.gem > https://bugs.ruby-lang.org/issues/14902#change-72897 > > * Author: k0kubun (Takashi Kokubun) > * Status: Closed > * Priority: Normal > * Assignee: k0kubun (Takashi Kokubun) > ---------------------------------------- > ## What's benchmark_driver.gem? > benchmark_driver.gem is a gem created by a project accepted as Ruby > Association Grant 2017. > https://github.com/benchmark-driver/benchmark-driver > > This is built as a general-purpose benchmark driver for Ruby and allows to > collect any metric and to output in any way by writing plugins. > > ## Why to replace benchmark/driver.rb? > First of all, the benchmark_driver.gem is created as a successor of > benchmark/driver.rb, requested by ko1 (author of benchmark/driver.rb). > After the yesterday's project report of Ruby Association Grant 2017, ko1 > and I agreed to replace benchmark/driver.rb with benchmark_driver.gem. > So I'll start working on this. > > ## How to replace benchmark/driver.rb? > Some metrics supported by benchmark/driver.rb can't be collected by > benchmark_driver.gem by default for now, but I'll leave them as plugins of > benchmark_driver.gem. > It's a gem implemented outside this repository, but I'm going to implement > make task to clone the gem repository like simplecov. > > Unfortunately the scripts in benchmark directory can't be run by > benchmark_driver.gem by default. So I'll implement some wrapper script to > allow benchmark_driver.gem to run them at first, rewrite benchmark scripts > to benchmark_driver.gem format one by one, drop old scripts and > benchmark/driver.gem in the end. > > During the migration, I'm going to keep the interface of `make benchmark` > and `make benchmark-each`, but directly running benchmark/driver.rb will be > broken at some moment. > > > > -- > https://bugs.ruby-lang.org/ > > Unsubscribe: > > --0000000000007c6e7f0570863c4e Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Oh I see. I'll fix so later. Thanks for notifica= tion.

On Mon, Jul 9, 2018 at 6:14 = <Greg.mpls@gmail.com> wrot= e:
Issue #14902 has been updated by= MSP-Greg (Greg L).


@k0kubun

Thank you for your work on this.=C2=A0 Somewhat off topic...

I ran the suite on ruby-loco, and the only change needed was adding the fol= lowing line to file_rename.yml, before the `max.times do` line:

```ruby
tmp.each { |t| t.close } if RUBY_PLATFORM[/mingw|mswin/]
```

Windows won't allow renaming an open file.=C2=A0 Thanks, Greg

----------------------------------------
Misc #14902: Replace benchmark/driver.rb with benchmark_driver.gem
https://bugs.ruby-lang.org/issues/14902#change-7= 2897

* Author: k0kubun (Takashi Kokubun)
* Status: Closed
* Priority: Normal
* Assignee: k0kubun (Takashi Kokubun)
----------------------------------------
## What's benchmark_driver.gem?
benchmark_driver.gem is a gem created by a project accepted as Ruby Associa= tion Grant 2017.
https://github.com/benchmark-driver/benchmark-dr= iver

This is built as a general-purpose benchmark driver for Ruby and allows to = collect any metric and to output in any way by writing plugins.

## Why to replace benchmark/driver.rb?
First of all, the benchmark_driver.gem is created as a successor of benchma= rk/driver.rb, requested by ko1 (author of benchmark/driver.rb).
After the yesterday's project report of Ruby Association Grant 2017, ko= 1 and I agreed to replace benchmark/driver.rb with benchmark_driver.gem. So I'll start working on this.

## How to replace benchmark/driver.rb?
Some metrics supported by benchmark/driver.rb can't be collected by ben= chmark_driver.gem by default for now, but I'll leave them as plugins of= benchmark_driver.gem.
It's a gem implemented outside this repository, but I'm going to im= plement make task to clone the gem repository like simplecov.

Unfortunately the scripts in benchmark directory can't be run by benchm= ark_driver.gem by default. So I'll implement some wrapper script to all= ow benchmark_driver.gem to run them at first, rewrite benchmark scripts to = benchmark_driver.gem format one by one, drop old scripts and benchmark/driv= er.gem in the end.

During the migration, I'm going to keep the interface of `make benchmar= k` and `make benchmark-each`, but directly running benchmark/driver.rb will= be broken at some moment.



--
https://bugs.ruby-lang.org/

Unsubscribe: <mailto:ruby-core-request@ruby-lang.org?subject=3Dunsubscribe= >
<http://lists.ruby-lang.org/cgi-bin/m= ailman/options/ruby-core>
--0000000000007c6e7f0570863c4e-- --===============0008526772== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline Unsubscribe: --===============0008526772==--