diff --git a/cli/command/cli.go b/cli/command/cli.go index 0306f242d3..4d8b9dc406 100644 --- a/cli/command/cli.go +++ b/cli/command/cli.go @@ -162,8 +162,8 @@ func (cli *DockerCli) ContentTrustEnabled() bool { // BuildKitEnabled returns buildkit is enabled or not. func (cli *DockerCli) BuildKitEnabled() (bool, error) { - // use DOCKER_BUILDKIT env var value if set - if v, ok := os.LookupEnv("DOCKER_BUILDKIT"); ok { + // use DOCKER_BUILDKIT env var value if set and not empty + if v := os.Getenv("DOCKER_BUILDKIT"); v != "" { enabled, err := strconv.ParseBool(v) if err != nil { return false, errors.Wrap(err, "DOCKER_BUILDKIT environment variable expects boolean value") diff --git a/cmd/docker/builder.go b/cmd/docker/builder.go index 71fe979b38..7807f80b1a 100644 --- a/cmd/docker/builder.go +++ b/cmd/docker/builder.go @@ -44,9 +44,9 @@ func processBuilder(dockerCli command.Cli, cmd *cobra.Command, args, osargs []st var buildKitDisabled, useBuilder, useAlias bool var envs []string - // check DOCKER_BUILDKIT env var is present and - // if not assume we want to use the builder component - if v, ok := os.LookupEnv("DOCKER_BUILDKIT"); ok { + // check DOCKER_BUILDKIT env var is not empty + // if it is assume we want to use the builder component + if v := os.Getenv("DOCKER_BUILDKIT"); v != "" { enabled, err := strconv.ParseBool(v) if err != nil { return args, osargs, nil, errors.Wrap(err, "DOCKER_BUILDKIT environment variable expects boolean value")