diff options
author | Jakob Unterwurzacher | 2016-06-05 14:26:16 +0200 |
---|---|---|
committer | Jakob Unterwurzacher | 2016-06-05 14:32:07 +0200 |
commit | 0c80cca674931c9dbfc69c25df24d53abbdd63a9 (patch) | |
tree | c11cae555954fc08f3e28f22b6ed23ea5717a083 /internal/toggledlog | |
parent | ca54b665e32a9b298ea3e70b5da0108db3a71364 (diff) |
toggledlog: convert remaing naked fmt.Print*
Several fatal errors were just printed to stdout, which
meant they were invisible when running the test suite.
Fix this by introducing toggledlog.Fatal and convert as
follows:
Fatal errors -> toggledlog.Fatal
Warnings -> toggledlog.Warn
Password prompts -> fmt.Fprintf
Diffstat (limited to 'internal/toggledlog')
-rw-r--r-- | internal/toggledlog/log.go | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/internal/toggledlog/log.go b/internal/toggledlog/log.go index 7f437c8..23d7347 100644 --- a/internal/toggledlog/log.go +++ b/internal/toggledlog/log.go @@ -49,18 +49,24 @@ func (l *toggledLogger) Println(v ...interface{}) { } } -// As defined by http://elinux.org/Debugging_by_printing#Log_Levels // Debug messages +// Can be enabled by passing "-d" var Debug *toggledLogger -// Informational message e.g. startup information +// Informational message +// Can be disabled by passing "-q" var Info *toggledLogger -// A warning, meaning nothing serious by itself but might indicate problems +// A warning, meaning nothing serious by itself but might indicate problems. +// Passing "-wpanic" will make this function panic after printing the message. var Warn *toggledLogger +// Fatal error, we are about to exit +var Fatal *toggledLogger + func init() { Debug = &toggledLogger{false, false, log.New(os.Stdout, "", 0)} Info = &toggledLogger{true, false, log.New(os.Stdout, "", 0)} Warn = &toggledLogger{true, false, log.New(os.Stderr, "", 0)} + Fatal = &toggledLogger{true, false, log.New(os.Stderr, "", 0)} } |