Bug 897909 - Cleanup some SIMPLE_PROGRAMS; r=gps

* * * FOLD
mozilla-spidermonkey · Aug 14, 2013 · 510bc70 · 510bc70
1 parent e4c44f4
commit 510bc70
Show file tree

Hide file tree

Showing 22 changed files with 179 additions and 165 deletions.
diff --git a/intl/uconv/tools/Makefile.in b/intl/uconv/tools/Makefile.in
@@ -10,17 +10,8 @@ VPATH		= @srcdir@
 
 include $(DEPTH)/config/autoconf.mk
 
-DISABLED_CSRCS		= \
-		umaptable.c \
-		$(NULL)
-
-SIMPLE_PROGRAMS	= $(DISABLED_CSRCS:.c=$(BIN_SUFFIX))
-
 include $(topsrcdir)/config/rules.mk
 
-INCLUDES	+= \
-		$(NULL)
-
 # needed for mac linux
 ifeq ($(OS_ARCH),Linux)
 ifneq ($(OS_RELEASE),1.2)

diff --git a/intl/uconv/tools/moz.build b/intl/uconv/tools/moz.build
@@ -4,11 +4,16 @@
 # License, v. 2.0. If a copy of the MPL was not distributed with this
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
+sources = [
+    'umaptable',
+]
+
 CSRCS += [
-    'umaptable.c',
+    '%s.c' % s for s in sources
 ]
 
 bin_suffix = CONFIG['BIN_SUFFIX']
 SIMPLE_PROGRAMS += [
-    "%s%s" % (fyl[0:-2], bin_suffix) for fyl in CSRCS
+    '%s%s' % (s, bin_suffix) for s in sources
 ]
+
diff --git a/netwerk/test/Makefile.in b/netwerk/test/Makefile.in
@@ -12,40 +12,6 @@ relativesrcdir = @relativesrcdir@
 FAIL_ON_WARNINGS := 1
 
 include $(DEPTH)/config/autoconf.mk
-
-cppsrcs = \
-  TestIncrementalDownload.cpp \
-  TestStreamLoader.cpp \
-  PropertiesTest.cpp \
-  urltest.cpp \
-  TestCallbacks.cpp \
-  TestPageLoad.cpp \
-  TestURLParser.cpp \
-  TestStandardURL.cpp \
-  TestUpload.cpp \
-  TestBlockingSocket.cpp \
-  TestDNS.cpp \
-  TestOpen.cpp \
-  TestCookie.cpp \
-  TestServ.cpp \
-  ReadNTLM.cpp \
-  TestProtocols.cpp \
-  $(NULL)
-
-DISABLED_SIMPLE_PROGRAMS := $(cppsrcs:.cpp=$(BIN_SUFFIX))
-
-# XXX Make this work in libxul builds.
-#SIMPLE_PROGRAMS += \
-#		TestPerf$(BIN_SUFFIX) \
-#		TestIDN$(BIN_SUFFIX) \
-#		TestSocketTransport$(BIN_SUFFIX) \
-#		TestStreamTransport$(BIN_SUFFIX) \
-#		TestStreamChannel$(BIN_SUFFIX) \
-#		TestStreamPump$(BIN_SUFFIX) \
-#		TestIOThreads$(BIN_SUFFIX) \
-#		TestUDPSocketProvider$(BIN_SUFFIX) \
-#		$(NULL)
-
 include $(topsrcdir)/config/config.mk
 
 LIBS		= $(XPCOM_LIBS) \

diff --git a/netwerk/test/moz.build b/netwerk/test/moz.build
@@ -14,28 +14,44 @@ XPCSHELL_TESTS_MANIFESTS += ['unit/xpcshell.ini']
 if CONFIG['OS_ARCH'] != 'Darwin':
     XPCSHELL_TESTS_MANIFESTS += ['unit_ipc/xpcshell.ini']
 
