ocaml-bench / sandmark

A benchmark suite for the OCaml compiler
The Unlicense
82 stars 40 forks source link

Remove unnecessary double-dash in pausetimes command #435

Closed punchagan closed 1 year ago

punchagan commented 1 year ago

When running the benchmarks via opam exec -- dune build @${BUILD_BENCH_TARGET} command, the additional -- before the paramwrapper doesn't interact well with the pausetimes bash wrapper that evals an expression eval "olly latency -o $TMP --json '$@'". Mysteriously, this eval fails with the error that olly executable couldn't be found when the additional -- is present in the command. It is not entirely clear how this additional -- affects the environment variables being set by opam exec, but this command removes them to get the benchmarks to run correctly.

shakthimaan commented 1 year ago

Thanks!