Skip to content

Commit

Permalink
cmd-buildextend-metal: clarify rm command
Browse files Browse the repository at this point in the history
The ${build_semaphore} is a file, not a directory so using -r on
the rm is confusing. Let's remove it.
  • Loading branch information
dustymabe committed Jan 31, 2024
1 parent 7d329a2 commit 03dc753
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/cmd-buildextend-metal
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ if [ -e "${build_semaphore}" ]; then
fatal "${build_semaphore} found: another process is building ${image_type}"
fi
touch "${build_semaphore}"
trap 'rm -rf ${build_semaphore}' EXIT
trap 'rm -f ${build_semaphore}' EXIT

# check if the image already exists in the meta.json
if [ -z "${force}" ]; then
Expand Down

0 comments on commit 03dc753

Please sign in to comment.