Separate io.Writer from console.File for NewPrinter

Signed-off-by: Edgar Lee <edgarl@netflix.com>
This commit is contained in:
Edgar Lee 2022-02-05 13:57:33 -08:00
parent 595285736c
commit 2d7540fb0a
4 changed files with 4 additions and 5 deletions

View File

@ -76,7 +76,7 @@ func runBake(dockerCli command.Cli, targets []string, in bakeOptions) (err error
ctx2, cancel := context.WithCancel(context.TODO())
defer cancel()
printer := progress.NewPrinter(ctx2, os.Stderr, in.progress)
printer := progress.NewPrinter(ctx2, os.Stderr, os.Stderr, in.progress)
defer func() {
if printer != nil {

View File

@ -239,7 +239,7 @@ func buildTargets(ctx context.Context, dockerCli command.Cli, opts map[string]bu
ctx2, cancel := context.WithCancel(context.TODO())
defer cancel()
printer := progress.NewPrinter(ctx2, os.Stderr, progressMode)
printer := progress.NewPrinter(ctx2, os.Stderr, os.Stderr, progressMode)
resp, err := build.Build(ctx, dis, opts, dockerAPI(dockerCli), confutil.ConfigDir(dockerCli), printer)
err1 := printer.Wait()

View File

@ -414,7 +414,7 @@ func boot(ctx context.Context, ngi *nginfo) (bool, error) {
return false, nil
}
printer := progress.NewPrinter(context.TODO(), os.Stderr, "auto")
printer := progress.NewPrinter(context.TODO(), os.Stderr, os.Stderr, "auto")
baseCtx := ctx
eg, _ := errgroup.WithContext(ctx)

View File

@ -41,7 +41,7 @@ func (p *Printer) Warnings() []client.VertexWarning {
return p.warnings
}
func NewPrinter(ctx context.Context, out console.File, mode string) *Printer {
func NewPrinter(ctx context.Context, w io.Writer, out console.File, mode string) *Printer {
statusCh := make(chan *client.SolveStatus)
doneCh := make(chan struct{})
@ -56,7 +56,6 @@ func NewPrinter(ctx context.Context, out console.File, mode string) *Printer {
go func() {
var c console.Console
var w io.Writer = out
switch mode {
case PrinterModeQuiet:
w = ioutil.Discard