Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/56341 )
Change subject: scons: Exclude parsetab.py when collecting python deps.
......................................................................
scons: Exclude parsetab.py when collecting python deps.
Because we don't have a good way to actually figure out what python
files we depend on, we have to use Glob and wildcard matching to depend
on all potential python files. Unfortunately that will pick up the
parsetab.py file that ply generates, which is a cached intermediate file
and not an input.
Change-Id: Id3dc0083e97c145deca04939182157623d6b780f
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/56341
Maintainer: Gabe Black <gabe.bl...@gmail.com>
Tested-by: kokoro <noreply+kok...@google.com>
Reviewed-by: Jason Lowe-Power <power...@gmail.com>
---
M src/arch/SConscript
1 file changed, 21 insertions(+), 1 deletion(-)
Approvals:
Jason Lowe-Power: Looks good to me, approved
Gabe Black: Looks good to me, approved
kokoro: Regressions pass
diff --git a/src/arch/SConscript b/src/arch/SConscript
index 3034dac..0178750 100644
--- a/src/arch/SConscript
+++ b/src/arch/SConscript
@@ -104,7 +104,8 @@
# output from the ISA description (*.isa) files.
#
-parser_files = Glob('isa_parser/*.py')
+# parsetab.py is a file generated by PLY which we don't want to add as a
dep.
+parser_files = Glob('isa_parser/*.py', exclude=['*/parsetab.py'])
micro_asm_py = File('micro_asm.py')
# import ply here because SCons screws with sys.path when performing
actions.
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/56341
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: Id3dc0083e97c145deca04939182157623d6b780f
Gerrit-Change-Number: 56341
Gerrit-PatchSet: 6
Gerrit-Owner: Gabe Black <gabe.bl...@gmail.com>
Gerrit-Reviewer: Gabe Black <gabe.bl...@gmail.com>
Gerrit-Reviewer: Jason Lowe-Power <power...@gmail.com>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-MessageType: merged
_______________________________________________
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
