Skip to content

Commit

Permalink
Removed unused parameter of buildOpenJDKViaDocker
Browse files Browse the repository at this point in the history
buildOpenJDKViaDocker do not need  ${BUILD_CONFIG[USE_DOCKER]} as
parameter. docker-build.sh  is inheriting the whole BUILD_CONFIG
  • Loading branch information
judovana committed May 8, 2024
1 parent 8ade4ab commit 67b41c1
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
3 changes: 0 additions & 3 deletions docker-build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -74,9 +74,6 @@ buildDockerContainer()
# Execute the (Adoptium) OpenJDK build inside the Docker Container
buildOpenJDKViaDocker()
{

DOCKER_PPODMAN="${1}"

local hostDir
hostDir="$(pwd)"
local pipelinesdir="${hostDir}"/workspace/pipelines
Expand Down
2 changes: 1 addition & 1 deletion makejdk-any-platform.sh
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ echo "$@" > ./workspace/config/makejdk-any-platform.args

# Let's build and test the (Adoptium) OpenJDK binary in Docker or natively
if [ ! "${BUILD_CONFIG[USE_DOCKER]}" == "false" ] ; then
buildOpenJDKViaDocker ${BUILD_CONFIG[USE_DOCKER]}
buildOpenJDKViaDocker
else
buildOpenJDKInNativeEnvironment
fi

0 comments on commit 67b41c1

Please sign in to comment.