Attention is currently required from: Daniel Gröber (dxld).
Jakub Czapiga has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44083 )
Change subject: tests: Add debug() macro enabled by TEST_DEBUG env-var
......................................................................
Patch Set 17: Code-Review-1
(1 comment)
File tests/debug.c:
https://review.coreboot.org/c/coreboot/+/44083/comment/836dbf29_580033eb
PS17, Line 11: if (argc > 1 && strcmp(argv[1], "--debug") == 0)
: test_debug = 1;
: }
There already is TEST_PRINT which enables printk() stubs.
If one wants to print messages from test code, they can use print_message() function form CMocka. To print form source just recompile test with TEST_PRINT=1
--
To view, visit
https://review.coreboot.org/c/coreboot/+/44083
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ifdea534e414c45cd9bc209636a03c6fadaa79e79
Gerrit-Change-Number: 44083
Gerrit-PatchSet: 17
Gerrit-Owner: Daniel Gröber (dxld)
coreboot@dxld.at
Gerrit-Reviewer: Jakub Czapiga
jacz@semihalf.com
Gerrit-Reviewer: Martin L Roth
gaumless@gmail.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Daniel Gröber
dxld@darkboxed.org
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Daniel Gröber (dxld)
coreboot@dxld.at
Gerrit-Comment-Date: Tue, 21 Feb 2023 15:02:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment