wafsamba: simplify SAMBA_PIDL_TABLES() rule
authorStefan Metzmacher <metze@samba.org>
Thu, 15 Nov 2018 12:37:58 +0000 (13:37 +0100)
committerAndrew Bartlett <abartlet@samba.org>
Tue, 20 Nov 2018 00:33:34 +0000 (01:33 +0100)
The builddir is not bin/default/ instead of just bin/,
so we don't need to strip 'default/' anymore.

And the '--output ${TGT}' part is not really implemented.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
buildtools/wafsamba/samba_pidl.py

index 04460d0e65665724e8afb195038fb9f8ada37572..1e55892127e5e538f19d96f87be1cf1d4212b68a 100644 (file)
@@ -121,6 +121,7 @@ Build.BuildContext.SAMBA_PIDL_LIST = SAMBA_PIDL_LIST
 @before('exec_rule')
 def collect(self):
     pidl_headers = LOCAL_CACHE(self.bld, 'PIDL_HEADERS')
+    # The first source is tables.pl itself
     self.source = Utils.to_list(self.source)
     for (name, hd) in pidl_headers.items():
         y = self.bld.get_tgen_by_name(name)
@@ -136,7 +137,7 @@ def SAMBA_PIDL_TABLES(bld, name, target):
     bld.SET_BUILD_GROUP('main')
     t = bld(
             features = 'collect',
-            rule     = '${PERL} ${SRC} --output ${TGT} | sed "s|default/||" > ${TGT}',
+            rule     = '${PERL} ${SRC} > ${TGT}',
             ext_out  = '.c',
             before   = 'c',
             update_outputs = True,