Anna Karas has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests: Fix a typo in tests/lib/Makefile.inc ......................................................................
tests: Fix a typo in tests/lib/Makefile.inc
Fix a typo in a comment in Makefile.inc.
Signed-off-by: Anna Karas aka@semihalf.com Change-Id: I236b040a83700bcd34d99db61e5dad0ff7abb28c --- M tests/lib/Makefile.inc 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/42312/1
diff --git a/tests/lib/Makefile.inc b/tests/lib/Makefile.inc index 565bb31..61f99bf 100644 --- a/tests/lib/Makefile.inc +++ b/tests/lib/Makefile.inc @@ -1,6 +1,6 @@ # SPDX-License-Identifier: GPL-2.0-only
-# object filest should be under build/tests/ build/test/src/ build/test/run/ +# object files should be under build/tests/ build/test/src/ build/test/run/ # two examples - first should be simply string.c, second should use -wrap
tests-y += string-test
HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests: Fix a typo in tests/lib/Makefile.inc ......................................................................
Patch Set 1: Code-Review+2
Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests: Fix a typo in tests/lib/Makefile.inc ......................................................................
Patch Set 1: Code-Review+2
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests: Fix a typo in tests/lib/Makefile.inc ......................................................................
Patch Set 1: Code-Review+2
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests: Fix a typo in tests/lib/Makefile.inc ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42312/1/tests/lib/Makefile.inc File tests/lib/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/42312/1/tests/lib/Makefile.inc@3 PS1, Line 3: # object files should be under build/tests/ build/test/src/ build/test/run/ Is this comment still accurate, anyway? As far as I know object files are under build/tests/<test-subdir>/<test-name>/ now. (I'm also not sure why we have a comment like this in the subdirectory Makefile.inc of some random test... if we want to explain how the test output directory is structured, that should probably go into the main tests/Makefile.inc instead.)
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests: Fix a typo in tests/lib/Makefile.inc ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42312/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/42312/1//COMMIT_MSG@7 PS1, Line 7: tests: Fix a typo in tests/lib/Makefile.inc
tests/lib/Makefile.inc: Fix typo in comment
Jan Dabros has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests: Fix a typo in tests/lib/Makefile.inc ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42312/1/tests/lib/Makefile.inc File tests/lib/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/42312/1/tests/lib/Makefile.inc@3 PS1, Line 3: # object files should be under build/tests/ build/test/src/ build/test/run/
Is this comment still accurate, anyway? As far as I know object files are under build/tests/<test-su […]
Yeah, I think that there is no value in putting such a comment here. Let's remove this.
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Paul Fagerburg, Julius Werner, Angel Pons, Jan Dabros, HAOUAS Elyes,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/42312
to look at the new patch set (#2).
Change subject: tests/lib/Makefile.inc: remove a comment ......................................................................
tests/lib/Makefile.inc: remove a comment
Remove a comment since is not useful anymore.
Signed-off-by: Anna Karas aka@semihalf.com Change-Id: I236b040a83700bcd34d99db61e5dad0ff7abb28c --- M tests/lib/Makefile.inc 1 file changed, 0 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/42312/2
Anna Karaś has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests/lib/Makefile.inc: remove a comment ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/42312/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/42312/1//COMMIT_MSG@7 PS1, Line 7: tests: Fix a typo in tests/lib/Makefile.inc
tests/lib/Makefile. […]
Done
https://review.coreboot.org/c/coreboot/+/42312/1/tests/lib/Makefile.inc File tests/lib/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/42312/1/tests/lib/Makefile.inc@3 PS1, Line 3: # object files should be under build/tests/ build/test/src/ build/test/run/
Yeah, I think that there is no value in putting such a comment here. Let's remove this.
Done.
Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests/lib/Makefile.inc: remove a comment ......................................................................
Patch Set 2: Code-Review+1
For any comments that you have resolved, you should mark them resolved so that we can see which ones are still outstanding.
Anna Karaś has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests/lib/Makefile.inc: remove a comment ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42312/1/tests/lib/Makefile.inc File tests/lib/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/42312/1/tests/lib/Makefile.inc@3 PS1, Line 3: # object files should be under build/tests/ build/test/src/ build/test/run/
Done.
Done
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests/lib/Makefile.inc: remove a comment ......................................................................
Patch Set 2: Code-Review+2
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/42312 )
Change subject: tests/lib/Makefile.inc: remove a comment ......................................................................
tests/lib/Makefile.inc: remove a comment
Remove a comment since is not useful anymore.
Signed-off-by: Anna Karas aka@semihalf.com Change-Id: I236b040a83700bcd34d99db61e5dad0ff7abb28c Reviewed-on: https://review.coreboot.org/c/coreboot/+/42312 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Paul Fagerburg pfagerburg@chromium.org Reviewed-by: Julius Werner jwerner@chromium.org --- M tests/lib/Makefile.inc 1 file changed, 0 insertions(+), 3 deletions(-)
Approvals: build bot (Jenkins): Verified Julius Werner: Looks good to me, approved Paul Fagerburg: Looks good to me, but someone else must approve
diff --git a/tests/lib/Makefile.inc b/tests/lib/Makefile.inc index 565bb31..c490317 100644 --- a/tests/lib/Makefile.inc +++ b/tests/lib/Makefile.inc @@ -1,8 +1,5 @@ # SPDX-License-Identifier: GPL-2.0-only
-# object filest should be under build/tests/ build/test/src/ build/test/run/ -# two examples - first should be simply string.c, second should use -wrap - tests-y += string-test
string-test-srcs += tests/lib/string-test.c