888574 - Remove TAR_CREATE_FLAGS_QUIET, make TAR_CREATE_FLAGS quiet by default

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v1

Removes 57 lines of output on a no-op build; this leaves ~25 non-NSS lines.

Comment 1

[Gregory Szorc [:gps]]

Comment on attachment 769331 [details] [diff] [review]
Patch v1

Review of attachment 769331 [details] [diff] [review]:
-----------------------------------------------------------------

I can't help but think that somewhere someone is utilizing the printed list of files and defaulting to non-verbose will upset them or make their workflow harder. That being said, I'm inclined to just land this then address any complaints that arise. This is certainly easy enough to back out if it causes anyone hurt.

The changes in this patch look good, but I'd like additional sign-off from someone on the automation side of the fence.

Comment 2

[(not currently active) Ted Mielczarek]

Comment on attachment 769331 [details] [diff] [review]
Patch v1

Review of attachment 769331 [details] [diff] [review]:
-----------------------------------------------------------------

A brief survey of mxr shows that we only ever seem to use TAR_CREATE_FLAGS when running tar with - as the output filename, which means this spew is all going to stderr anyway, so someone would have to be jumping through great hoops to do anything with it. I'm not aware of anywhere that we actually need the output.

Comment 3

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://hg.mozilla.org/mozilla-central/rev/82df23e7392f