-
Notifications
You must be signed in to change notification settings - Fork 640
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dockerfile ARG default throws NPE if ARG is null #1528
Comments
@twendelmuth : Hi, was this working before v0.39.0 . I'm afraid it might be due to #1524 Would it be possible for you to raise a PR to fix this? |
…exists but value is null
…exists but value is null Signed-off-by: thorsten.wendelmuth <[email protected]>
@rohanKanojia Sure. I don't think this was working before though.
|
…exists but value is null Signed-off-by: thorsten.wendelmuth <[email protected]>
…t value is null Signed-off-by: thorsten.wendelmuth <[email protected]>
Hello, Looks like the issue is still here Execution fabric8-build of goal io.fabric8:docker-maven-plugin:0.40.2:build failed: null value in entry: TEST=null Caused by: java.lang.NullPointerException: null value in entry: TEST=null |
@flocard02 : Could you please create a new issue for this? |
Description
It seems like if you're building a Dockerfile with
ARG
and defining a default likeARG TEST=test
and pass<TEST></TEST>
to the plugin, this will fail with NullpointerException in https://github.com/fabric8io/docker-maven-plugin/blob/master/src/main/java/io/fabric8/maven/docker/util/DockerFileUtil.java#L268Info
docker-maven-plugin version : 0.39.0
pom.xml configuration
The example is obviously not what is happening in the end (with that you could just ask me to remove the
arg
if I'm not setting it, but rather the problem that I have another variable in there which potentially is not set)The text was updated successfully, but these errors were encountered: