Skip to content
  • Iustin Pop's avatar
    Fix utils unittest TestRunCmd.testTimeoutKill · 815bf6d5
    Iustin Pop authored
    
    
    Currently the test executes "/bin/sh -c 'trap "" TERM …" via the shell,
    which means we have two shells involved (the parent one and the child
    one, which does the trap). For some weird reason, this works for older
    OSes: the first shell exec's the second one, whereas on squeeze it forks
    and only then exec. This might be due to changes in Python or bash or
    whatever.
    
    In any case, the point is that this is a hacky way to call a shell, so
    we change this to simply execute the desired shell directly, without any
    intermediate shells (I don't understand why it was written as such
    originally).
    
    Signed-off-by: default avatarIustin Pop <iustin@google.com>
    Reviewed-by: default avatarRené Nussbaumer <rn@google.com>
    815bf6d5