Skip to content

Commit

Permalink
Fix todo
Browse files Browse the repository at this point in the history
  • Loading branch information
martin-schulze-vireso authored and sublimino committed Sep 16, 2020
1 parent 3ed95c4 commit 0a24927
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion lib/bats-core/semaphore.bash
Expand Up @@ -33,8 +33,9 @@ bats_semaphore_release_wrapper() {
mkdir -p "$output_dir"
"$@" 2>"$output_dir/stderr" >"$output_dir/stdout"

# TODO: why is this necessary? shouldn't the EXIT trap do this already? does it work with signals then?
# bash bug: the exit trap is not called for the background process
bats_semaphore_release_slot
trap '' EXIT # avoid calling release twice
}

# block until a semaphore slot becomes free
Expand Down

0 comments on commit 0a24927

Please sign in to comment.