From: Gabe Black <gabe.black@gmail.com>
Date: Thu, 26 Mar 2020 10:47:38 +0000 (-0700)
Subject: scons: Get rid of a redundant "Warning:" in the SConstruct.
X-Git-Tag: v20.0.0.0~244
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=ec236a55eeef31435df999fc790f5c501aa657c6;p=gem5.git

scons: Get rid of a redundant "Warning:" in the SConstruct.

The "warning()" method already prints "Warning:", so putting it in the
message itself means it gets printed twice.

Change-Id: Ic157355958fdf56739f865a926ecba071bb25c5e
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/27127
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Bobby R. Bruce <bbruce@ucdavis.edu>
Tested-by: kokoro <noreply+kokoro@google.com>
---

diff --git a/SConstruct b/SConstruct
index f5eb0ca63..8d5ac5748 100755
--- a/SConstruct
+++ b/SConstruct
@@ -425,9 +425,9 @@ if main['GCC']:
     disable_lto = GetOption('no_lto')
     if not disable_lto and main.get('BROKEN_INCREMENTAL_LTO', False) and \
             not GetOption('force_lto'):
-        warning('Warning: Your compiler doesn\'t support incremental linking '
-                'and lto at the same time, so lto is being disabled. To force '
-                'lto on anyway, use the --force-lto option. That will disable '
+        warning('Your compiler doesn\'t support incremental linking and lto '
+                'at the same time, so lto is being disabled. To force lto on '
+                'anyway, use the --force-lto option. That will disable '
                 'partial linking.')
         disable_lto = True