diff options
author | Gabe Black <gabeblack@google.com> | 2017-11-20 18:30:41 -0800 |
---|---|---|
committer | Gabe Black <gabeblack@google.com> | 2017-11-27 22:10:32 +0000 |
commit | 82f76b8df0f4e6f46f5b0322dd30e5585c8eb9f9 (patch) | |
tree | a3bf08cd03c912300723b3727a828b6cf78930b4 /src | |
parent | 08d60882d72e50498536b7d02f4b8838db8d81c4 (diff) | |
download | gem5-82f76b8df0f4e6f46f5b0322dd30e5585c8eb9f9.tar.xz |
scons: Remove the extra_deps option from the helper function make_obj.
This had only one use, and made the function more complicated than it
needed to be.
Change-Id: I22147a0c1239b457c577a6a24b423065e539833b
Reviewed-on: https://gem5-review.googlesource.com/5983
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>
Diffstat (limited to 'src')
-rwxr-xr-x | src/SConscript | 21 |
1 files changed, 8 insertions, 13 deletions
diff --git a/src/SConscript b/src/SConscript index 34bbcc02c..61af95b55 100755 --- a/src/SConscript +++ b/src/SConscript @@ -951,19 +951,12 @@ def makeEnv(env, label, objsfx, strip=False, disable_partial=False, **kwargs): new_env.Label = label new_env.Append(**kwargs) - def make_obj(source, static, extra_deps=None): - '''This function creates a scons node of the requested type, and sets - up any additional dependencies.''' - + def make_obj(source, static): + '''This function creates a scons node of the requested type.''' if static: - obj = new_env.StaticObject(source.tnode) + return new_env.StaticObject(source.tnode) else: - obj = new_env.SharedObject(source.tnode) - - if extra_deps: - new_env.Depends(obj, extra_deps) - - return obj + return new_env.SharedObject(source.tnode) lib_sources = Source.all.with_tag('gem5 lib') @@ -1008,10 +1001,12 @@ def makeEnv(env, label, objsfx, strip=False, disable_partial=False, **kwargs): partial = env.PartialShared(target=target, source=source_objs) shared_objs.append(partial) - static_date = make_obj(date_source, static=True, extra_deps=static_objs) + static_date = make_obj(date_source, static=True) + new_env.Depends(static_date, static_objs) static_objs.append(static_date) - shared_date = make_obj(date_source, static=False, extra_deps=shared_objs) + shared_date = make_obj(date_source, static=False) + new_env.Depends(shared_date, shared_objs) shared_objs.append(shared_date) # First make a library of everything but main() so other programs can |