Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/49389 )

Change subject: scons: Use source filters to exclude python source if --without-python.
......................................................................

scons: Use source filters to exclude python source if --without-python.

We were simply not declaring the source files for PySource files if
built --without-python. Instead, we should declare them, but then
explicitly exclude them if that option is set.

Since we're already doing that, we can simply remove the check from the
PySource constructor.

Change-Id: I437ebeee1082fa00065bedd61f91d5721b915ae5
---
M src/SConscript
1 file changed, 1 insertion(+), 2 deletions(-)



diff --git a/src/SConscript b/src/SConscript
index a3e2a31..cbc398a 100644
--- a/src/SConscript
+++ b/src/SConscript
@@ -172,8 +172,7 @@
                 MakeAction(embedPyFile, Transform("EMBED PY"),
                     varlist=overrides.keys()),
                 **overrides)
-        if main['USE_PYTHON']:
-            Source(cpp, tags=self.tags, add_tags='python')
+        Source(cpp, tags=self.tags, add_tags='python')

 class SimObject(PySource):
     '''Add a SimObject python file as a python source object and add

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/49389
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: I437ebeee1082fa00065bedd61f91d5721b915ae5
Gerrit-Change-Number: 49389
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <[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