Skip to content

Commit

Permalink
Updates from review comments on build changes
Browse files Browse the repository at this point in the history
  • Loading branch information
gary.archer committed Sep 4, 2024
1 parent bfdf8b6 commit 3c427fb
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -58,10 +58,7 @@ if [[ ! -r $NGINX_TARBALL ]]; then
fi

docker build --no-cache -t nginx-module-builder \
--build-arg NGINX_SRC_DIR=/tmp/nginx-"$NGINX_VERSION" \
--build-arg NGINX_VERSION="$NGINX_VERSION" \
--build-arg NGINX_DEBUG=n \
--build-arg DYNAMIC_MODULE=Y \
-f builders/$LINUX_DISTRO.Dockerfile .
if [ $? -ne 0 ]; then
echo "Docker build problem encountered for OS $LINUX_DISTRO and NGINX $NGINX_VERSION"
Expand All @@ -70,8 +67,7 @@ fi

mkdir -p build
LIBRARY_PREFIX=$(getLibraryPrefix)
docker run --name nginx-modules -d nginx-module-builder
docker run --name nginx-modules nginx-module-builder
docker cp nginx-modules:/tmp/nginx-$NGINX_VERSION/objs/ngx_curity_http_phantom_token_module.so ./build/$LIBRARY_PREFIX.ngx_curity_http_phantom_token_module_$NGINX_VERSION.so
docker stop -t 0 nginx-modules
docker rm nginx-modules
docker rmi nginx-module-builder

0 comments on commit 3c427fb

Please sign in to comment.