-
Notifications
You must be signed in to change notification settings - Fork 436
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
Added dockerBuildInit option #1438
Conversation
77818a8
to
52d654b
Compare
@@ -152,9 +152,10 @@ object DockerPlugin extends AutoPlugin { | |||
).toOption | |||
.map(_.trim) | |||
.flatMap(DockerApiVersion.parse), | |||
dockerBuildInit := false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we can use the dockerVersion
or dockerApiVersion
to set this true when possible?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Possible, however, dockerVersion
and dockerApiVersion
are task keys. dockerBuildOptions
and dockerBuildCommand
are setting keys, and so can't depend on dockerVersion
or dockerApiVersion
. We'd have to depend on this in publishLocal
, which I think would be bad because it would mean dockerBuildCommand
would no longer be the source of truth as to what was executed.
@@ -68,4 +68,10 @@ private[packager] trait DockerKeysEx extends DockerKeys { | |||
) | |||
val dockerLayerMappings = | |||
taskKey[Seq[LayeredMapping]]("List of layer, source file and destination in Docker image.") | |||
val dockerBuildInit = SettingKey[Boolean]( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved those to the DockerKeysEx
, to avoid binary incompatibilities
Hi, Tried to activate this new option and rebuilt my image but I get the following error:
I'm on Mac, is it only available on Linux? 🤔 |
See #1257.