makeman: replace -\|/ twiddles with dots
ClosedPublic
Actions

Authored by brooks on Mar 8 2023, 6:42 PM.

Details

Reviewers

emaste

Commits

rGcc36164a77f9: makeman: replace -\|/ twiddles with dots

Summary

It shows progress more clearly, there aren't so many that the results
are excessive, and it's more friendly for things like Cirrus-CI's
logging that print the backspaces as boxes.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 50238
Build 47130: arc lint + arc unit

Event Timeline

brooks created this revision.Mar 8 2023, 6:42 PM

Herald added a subscriber: imp. · View Herald TranscriptMar 8 2023, 6:42 PM

brooks requested review of this revision.Mar 8 2023, 6:42 PM

Harbormaster completed remote builds in B50238: Diff 118544.Mar 8 2023, 6:42 PM

emaste accepted this revision.Mar 8 2023, 6:44 PM

emaste added inline comments.

tools/build/options/makeman
322	or `echo -n`

This revision is now accepted and ready to land.Mar 8 2023, 6:44 PM

brooks added inline comments.Mar 8 2023, 7:12 PM

tools/build/options/makeman
322	Not that it really matters, but `printf` is in posix and `echo -n` isn't.

emaste added inline comments.Mar 8 2023, 7:14 PM

tools/build/options/makeman
322	TIL :)

Closed by commit rGcc36164a77f9: makeman: replace -\|/ twiddles with dots (authored by brooks). · Explain WhyMar 8 2023, 9:02 PM

This revision was automatically updated to reflect the committed changes.

brooks added a commit: rGcc36164a77f9: makeman: replace -\|/ twiddles with dots.

Revision Contents
Changeset List

Path

Size

tools/

build/

options/

makeman

12 lines

Diff 118544

View Options

tools/build/options/makeman

makeman: replace -\|/ twiddles with dotsClosedPublicActions