Bug 35712 – setenv messages too noisy in running build-webkit

• Status: RESOLVED
• Resolution: WONTFIX
• Component: Tools / Tests

[Yuzo Fujishima]

build-webkit script prints too many setenv messages to see more important information.

[Yuzo Fujishima]

Created attachment 49974 [details]
Remove setenv messages from build-webkit output.

[Yuzo Fujishima]

Created attachment 49975 [details]
Remove setenv messages from build-webkit output.

[Mark Rowe (bdash)]

Comment on attachment 49975 [details]
Remove setenv messages from build-webkit output.

Please don’t do this.  These message are necessary to track down certain classes of build errors.

[Yuzo Fujishima]

Created attachment 49981 [details]
By default, suppress setenv messages from build-webkit/build-dumprendertree output.

[Yuzo Fujishima]

Hi, Mark,

Thank you for the review.

I understand sometimes the setenv messages are needed.
But can I assume they are needed only occasionally?

I'd like to propose to use --verbose flag to see setenv messages
and suppress them by default.

Can you review the revised patch?

[Eric Seidel (no email)]

Comment on attachment 49981 [details]
By default, suppress setenv messages from build-webkit/build-dumprendertree output.

Bleh.  Yet another option hacked into build-webkit.

We had this support at some point.  Maybe some of the build scripts still do.  The set envs are useless for most people, but I do think that the bots might still want them (or at least that's what Mark's comment implies).

We also have a --quiet flag that we pass to run-webkit-tests which hides the build output from build-dumprendertree entirely if I remember correctly.

So I like this change in principle.  But I think we may need to change the buildbots to call build-* with --verbose to satisify mark's concerns.  Also, it's sad that we continue to hack options in like this.  Would be better if (like python) we had some sort of option dictionary that we just passed around to all these methods.

r- because this woudl change buildbot behavior.

[Yuzo Fujishima]

Hi, Mark, Eric,

Thank you for your reviews.
OK, I gave up. I close this bug.

Yuzo