Hello Gabe Black,

I'd like you to do a code review. Please visit

    https://gem5-review.googlesource.com/c/public/gem5/+/27130

to review the following change.


Change subject: scons: Call summarize_warnings() when scons finishes building.
......................................................................

scons: Call summarize_warnings() when scons finishes building.

This will ensure that warnings are not all shoved off the end of the
scrollback buffer or lost in a sea of compiler lines, and that the user
will actually have a chance to see and read them.

Change-Id: I7129560482ebca903ec597f8b1cf8a9a84d98c9c
---
M SConstruct
1 file changed, 3 insertions(+), 0 deletions(-)



diff --git a/SConstruct b/SConstruct
index 8d5ac57..0f36eac 100755
--- a/SConstruct
+++ b/SConstruct
@@ -78,6 +78,7 @@
 from __future__ import print_function

 # Global Python includes
+import atexit
 import itertools
 import os
 import re
@@ -1274,3 +1275,5 @@

 %(local_vars)s
 ''' % help_texts)
+
+atexit.register(summarize_warnings)

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/27130
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I7129560482ebca903ec597f8b1cf8a9a84d98c9c
Gerrit-Change-Number: 27130
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <gabebl...@google.com>
Gerrit-Reviewer: Gabe Black <gabe.bl...@gmail.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to