Skip to content

Commit

Permalink
Revert "dejagnu: launch nsimdrv under time utility"
Browse files Browse the repository at this point in the history
This reverts commit 6f75537.

On a Debian machine without the "time" executable (/usr/bin/time),
the shell builtin is used which doesn't accept "-p" flag, which
leads to a misleading error in the log:

  Simulator time -p /path/to/nsim/bin/nsimdrv ... missing.
  FAIL: gcc.dg/20000623-1.c execution test

Since this change was only for debugging purposes, I don't see
the point to craft it even fruther like using "command -v time"
or optional "-p" flags, etc. Let's keep it simple.
  • Loading branch information
Shahab Vahedi authored and temap committed Dec 15, 2021
1 parent f06d923 commit db00552
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion dejagnu/baseboards/arc-sim-nsimdrv.exp
Original file line number Diff line number Diff line change
Expand Up @@ -277,7 +277,7 @@ if { [info exists env(ARC_NSIM_OPTS) ] } {
}

# Setup nSIM.
set_board_info sim "time -p $nsim_bin [join $nsim_flags]"
set_board_info sim "$nsim_bin [join $nsim_flags]"
# is_simulator should be set already by 'load_generic_config "sim"'. Do it here
# to be safe.
set_board_info is_simulator 1
Expand Down

0 comments on commit db00552

Please sign in to comment.