Project

General

Profile

Actions

Bug #7380

closed

benchmark/driver.rb: --executables separator

Added by Eregon (Benoit Daloze) over 11 years ago. Updated over 11 years ago.

Status:
Closed
Target version:
ruby -v:
ruby 2.0.0dev (2012-11-17 trunk 37693) [x86_64-darwin10.8.0]
Backport:
[ruby-core:49488]

Description

Hello,

The separator used between ruby executables for the benchmark/driver.rb script is ";".
This requires the argument to be quoted (at least in bash/zsh) and produce an unintuitive result if not: only the first executable is taken by the script, the second executable launches the ruby interpreter with no arguments, thus waiting on stdin, which feels like the benchmark script hangs.

I proposed to use ":", the standard path separator instead.
For compatibility, ";" could split too, but I wish to document ":" in the help.

What do you think? May I go ahead?

Updated by mame (Yusuke Endoh) over 11 years ago

  • Status changed from Open to Assigned
  • Assignee set to ko1 (Koichi Sasada)
  • Target version set to 2.0.0

ko1, what do you think?

--
Yusuke Endoh

Updated by ko1 (Koichi Sasada) over 11 years ago

I always use -e with quote strings. does it bother you?

By the way, -e accept `:'
like -e "foo::path1; bar::path2"
as a label of path.

Actions #3

Updated by ko1 (Koichi Sasada) over 11 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r37854.
Benoit, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • benchmark/driver.rb: accept multiple -e'. You don't need to use ;' separation character.
    [ruby-core:50139] [ruby-trunk - Bug #7380]

Updated by Eregon (Benoit Daloze) over 11 years ago

ko1 (Koichi Sasada) wrote:

I always use -e with quote strings. does it bother you?

By the way, -e accept `:'
like -e "foo::path1; bar::path2"
as a label of path.

Oh, right, I did not think to the name::path usage, it would be weird to have both : and ::.

I like the solution to pass multiple -e options, thank you!

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0