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

Change subject: scons: Simplify the PySource class slightly.
......................................................................

scons: Simplify the PySource class slightly.

Demote the cpp attribute to a local variable, and get rid of the unused
"package" attribute.

Change-Id: I190792274ea9bdd9853aa3b6e07ce4151b378251
---
M src/SConscript
1 file changed, 4 insertions(+), 4 deletions(-)



diff --git a/src/SConscript b/src/SConscript
index 0aa4deb..a3e2a31 100644
--- a/src/SConscript
+++ b/src/SConscript
@@ -156,24 +156,24 @@
         if not os.path.exists(abspath):
             abspath = self.tnode.abspath

-        self.package = package
         self.modname = modname
         self.modpath = modpath
         self.abspath = abspath
-        self.cpp = File(self.filename + '.cc')

         PySource.modules[modpath] = self

+        cpp = File(self.filename + '.cc')
+
         overrides = {
                 'PYSOURCE_MODPATH': modpath,
                 'PYSOURCE_ABSPATH': abspath,
         }
-        marshal_env.Command(self.cpp, [ py_marshal, File(source) ],
+        marshal_env.Command(cpp, [ py_marshal, File(source) ],
                 MakeAction(embedPyFile, Transform("EMBED PY"),
                     varlist=overrides.keys()),
                 **overrides)
         if main['USE_PYTHON']:
-            Source(self.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/+/49388
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: I190792274ea9bdd9853aa3b6e07ce4151b378251
Gerrit-Change-Number: 49388
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <gabe.bl...@gmail.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to