benchmark/driver.rb: --executables separator
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 ko1 (Koichi Sasada) almost 8 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) almost 8 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!