Name of user not set #1002873 has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40538 )
Change subject: tests: Add lib/string-test test case ......................................................................
Patch Set 5:
(1 comment)
https://review.coreboot.org/c/coreboot/+/40538/1/tests/lib/string-test.c File tests/lib/string-test.c:
https://review.coreboot.org/c/coreboot/+/40538/1/tests/lib/string-test.c@22 PS1, Line 22: strlen
I wasn't aware that libc marks its symbols weak (and I'm not sure that this is the case everywhere […]
Oh, looks like some race condition, I haven't seen your comment earlier Julius.
Yes, my first idea was that when you are writing tests, you really try to break something. I mean, at least I will be very suspicious if everything work out of the box for me... :)
That being said, it is enough to simply not include uut to srcs and we won't be aware that something bad happened. We need to be very careful (in general, not only because of this issue) when writing tests for coreboot libc implementation.