| Message ID | 50b239f40ddbf4bd3e067c39330f99bf63c71147.1690489039.git.falcon@tinylab.org (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | selftests/nolibc: add minimal kernel config support - part1 | expand |
On 2023-07-28 04:22:42+0800, Zhangjin Wu wrote: > After the tests finish, it is valuable to report and summarize with > existing test log. > > This avoid rerun or run the tests again when not necessary. > > Signed-off-by: Zhangjin Wu <falcon@tinylab.org> > --- > tools/testing/selftests/nolibc/Makefile | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/tools/testing/selftests/nolibc/Makefile b/tools/testing/selftests/nolibc/Makefile > index 2e9694370913..75419b695f0d 100644 > --- a/tools/testing/selftests/nolibc/Makefile > +++ b/tools/testing/selftests/nolibc/Makefile > @@ -207,6 +207,10 @@ rerun: > $(Q)qemu-system-$(QEMU_ARCH) -display none -no-reboot -kernel "$(srctree)/$(IMAGE)" -serial stdio $(QEMU_ARGS) > "$(CURDIR)/run.out" > $(Q)$(REPORT) $(CURDIR)/run.out > > +# report with existing test log > +report: > + $(Q)$(REPORT_RUN_OUT) Isn't REPORT_RUN_OUT gone in this revision? > + > clean: > $(call QUIET_CLEAN, sysroot) > $(Q)rm -rf sysroot > -- > 2.25.1 >
> On 2023-07-28 04:22:42+0800, Zhangjin Wu wrote: > > After the tests finish, it is valuable to report and summarize with > > existing test log. > > > > This avoid rerun or run the tests again when not necessary. > > > > Signed-off-by: Zhangjin Wu <falcon@tinylab.org> > > --- > > tools/testing/selftests/nolibc/Makefile | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/tools/testing/selftests/nolibc/Makefile b/tools/testing/selftests/nolibc/Makefile > > index 2e9694370913..75419b695f0d 100644 > > --- a/tools/testing/selftests/nolibc/Makefile > > +++ b/tools/testing/selftests/nolibc/Makefile > > @@ -207,6 +207,10 @@ rerun: > > $(Q)qemu-system-$(QEMU_ARCH) -display none -no-reboot -kernel "$(srctree)/$(IMAGE)" -serial stdio $(QEMU_ARGS) > "$(CURDIR)/run.out" > > $(Q)$(REPORT) $(CURDIR)/run.out > > > > +# report with existing test log > > +report: > > + $(Q)$(REPORT_RUN_OUT) > > Isn't REPORT_RUN_OUT gone in this revision? > Yeah, I moved it as the first generic patch but forgot it have used a later macro, and therefore no recheck in this revision, thanks a lot. Thanks Zhangjin > > + > > clean: > > $(call QUIET_CLEAN, sysroot) > > $(Q)rm -rf sysroot > > -- > > 2.25.1 > >
diff --git a/tools/testing/selftests/nolibc/Makefile b/tools/testing/selftests/nolibc/Makefile index 2e9694370913..75419b695f0d 100644 --- a/tools/testing/selftests/nolibc/Makefile +++ b/tools/testing/selftests/nolibc/Makefile @@ -207,6 +207,10 @@ rerun: $(Q)qemu-system-$(QEMU_ARCH) -display none -no-reboot -kernel "$(srctree)/$(IMAGE)" -serial stdio $(QEMU_ARGS) > "$(CURDIR)/run.out" $(Q)$(REPORT) $(CURDIR)/run.out +# report with existing test log +report: + $(Q)$(REPORT_RUN_OUT) + clean: $(call QUIET_CLEAN, sysroot) $(Q)rm -rf sysroot
After the tests finish, it is valuable to report and summarize with existing test log. This avoid rerun or run the tests again when not necessary. Signed-off-by: Zhangjin Wu <falcon@tinylab.org> --- tools/testing/selftests/nolibc/Makefile | 4 ++++ 1 file changed, 4 insertions(+)