Page MenuHomePhabricator

[CI] Log sanitizers output to stdout instead of log files

Authored by Fabien on Mon, Jun 29, 15:50.



This will allow to correlate the output with the running test timewise,
which can be helpful during debugging.

Test Plan
./contrib/teamcity/ build-asan
./contrib/teamcity/ build-ubsan
./contrib/teamcity/ build-tsan

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien created this revision.Mon, Jun 29, 15:50
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Jun 29, 15:50
Fabien requested review of this revision.Mon, Jun 29, 15:50
jasonbcox accepted this revision.Mon, Jun 29, 15:59
jasonbcox added a subscriber: jasonbcox.
jasonbcox added inline comments.
31 ↗(On Diff #21852)

Not really related to this diff, but should suppressions also be moved into the config?

This revision is now accepted and ready to land.Mon, Jun 29, 15:59
Fabien added inline comments.Mon, Jun 29, 16:04
31 ↗(On Diff #21852)

Suppression files are already set by cmake. The only reason for having it here is that we are not calling a ninja target. If it is moved to the configuration it would end up being duplicated for all the ninja calls.

This revision was automatically updated to reflect the committed changes.