common/loggers: Write PrintTimerIfDelayed output to stdErr

[jmooring]

In #13138 we made all logging (info, warn, error) log to stderr and let stdout be reserved for program output.

When you run hugo against a project that includes a module that has not been downloaded yet, this is written to stdout:

hugo: collected modules in 962 ms

If you run the command hugo list all before the module has been downloaded, the resulting CSV data is invalid.

If seems like the message above should be logged to stdErr instead.

hugo/modules/collect.go

Line 509 in 5d64b49

defer c.logger.PrintTimerIfDelayed(time.Now(), "hugo: collected modules")

hugo/common/loggers/logger.go

Lines 270 to 279 in 5d64b49

	// PrintTimerIfDelayed prints a time statement to the FEEDBACK logger
	// if considerable time is spent.
	func (l *logAdapter) PrintTimerIfDelayed(start time.Time, name string) {
	elapsed := time.Since(start)
	milli := int(1000 * elapsed.Seconds())
	if milli < 500 {
	return
	}
	l.Printf("%s in %v ms", name, milli)
	}

Can we replace the last line in the above with this?

fmt.Fprintf(l.stdErr, "%s in %v ms", name, milli)

[bep]

fmt.Fprintf(l.stdErr, "%s in %v ms", name, milli)

Yes. I will do a patch release, possibly Monday.

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.