By default Buildbot prints worker's environment variables in each build step. This is unnecessary and makes the logs hard to read. We should disable printing environment variable in most of the build steps.
This is part 2 of https://bugs.webkit.org/show_bug.cgi?id=197319
Created attachment 373323 [details]
Attachment 373323 [details] did not pass style-queue:
ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:117: [CheckOutSpecificRevision.__init__] Use of super on an old style class [pylint/E1002] 
ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:139: [CleanWorkingDirectory.__init__] Use of super on an old style class [pylint/E1002] 
ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:622: [CompileWebKit.__init__] Use of super on an old style class [pylint/E1002] 
ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:1131: [ArchiveTestResults.__init__] Use of super on an old style class [pylint/E1002] 
Total errors found: 4 in 3 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Do we have an example of different logs? Seems like a good idea, I'd just like to see the difference.
(In reply to Jonathan Bedard from comment #3)
> Do we have an example of different logs? Seems like a good idea, I'd just like to see the difference.
Similarly, from https://bugs.webkit.org/show_bug.cgi?id=197319:
Comment on attachment 373323 [details]
Clearing flags on attachment: 373323
Committed r247069: <https://trac.webkit.org/changeset/247069>
All reviewed patches have been landed. Closing bug.