waf: don't auto-depend on subsystems
authorAndrew Tridgell <tridge@samba.org>
Sat, 30 Oct 2010 00:19:08 +0000 (11:19 +1100)
committerAndrew Tridgell <tridge@samba.org>
Sat, 30 Oct 2010 12:49:00 +0000 (23:49 +1100)
this is part of an effort to reduce the amount of automatic dependency
munging we do. When working on the correct dependency tree for Samba
it is easier to add the right dependencies in the wscript files than
rely on them being added by the waf rules. This change removes the
automatic dependency of modules on their subsystem. Many modules don't
actually need this dependency anyway, so it is better to add it where
they are needed (for example, for ldb modules)

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

buildtools/wafsamba/samba_deps.py

index 3bcf8fd92ef6b56807fdc609f147918809f482ef..56af446df316e744cd75e6a242dd2bd9e9a8688c 100644 (file)
@@ -83,13 +83,6 @@ def expand_subsystem_deps(bld):
                 # subsystem it is part of needs to have it as a dependency, so targets
                 # that depend on this subsystem get the modules of that subsystem
                 subsystem.samba_deps_extended.append(module_name)
-            module = bld.name_to_obj(module_name, bld.env)
-            bld.ASSERT(module is not None, "Unable to find module %s in subsystem %s" % (module_name, subsystem_name))
-            module.samba_includes_extended.extend(subsystem.samba_includes_extended)
-            if targets[subsystem_name] in ['SUBSYSTEM']:
-                # if a subsystem is a plain object type (not a library) then any modules
-                # in that subsystem need to depend on the subsystem
-                module.samba_deps_extended.extend(subsystem.samba_deps_extended)
         subsystem.samba_deps_extended = unique_list(subsystem.samba_deps_extended)