Skip to content

Commit c566fa9

Browse files
Martin Hafskjold Thoreseneldering
Martin Hafskjold Thoresen
authored andcommitted
Fix sed regex hyphen placement
`sed` complained about the former rule contained an illegal charanger range. According to [0], 9.3.5 RE Bracked Expression (7) one's supposed to have the - be either first or last for it to be treated literally. This is also different from some languages, e.g. python, in which the previous regex works just fine. [0]: https://www2.cs.sfu.ca/~anoop/courses/CMPT-379-Fall-2004/posix-regexp.html (cherry picked from commit 6b5c18c)
1 parent adf7f65 commit c566fa9

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Diff for: docker/build-judgehost.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ docker_tag="$1"
1010
docker build -t "${docker_tag}-build" -f judgehost/Dockerfile.build .
1111

1212
# Build chroot
13-
builder_name=$(echo "${docker_tag}" | sed 's/[^a-zA-Z0-9\-_]/-/g')
13+
builder_name=$(echo "${docker_tag}" | sed 's/[^a-zA-Z0-9_-]/-/g')
1414
docker rm -f "${builder_name}" > /dev/null 2>&1 || true
1515
docker run --name "${builder_name}" --cap-add=sys_admin "${docker_tag}-build"
1616
docker cp "${builder_name}:/chroot.tar.gz" .

0 commit comments

Comments
 (0)