Revert "wafsamba: flags from enviroment are put before our own internal versions"
authorRalph Boehme <slow@samba.org>
Sat, 25 Apr 2015 01:38:48 +0000 (03:38 +0200)
committerRalph Böhme <slow@samba.org>
Sun, 26 Apr 2015 16:40:39 +0000 (18:40 +0200)
This reverts commit b2bb6aeb8057ac725f6ad12378344b201c3a3ba2.

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jelmer Vernooij <jelmer@samba.org>
Autobuild-User(master): Ralph Böhme <slow@samba.org>
Autobuild-Date(master): Sun Apr 26 18:40:39 CEST 2015 on sn-devel-104

buildtools/wafsamba/samba_optimisation.py

index 5def5803804ce9e4b809c7af406abe490b0820a3..51d514ec75a18463f2c427cf1da3cb955909f19a 100644 (file)
@@ -287,45 +287,3 @@ def samba_before_apply_obj_vars(self):
     for i in v['LIBPATH']:
         if is_standard_libpath(v, i):
             v['LIBPATH'].remove(i)
-
-@feature('cc')
-@before('apply_incpaths', 'apply_obj_vars_cc')
-def samba_stash_cppflags(self):
-    """Fix broken waf ordering of CPPFLAGS"""
-
-    self.env.SAVED_CPPFLAGS = self.env.CPPFLAGS
-    self.env.CPPFLAGS = []
-
-@feature('cc')
-@after('apply_incpaths', 'apply_obj_vars_cc')
-def samba_pop_cppflags(self):
-    """append stashed user CPPFLAGS after our internally computed flags"""
-
-    #
-    # Note that we don't restore the values to 'CPPFLAGS',
-    # but to _CCINCFLAGS instead.
-    #
-    # buildtools/wafadmin/Tools/cc.py defines the 'cc' task generator as
-    # '${CC} ${CCFLAGS} ${CPPFLAGS} ${_CCINCFLAGS} ${_CCDEFFLAGS} ${CC_SRC_F}${SRC} ${CC_TGT_F}${TGT}'
-    #
-    # Our goal is to effectively invert the order of ${CPPFLAGS} and
-    # ${_CCINCFLAGS}.
-    self.env.append_value('_CCINCFLAGS', self.env.SAVED_CPPFLAGS)
-    self.env.SAVED_CPPFLAGS = []
-
-@feature('cprogram', 'cshlib', 'cstaticlib')
-@before('apply_obj_vars', 'add_extra_flags')
-def samba_stash_linkflags(self):
-    """stash away LINKFLAGS in order to fix waf's broken ordering wrt or
-    user LDFLAGS"""
-
-    self.env.SAVE_LINKFLAGS = self.env.LINKFLAGS
-    self.env.LINKFLAGS = []
-
-@feature('cprogram', 'cshlib', 'cstaticlib')
-@after('apply_obj_vars', 'add_extra_flags')
-def samba_pop_linkflags(self):
-    """after apply_obj_vars append saved LDFLAGS"""
-
-    self.env.append_value('LINKFLAGS', self.env.SAVE_LINKFLAGS)
-    self.env.SAVE_LINKFLAGS = []