From 0d760731a1d808f65d41880419bf46531a597edb Mon Sep 17 00:00:00 2001 From: Carlos Alexandro Becker Date: Thu, 22 Jun 2017 00:25:52 -0300 Subject: [PATCH] debug, warning etc --- goreleaserlib/goreleaser.go | 3 +++ main.go | 6 +++++- pipeline/brew/brew.go | 6 +++--- pipeline/build/build.go | 13 ++++++------- pipeline/build/target.go | 4 ++-- pipeline/release/release.go | 2 +- 6 files changed, 20 insertions(+), 14 deletions(-) diff --git a/goreleaserlib/goreleaser.go b/goreleaserlib/goreleaser.go index 6b355eced..0eb45fdeb 100644 --- a/goreleaserlib/goreleaser.go +++ b/goreleaserlib/goreleaser.go @@ -44,6 +44,9 @@ type Flags interface { func Release(flags Flags) error { var file = flags.String("config") var notes = flags.String("release-notes") + if flags.Bool("debug") { + log.SetLevel(log.DebugLevel) + } cfg, err := config.Load(file) if err != nil { // Allow file not found errors if config file was not diff --git a/main.go b/main.go index bdc5eb231..036dd91f0 100644 --- a/main.go +++ b/main.go @@ -5,7 +5,7 @@ import ( "os" "github.com/apex/log" - lcli "github.com/apex/log/handlers/cli" + lcli "github.com/apex/log/handlers/text" "github.com/goreleaser/goreleaser/goreleaserlib" "github.com/urfave/cli" ) @@ -47,6 +47,10 @@ func main() { Name: "snapshot", Usage: "Generate an unversioned snapshot release", }, + cli.BoolFlag{ + Name: "debug", + Usage: "Enable debug mode", + }, } app.Action = func(c *cli.Context) error { log.Infof("Running goreleaser %v", version) diff --git a/pipeline/brew/brew.go b/pipeline/brew/brew.go index b1fb383e8..88e2e0d66 100644 --- a/pipeline/brew/brew.go +++ b/pipeline/brew/brew.go @@ -95,15 +95,15 @@ func (Pipe) Run(ctx *context.Context) error { func doRun(ctx *context.Context, client client.Client) error { if !ctx.Publish { - log.Info("Skipped because --skip-publish is set") + log.Warn("Skipped because --skip-publish is set") return nil } if ctx.Config.Brew.GitHub.Name == "" { - log.Info("Skipped because brew section is not configured") + log.Warn("Skipped because brew section is not configured") return nil } if ctx.Config.Release.Draft { - log.Info("Skipped because release is marked as draft") + log.Warn("Skipped because release is marked as draft") return nil } path := filepath.Join(ctx.Config.Brew.Folder, ctx.Config.Build.Binary+".rb") diff --git a/pipeline/build/build.go b/pipeline/build/build.go index 8a17bd23f..4795add9e 100644 --- a/pipeline/build/build.go +++ b/pipeline/build/build.go @@ -81,15 +81,14 @@ func build(ctx *context.Context, name string, target buildTarget) error { } func run(target buildTarget, command, env []string) error { - cmd := exec.Command(command[0], command[1:]...) + var cmd = exec.Command(command[0], command[1:]...) + env = append(env, "GOOS="+target.goos, "GOARCH="+target.goarch, "GOARM="+target.goarm) cmd.Env = append(cmd.Env, os.Environ()...) cmd.Env = append(cmd.Env, env...) - cmd.Env = append( - cmd.Env, - "GOOS="+target.goos, - "GOARCH="+target.goarch, - "GOARM="+target.goarm, - ) + log.WithField("target", target.PrettyString()). + WithField("env", env). + WithField("args", cmd.Args). + Debug("Running") if out, err := cmd.CombinedOutput(); err != nil { return fmt.Errorf("build failed: %s\n%v", target.PrettyString(), string(out)) } diff --git a/pipeline/build/target.go b/pipeline/build/target.go index 1cad3a7e7..bbe7cfd23 100644 --- a/pipeline/build/target.go +++ b/pipeline/build/target.go @@ -26,11 +26,11 @@ func (t buildTarget) PrettyString() string { func buildTargets(ctx *context.Context) (targets []buildTarget) { for _, target := range allBuildTargets(ctx) { if !valid(target) { - log.WithField("target", target.PrettyString()).Info("Skipped invalid build") + log.WithField("target", target.PrettyString()).Warn("Skipped invalid build") continue } if ignored(ctx, target) { - log.WithField("target", target.PrettyString()).Info("Skipped ignored build") + log.WithField("target", target.PrettyString()).Warn("Skipped ignored build") continue } targets = append(targets, target) diff --git a/pipeline/release/release.go b/pipeline/release/release.go index c81a4cdb8..203f1a6bf 100644 --- a/pipeline/release/release.go +++ b/pipeline/release/release.go @@ -27,7 +27,7 @@ func (Pipe) Run(ctx *context.Context) error { func doRun(ctx *context.Context, client client.Client) error { if !ctx.Publish { - log.Info("Skipped because --skip-publish is set") + log.Warn("Skipped because --skip-publish is set") return nil } log.WithField("tag", ctx.Git.CurrentTag).