summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Tridgell <tridge@samba.org>2010-10-29 11:50:25 +1100
committerAndrew Tridgell <tridge@samba.org>2010-10-30 23:49:00 +1100
commitd489880ecd8b68aaabb6c154c85cc72a454d712a (patch)
treed38029e2bee25690c5a67e291c187d79c86b40a1
parent65743f932b511db009655847e77288c95c0aa525 (diff)
downloadsamba-d489880ecd8b68aaabb6c154c85cc72a454d712a.tar.gz
samba-d489880ecd8b68aaabb6c154c85cc72a454d712a.tar.bz2
samba-d489880ecd8b68aaabb6c154c85cc72a454d712a.zip
waf: added env.DEVELOPER_MODE flag
this makes it easier to add project rules that are only run in developer mode Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>
-rw-r--r--buildtools/wafsamba/samba_autoconf.py1
-rw-r--r--buildtools/wafsamba/symbols.py308
2 files changed, 309 insertions, 0 deletions
diff --git a/buildtools/wafsamba/samba_autoconf.py b/buildtools/wafsamba/samba_autoconf.py
index 34acfdd1a9..f987d1d41a 100644
--- a/buildtools/wafsamba/samba_autoconf.py
+++ b/buildtools/wafsamba/samba_autoconf.py
@@ -577,6 +577,7 @@ def SAMBA_CONFIG_H(conf, path=None):
# we add these here to ensure that -Wstrict-prototypes is not set during configure
conf.ADD_CFLAGS('-Wall -g -Wshadow -Wstrict-prototypes -Wpointer-arith -Wcast-qual -Wcast-align -Wwrite-strings -Werror-implicit-function-declaration -Wformat=2 -Wno-format-y2k',
testflags=True)
+ conf.env.DEVELOPER_MODE = True
if Options.options.picky_developer:
conf.ADD_CFLAGS('-Werror', testflags=True)
diff --git a/buildtools/wafsamba/symbols.py b/buildtools/wafsamba/symbols.py
new file mode 100644
index 0000000000..dfa3a8c61d
--- /dev/null
+++ b/buildtools/wafsamba/symbols.py
@@ -0,0 +1,308 @@
+# a waf tool to extract symbols from object files or libraries
+# using nm, producing a set of exposed defined/undefined symbols
+
+import Utils, Build, subprocess, Logs
+from samba_wildcard import fake_build_environment
+from samba_utils import *
+
+def symbols_extract(objfiles, dynamic=False):
+ '''extract symbols from objfile, returning a dictionary containing
+ the set of undefined and public symbols for each file'''
+
+ ret = {}
+
+ cmd = ["nm"]
+ if dynamic:
+ # needed for some .so files
+ cmd.append("-D")
+ cmd.extend(objfiles)
+
+ nmpipe = subprocess.Popen(cmd, stdout=subprocess.PIPE).stdout
+ if len(objfiles) == 1:
+ filename = objfiles[0]
+ ret[filename] = { "PUBLIC": set(), "UNDEFINED" : set()}
+
+ for line in nmpipe:
+ line = line.strip()
+ if line.endswith(':'):
+ filename = line[:-1]
+ ret[filename] = { "PUBLIC": set(), "UNDEFINED" : set() }
+ continue
+ cols = line.split(" ")
+ if cols == ['']:
+ continue
+ # see if the line starts with an address
+ if len(cols) == 3:
+ symbol_type = cols[1]
+ symbol = cols[2]
+ else:
+ symbol_type = cols[0]
+ symbol = cols[1]
+ if symbol_type in "BDGTRVWS":
+ # its a public symbol
+ ret[filename]["PUBLIC"].add(symbol)
+ elif symbol_type in "U":
+ ret[filename]["UNDEFINED"].add(symbol)
+
+ return ret
+
+
+def real_name(name):
+ if name.find(".objlist") != -1:
+ name = name[:-8]
+ return name
+
+
+def find_syslib_path(bld, libname, deps):
+ '''find the path to the syslib we will link against'''
+ # the strategy is to use the targets that depend on the library, and run ldd
+ # on it to find the real location of the library that is used
+
+ linkpath = deps[0].link_task.outputs[0].abspath(bld.env)
+
+ if libname == "python":
+ libname += bld.env.PYTHON_VERSION
+
+ ret = None
+
+ lddpipe = subprocess.Popen(['ldd', linkpath], stdout=subprocess.PIPE).stdout
+ for line in lddpipe:
+ line = line.strip()
+ cols = line.split(" ")
+ if len(cols) < 3 or cols[1] != "=>":
+ continue
+ if cols[0].startswith("lib%s." % libname.lower()):
+ ret = cols[2]
+ if cols[0].startswith("libc."):
+ # save this one too
+ bld.env.libc_path = cols[2]
+ return ret
+
+
+def build_symbol_sets(bld, tgt_list):
+ '''build the public_symbols and undefined_symbols attributes for each target'''
+
+ objlist = [] # list of object file
+ objmap = {} # map from object filename to target
+
+
+ for t in tgt_list:
+ t.public_symbols = set()
+ t.undefined_symbols = set()
+ for tsk in getattr(t, 'compiled_tasks', []):
+ for output in tsk.outputs:
+ objpath = output.abspath(bld.env)
+ objlist.append(objpath)
+ objmap[objpath] = t
+
+ symbols = symbols_extract(objlist)
+ for obj in objlist:
+ t = objmap[obj]
+ t.public_symbols = t.public_symbols.union(symbols[obj]["PUBLIC"])
+ t.undefined_symbols = t.public_symbols.union(symbols[obj]["UNDEFINED"])
+
+ t.undefined_symbols = t.undefined_symbols.difference(t.public_symbols)
+
+ # and the reverse map of public symbols to subsystem name
+ bld.env.symbol_map = {}
+
+ for t in tgt_list:
+ for s in t.public_symbols:
+ bld.env.symbol_map[s] = real_name(t.sname)
+
+ # now do the same for syslibs
+
+ # work out what syslibs we depend on, and what targets those are used in
+ syslibs = {}
+ objmap = {}
+ for t in tgt_list:
+ if getattr(t, 'uselib', []) and t.samba_type in [ 'LIBRARY', 'BINARY', 'PYTHON' ]:
+ for lib in t.uselib:
+ if lib in ['PYEMBED', 'PYEXT']:
+ lib = "python"
+ if not lib in syslibs:
+ syslibs[lib] = []
+ syslibs[lib].append(t)
+
+ # work out the paths to each syslib
+ syslib_paths = []
+ for lib in syslibs:
+ path = find_syslib_path(bld, lib, syslibs[lib])
+ if path is None:
+ print("Unable to find syslib path for %s used by %s" % lib)
+ if path is not None:
+ syslib_paths.append(path)
+ objmap[path] = lib
+
+ # add in libc
+ syslib_paths.append(bld.env.libc_path)
+ objmap[bld.env.libc_path] = 'c'
+
+ symbols = symbols_extract(syslib_paths, dynamic=True)
+
+ # keep a map of syslib names to public symbols
+ bld.env.syslib_symbols = {}
+ for lib in symbols:
+ bld.env.syslib_symbols[lib] = symbols[lib]["PUBLIC"]
+
+ # add to the map of symbols to dependencies
+ for lib in symbols:
+ for sym in symbols[lib]["PUBLIC"]:
+ bld.env.symbol_map[sym] = objmap[lib].lower()
+
+ # keep the libc symbols as well, as these are useful for some of the
+ # sanity checks
+ bld.env.libc_symbols = symbols[bld.env.libc_path]["PUBLIC"]
+
+ # a combined map of dependency name to public_symbols
+ bld.env.all_symbols = {}
+ for lib in bld.env.syslib_symbols:
+ bld.env.all_symbols[lib] = bld.env.syslib_symbols[lib]
+ for t in tgt_list:
+ name = real_name(t.sname)
+ if bld.name_to_obj(t.sname + '.objlist', bld.env):
+ continue
+ bld.env.all_symbols[name] = t.public_symbols
+
+
+def build_autodeps(bld, t):
+ '''build the set of dependencies for a target'''
+ deps = set()
+ name = real_name(t.sname)
+
+ targets = LOCAL_CACHE(bld, 'TARGET_TYPE')
+
+ for sym in t.undefined_symbols:
+ if sym in t.public_symbols:
+ continue
+ if sym in bld.env.symbol_map:
+ depname = bld.env.symbol_map[sym]
+ if depname == name:
+ # self dependencies aren't interesting
+ continue
+ if t.in_library == [depname]:
+ # no need to depend on the library we are part of
+ continue
+ if depname in ['c', 'python']:
+ # these don't go into autodeps
+ continue
+ if targets[depname] in [ 'SYSLIB' ]:
+ deps.add(depname)
+ continue
+ t2 = bld.name_to_obj(depname, bld.env)
+ if len(t2.in_library) != 1:
+ deps.add(depname)
+ continue
+ if t2.in_library == t.in_library:
+ # if we're part of the same library, we don't need to autodep
+ continue
+ print("adding library %s for symbol %s" % (t2.in_library[0], sym))
+ deps.add(t2.in_library[0])
+ t.autodeps = deps
+
+
+def build_library_names(bld, tgt_list):
+ '''add a in_library attribute to all targets that are part of a library'''
+ for t in tgt_list:
+ t.in_library = []
+
+ for t in tgt_list:
+ if t.samba_type in [ 'LIBRARY' ]:
+ for obj in t.samba_deps_extended:
+ t2 = bld.name_to_obj(obj, bld.env)
+ if t2 and t2.samba_type in [ 'SUBSYSTEM', 'ASN1' ]:
+ t2.in_library.append(t.sname)
+
+
+def check_library_deps(bld, t):
+ '''check that all the autodeps that have mutual dependency of this
+ target are in the same library as the target'''
+
+ name = real_name(t.sname)
+
+ if len(t.in_library) > 1:
+ Logs.warn("WARNING: Target '%s' in multiple libraries: %s" % (t.sname, t.in_library))
+
+ for dep in t.autodeps:
+ t2 = bld.name_to_obj(dep, bld.env)
+ if t2 is None:
+ continue
+ for dep2 in t2.autodeps:
+ if dep2 == name and t.in_library != t2.in_library:
+ Logs.error("Illegal mutual dependency %s <=> %s" % (name, real_name(t2.sname)))
+ Logs.error("Libraries must match. %s != %s" % (t.in_library, t2.in_library))
+ sys.exit(1)
+
+
+def check_syslib_collisions(bld, tgt_list):
+ '''check if a target has any symbol collisions with a syslib
+
+ We do not want any code in Samba to use a symbol name from a
+ system library. The chance of that causing problems is just too
+ high. Note that libreplace uses a rep_XX approach of renaming
+ symbols via macros
+ '''
+
+ has_error = False
+ for t in tgt_list:
+ for lib in bld.env.syslib_symbols:
+ common = t.public_symbols.intersection(bld.env.syslib_symbols[lib])
+ if common:
+ Logs.error("ERROR: Target '%s' has symbols '%s' which is also in syslib '%s'" % (t.sname, common, lib))
+ has_error = True
+ if has_error:
+ raise Utils.WafError("symbols in common with system libraries")
+
+
+def check_dep_list(bld, t):
+ '''check for depenencies that can be removed'''
+ if bld.name_to_obj(t.sname + ".objlist", bld.env):
+ return
+ deps = set(t.samba_deps)
+ diff = deps.difference(t.autodeps)
+ for d in ['replace']:
+ if d in diff:
+ diff.remove(d)
+ if diff:
+ Logs.info("Target '%s' could remove deps: %s" % (real_name(t.sname), " ".join(diff)))
+ diff = t.autodeps.difference(deps)
+ for d in diff:
+ Logs.info("Target '%s' should add dep '%s' for symbols %s" % (
+ real_name(t.sname), d, t.undefined_symbols.intersection(bld.env.all_symbols[d])))
+
+
+def symbols_autodep(task):
+ '''check the dependency lists'''
+ bld = task.env.bld
+ tgt_list = get_tgt_list(bld)
+
+ build_symbol_sets(bld, tgt_list)
+ build_library_names(bld, tgt_list)
+
+ t = bld.name_to_obj('SERVICE_SMB', bld.env)
+ build_autodeps(bld, t)
+ check_dep_list(bld, t)
+ return
+
+ for t in tgt_list:
+ t.autodeps = set()
+ if getattr(t, 'source', ''):
+ build_autodeps(bld, t)
+
+ for t in tgt_list:
+ check_library_deps(bld, t)
+
+ check_syslib_collisions(bld, tgt_list)
+
+
+ for t in tgt_list:
+ check_dep_list(bld, t)
+
+
+def AUTODEP(bld):
+ '''check our dependency lists'''
+ if bld.env.DEVELOPER_MODE:
+ bld.SET_BUILD_GROUP('final')
+ task = bld(rule=symbols_autodep, always=True, name='Autodep')
+ task.env.bld = bld
+Build.BuildContext.AUTODEP = AUTODEP