Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48644 )

Change subject: scons: Get rid of some redundant text in a print out.
......................................................................

scons: Get rid of some redundant text in a print out.

When printing the LSAN_OPTIONS setting to use, we were manually
prefixing "suppressions=" to it, but it already had that built into the
option. This made that text print twice. Remove the duplicate.

Change-Id: Ie599e02d0cf17bcf18278d72350831a2e0ac09f6
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/48644
Reviewed-by: Bobby R. Bruce <[email protected]>
Maintainer: Bobby R. Bruce <[email protected]>
Tested-by: kokoro <[email protected]>
---
M SConstruct
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Bobby R. Bruce: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass



diff --git a/SConstruct b/SConstruct
index c980300..d9d2db3 100755
--- a/SConstruct
+++ b/SConstruct
@@ -414,7 +414,7 @@
     warning('To suppress false positive leaks, set the LSAN_OPTIONS '
             'environment variable to "%s" when running gem5' %
             suppressions_opt)
-    warning('LSAN_OPTIONS=suppressions=%s' % suppressions_opt)
+    warning('LSAN_OPTIONS=%s' % suppressions_opt)
     print()
 if sanitizers:
     sanitizers = ','.join(sanitizers)

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/48644
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: Ie599e02d0cf17bcf18278d72350831a2e0ac09f6
Gerrit-Change-Number: 48644
Gerrit-PatchSet: 2
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-Reviewer: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Nikos Nikoleris <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-CC: Giacomo Travaglini <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to