Bobby Bruce has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/54843 )

Change subject: scons: Re-add 'no-error=deprecated' compiler flags
......................................................................

scons: Re-add 'no-error=deprecated' compiler flags

Prior to this commit, `scons build/NULL/unittests.opt` failed with the
following error:

```
tests/build/ARM/base/trace.hh:229:32: error: 'DPRINTF_UNCONDITIONAL' is deprecated: Use DPRINTFN or DPRINTF with a debug flag instead. [-Werror=deprecated-declarations]
  229 |     GEM5_DEPRECATED_MACRO_STMT(DPRINTF_UNCONDITIONAL,      \
      |                                ^~~~~~~~~~~~~~~~~~~~~

...

cc1plus: all warnings being treated as errors
scons: *** [tests/build/ARM/base/trace.test.to] Error 1
scons: building terminated because of errors.
```

This is despire the `-Werror` flag being removed in
https://gem5-review.googlesource.com/c/public/gem5/+/54463.
It is unknown why a warning is being passed as an error in this
instance.

For the sake of the v21.2 release, the
'-Wno-error=deprecated-declarations' and '-Wno-error=deprecated' flags
have been re-added to avoid this error.

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



diff --git a/SConstruct b/SConstruct
index ceeb1ba..49246a1 100755
--- a/SConstruct
+++ b/SConstruct
@@ -348,6 +348,9 @@
     if GetOption('gold_linker'):
         main.Append(LINKFLAGS='-fuse-ld=gold')

+    main.Append(CCFLAGS=['-Wno-error=deprecated-declarations',
+                         '-Wno-error=deprecated',
+                        ])
 else:
     error('\n'.join((
           "Don't know what compiler options to use for your compiler.",

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

Gerrit-Project: public/gem5
Gerrit-Branch: release-staging-v21-2
Gerrit-Change-Id: Iafe37f3a26782b419a95a75586c649ecf1b4658b
Gerrit-Change-Number: 54843
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby Bruce <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
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