From 02c33896d57736d31446f1a6eafe0f28099917a8 Mon Sep 17 00:00:00 2001 From: Daniele Rondina Date: Fri, 10 Apr 2020 09:31:08 +0200 Subject: [PATCH] simpledocker: Move warning before return --- pkg/compiler/backend/simpledocker.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/pkg/compiler/backend/simpledocker.go b/pkg/compiler/backend/simpledocker.go index 88538b00..63c5f289 100644 --- a/pkg/compiler/backend/simpledocker.go +++ b/pkg/compiler/backend/simpledocker.go @@ -248,14 +248,15 @@ func (*SimpleDocker) Changes(fromImage, toImage string) ([]compiler.ArtifactLaye cmd := exec.Command("container-diff", diffargs...) cmd.Stderr = &errorBuffer out, err := cmd.Output() - if err != nil { - return []compiler.ArtifactLayer{}, errors.Wrap(err, "Failed Resolving layer diffs: "+string(out)) - } if string(errorBuffer.Bytes()) != "" { Warning("container-diff errored with: " + string(errorBuffer.Bytes())) } + if err != nil { + return []compiler.ArtifactLayer{}, errors.Wrap(err, "Failed Resolving layer diffs: "+string(out)) + } + if config.LuetCfg.GetGeneral().ShowBuildOutput { Info(string(out)) }