+sources = [
+    'PropertiesTest',
+    'ReadNTLM',
+    'TestBlockingSocket',
+    'TestCallbacks',
+    'TestCookie',
+    'TestDNS',
+    'TestIncrementalDownload',
+    'TestOpen',
+    'TestPageLoad',
+    'TestProtocols',
+    'TestServ',
+    'TestStandardURL',
+    'TestStreamLoader',
+    'TestURLParser',
+    'TestUpload',
+    'urltest',
+]
+
+# XXX Make this work in libxul builds.
+#sources += [
+#    TestIDN',
+#    TestIOThreads',
+#    TestPerf',
+#    TestSocketTransport',
+#    TestStreamChannel',
+#    TestStreamPump',
+#    TestStreamTransport',
+#    TestUDPSocketProvider',
+#]
+
 CPP_SOURCES += [
-    'PropertiesTest.cpp',
-    'ReadNTLM.cpp',
-    'TestBlockingSocket.cpp',
-    'TestCallbacks.cpp',
-    'TestCookie.cpp',
-    'TestDNS.cpp',
-    'TestIncrementalDownload.cpp',
-    'TestOpen.cpp',
-    'TestPageLoad.cpp',
-    'TestProtocols.cpp',
-    'TestServ.cpp',
-    'TestStandardURL.cpp',
-    'TestStreamLoader.cpp',
-    'TestURLParser.cpp',
-    'TestUpload.cpp',
-    'urltest.cpp',
+    '%s.cpp' % s for s in sources
 ]
 
 bin_suffix = CONFIG['BIN_SUFFIX']
 SIMPLE_PROGRAMS += [
-    "%s%s" % (x[0:-4], bin_suffix) for x in CPP_SOURCES
+    '%s%s' % (s, bin_suffix) for s in sources
 ]
 
 CPP_UNIT_TESTS += [

diff --git a/security/manager/ssl/tests/unit/test_ocsp_stapling/Makefile.in b/security/manager/ssl/tests/unit/test_ocsp_stapling/Makefile.in
@@ -13,12 +13,6 @@ FAIL_ON_WARNINGS := 1
 
 include $(DEPTH)/config/autoconf.mk
 
-CPPSRCS = \
-  OCSPStaplingServer.cpp \
-  $(NULL)
-
-SIMPLE_PROGRAMS := $(CPPSRCS:.cpp=$(BIN_SUFFIX))
-
 include $(topsrcdir)/config/config.mk
 
 LIBS = \

diff --git a/security/manager/ssl/tests/unit/test_ocsp_stapling/moz.build b/security/manager/ssl/tests/unit/test_ocsp_stapling/moz.build
@@ -6,3 +6,15 @@
 
 MODULE = 'pipnss'
 
+sources = [
+    'OCSPStaplingServer',
+]
+
+CPP_SOURCES += [
+    '%s.cpp' % s for s in sources
+]
+
+bin_suffix = CONFIG['BIN_SUFFIX']
+SIMPLE_PROGRAMS += [
+    '%s%s' % (s, bin_suffix) for s in sources
+]
diff --git a/toolkit/mozapps/plugins/tests/Makefile.in b/toolkit/mozapps/plugins/tests/Makefile.in
@@ -21,17 +21,15 @@ MOCHITEST_BROWSER_FILES = \
   BadExtension.xpi \
   $(NULL)
 
-SIMPLE_PROGRAMS = $(CPPSRCS:.cpp=$(BIN_SUFFIX))
-
-PROGRAMS = $(SIMPLE_PROGRAMS:%$(BIN_SUFFIX)=$(TESTROOT)/%)
+programs = $(SIMPLE_PROGRAMS:%$(BIN_SUFFIX)=$(TESTROOT)/%)
 
 include $(topsrcdir)/config/rules.mk
 
-$(PROGRAMS): $(TESTROOT)
+$(programs): $(TESTROOT)
 $(TESTROOT):
 	$(NSINSTALL) -D $@
 
-$(PROGRAMS): $(TESTROOT)/% : %$(BIN_SUFFIX)
+$(programs): $(TESTROOT)/% : %$(BIN_SUFFIX)
 	cp $< $@
 
-libs:: $(PROGRAMS)
+libs:: $(programs)
diff --git a/toolkit/mozapps/plugins/tests/moz.build b/toolkit/mozapps/plugins/tests/moz.build
@@ -6,8 +6,16 @@
 
 MODULE = 'test_plugins'
 
+sources = [
+    'BadPlugin',
+    'GoodPlugin',
+]
+
 CPP_SOURCES += [
-    'BadPlugin.cpp',
-    'GoodPlugin.cpp',
+    '%s.cpp' % s for s in sources
 ]
 
+bin_suffix = CONFIG['BIN_SUFFIX']
+SIMPLE_PROGRAMS += [
+    '%s%s' % (s, bin_suffix) for s in sources
+]
diff --git a/toolkit/xre/test/win/Makefile.in b/toolkit/xre/test/win/Makefile.in
@@ -11,8 +11,6 @@ include $(DEPTH)/config/autoconf.mk
 
 OS_LIBS += $(call EXPAND_LIBNAME,comctl32 ws2_32 shell32)
 
-SIMPLE_PROGRAMS	= $(CPPSRCS:.cpp=$(BIN_SUFFIX))
-
 LOCAL_INCLUDES += \
   -I$(srcdir) \
   -I$(topsrcdir)/toolkit/xre \

diff --git a/toolkit/xre/test/win/moz.build b/toolkit/xre/test/win/moz.build
@@ -6,8 +6,17 @@
 
 MODULE = 'test_xulapp'
 
+sources = [
+    'TestDllInterceptor',
+    'TestXREMakeCommandLineWin',
+]
+
 CPP_SOURCES += [
-    'TestDllInterceptor.cpp',
-    'TestXREMakeCommandLineWin.cpp',
+    '%s.cpp' % s for s in sources
+]
+
+bin_suffix = CONFIG['BIN_SUFFIX']
+SIMPLE_PROGRAMS += [
+    '%s%s' % (s, bin_suffix) for s in sources
 ]
 
diff --git a/tools/trace-malloc/Makefile.in b/tools/trace-malloc/Makefile.in
@@ -10,22 +10,6 @@ VPATH		= @srcdir@
 
 include $(DEPTH)/config/autoconf.mk
 
-
-SIMPLECSRCS	+= \
-		leakstats.c \
-		tmstats.c \
-		$(NULL)
-
-SIMPLECPPSRCS	= \
-		leaksoup.cpp \
-		bloatblame.cpp \
-		$(NULL)
-
-SIMPLE_PROGRAMS	= \
-		$(SIMPLECSRCS:.c=$(BIN_SUFFIX)) \
-		$(SIMPLECPPSRCS:.cpp=$(BIN_SUFFIX)) \
-		$(NULL)
-
 EXTRACSRCS = tmreader.c
 EXTRACPPSRCS = adreader.cpp
 
@@ -40,7 +24,9 @@ PROGCSRCS = \
 PROGOBJS = $(PROGCSRCS:.c=.$(OBJ_SUFFIX))
 endif
 
-CSRCS = $(SIMPLECSRCS) $(EXTRACSRCS) $(PROGCSRCS)
+CSRCS += $(EXTRACSRCS) $(PROGCSRCS)
+CPPSRCS += $(EXTRACPPSRCS)
+
 ifeq ($(OS_ARCH),WINNT)
 LOCAL_INCLUDES	+= -I$(topsrcdir)/config/os2
 endif

diff --git a/tools/trace-malloc/moz.build b/tools/trace-malloc/moz.build
@@ -7,7 +7,33 @@
 if not CONFIG['MOZ_PROFILE_GENERATE']:
     PROGRAM = 'spacetrace'
 
+bin_suffix = CONFIG['BIN_SUFFIX']
+
+
+simple_c_sources = [
+    'leakstats',
+    'tmstats',
+]
+
+CSRCS += [
+    '%s.c' % s for s in simple_c_sources
+]
+
+SIMPLE_PROGRAMS += [
+    '%s%s' % (s, bin_suffix) for s in simple_c_sources
+]
+
+
+simple_cpp_sources = [
+    'bloatblame',
+    'leaksoup',
+]
+
 CPP_SOURCES += [
-    '$(EXTRACPPSRCS)',
-    '$(SIMPLECPPSRCS)',
+    '%s.cpp' % s for s in simple_cpp_sources
 ]
+
+SIMPLE_PROGRAMS += [
+    '%s%s' % (s, bin_suffix) for s in simple_cpp_sources
+]
+
diff --git a/uriloader/exthandler/tests/Makefile.in b/uriloader/exthandler/tests/Makefile.in
@@ -12,8 +12,6 @@ include $(DEPTH)/config/autoconf.mk
 
 FAIL_ON_WARNINGS = 1
 
-SIMPLE_PROGRAMS = $(CPPSRCS:.cpp=$(BIN_SUFFIX))
-
 include $(topsrcdir)/config/config.mk
 
 LIBS +=		\

diff --git a/uriloader/exthandler/tests/moz.build b/uriloader/exthandler/tests/moz.build
@@ -14,6 +14,16 @@ XPCSHELL_TESTS_MANIFESTS += ['unit/xpcshell.ini']
 if CONFIG['OS_ARCH'] != 'Darwin':
     XPCSHELL_TESTS_MANIFESTS += ['unit_ipc/xpcshell.ini']
 
+sources = [
+    'WriteArgument',
+]
+
 CPP_SOURCES += [
-    'WriteArgument.cpp',
+    '%s.cpp' % s for s in sources
 ]
+
+bin_suffix = CONFIG['BIN_SUFFIX']
+SIMPLE_PROGRAMS += [
+    '%s%s' % (s, bin_suffix) for s in sources
+]
+
diff --git a/xpcom/reflect/xptcall/tests/Makefile.in b/xpcom/reflect/xptcall/tests/Makefile.in
@@ -10,8 +10,6 @@ VPATH		= @srcdir@
 
 include $(DEPTH)/config/autoconf.mk
 
-SIMPLE_PROGRAMS	= TestXPTCInvoke$(BIN_SUFFIX)
-
 LIBS		= \
 		$(XPCOM_GLUE_LDOPTS) \
 		$(NSPR_LIBS) \

diff --git a/xpcom/reflect/xptcall/tests/moz.build b/xpcom/reflect/xptcall/tests/moz.build
@@ -6,7 +6,15 @@
 
 MODULE = 'TestXPTC'
 
+sources = [
+    'TestXPTCInvoke',
+]
+
 CPP_SOURCES += [
-    'TestXPTCInvoke.cpp',
+    '%s.cpp' % s for s in sources
+]
+
+SIMPLE_PROGRAMS += [
+    '%s%s' % (s, bin_suffix) for s in sources
 ]
 
diff --git a/xpcom/tests/Makefile.in b/xpcom/tests/Makefile.in
@@ -14,35 +14,6 @@ FAIL_ON_WARNINGS = 1
 
 VPATH += $(topsrcdir)/build
 
-# This can be deleted when SIMPLE_PROGRAMS is in moz.build
-cppsrcs = \
-  nsIFileEnumerator.cpp \
-  TestCallTemplates.cpp \
-  TestINIParser.cpp \
-  TestPRIntN.cpp \
-  TestRacingServiceManager.cpp \
-  TestRegistrationOrder.cpp \
-  TestThreadPoolListener.cpp \
-  TestTimers.cpp \
-  TestBlockingProcess.cpp \
-  TestQuickReturn.cpp \
-  TestArguments.cpp \
-  TestUnicodeArguments.cpp \
-  $(NULL)
-
-# This can be deleted when SIMPLE_PROGRAMS is in moz.build
-ifeq ($(OS_TARGET),WINNT)
-cppsrcs += TestBase64.cpp
-endif
-
-# This can be deleted when SIMPLE_PROGRAMS is in moz.build
-ifdef WRAP_STL_INCLUDES
-cppsrcs += TestSTLWrappers.cpp
-endif
-
-# TODO: Use CPPSRCS from moz.build
-SIMPLE_PROGRAMS	:= $(cppsrcs:.cpp=$(BIN_SUFFIX))
-
 include $(topsrcdir)/config/config.mk
 
 LIBS += $(XPCOM_LIBS)