Sp 3544 #65
Sp 3544 #65
Conversation
| @@ -0,0 +1,16 @@ | |||
| ARG JAVA_VERSION=11 | |||
|
|
|||
| FROM gradle:7-jdk${JAVA_VERSION} as builder | |||
There was a problem hiding this comment.
After the FROM directive, some Docker build kits (e.g. Ubuntu 22.04) require ARG JAVA_VERSION, with no value set, to build the Docker image correctly.
There was a problem hiding this comment.
Understood, thanks. If one requires the JAVA_VERSION environment variable inside the build, then this is necessary. I just need it to identify the image to use.
| && rsync -arvc node-v18.18.0-linux-x64/* /usr/local/ | ||
|
|
||
| FROM gradle:7-jdk${JAVA_VERSION} | ||
|
|
There was a problem hiding this comment.
Same comment as above: ARG JAVA_VERSION after the FROM directive.
There was a problem hiding this comment.
Understood, thanks. If one requires the JAVA_VERSION environment variable inside the build, then this is necessary. I just need it to identify the image to use.
| @@ -0,0 +1,16 @@ | |||
| ARG JAVA_VERSION=11 | |||
There was a problem hiding this comment.
Is the name Dockerfile.build the accepted standard?
There was a problem hiding this comment.
No. This file is intentionally named differently as it's used to create the build environment. The Science Portal requires Gradle, JDK 11, and NPM, and if you like keeping your desktop clean, then this is used to build an image that can be used to build the Science Portal.
docker run --rm -ti -v $(pwd):/app -w /app image-from-dockerfile-build gradle -i clean build test
No description provided.