Make "make clean" rules a bit more standard.

Summary: make-generic is some sort of internal undocumented thing. make-local is what is supposed to be used for this stuff. Also use $(RM) instead of "rm -f". Reviewed By: marc Test Plan: make clean git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@665498 13f79535-47bb-0310-9956-ffa450edef68
twitter-forks · Feb 24, 2008 · d55f41d · d55f41d
1 parent b556465
commit d55f41d
Show file tree

Hide file tree

Showing 11 changed files with 12 additions and 12 deletions.
diff --git a/compiler/cpp/Makefile.am b/compiler/cpp/Makefile.am
@@ -68,4 +68,4 @@ thrift_LDADD = @LEXLIB@
 EXTRA_DIST = README
 
 clean-local:
-	rm -rf thriftl.cc thrifty.cc thrifty.h
+	$(RM) thriftl.cc thrifty.cc thrifty.h
diff --git a/lib/java/Makefile.am b/lib/java/Makefile.am
@@ -4,7 +4,7 @@ all:
 install-exec-hook:
 	$(ANT) install
 
-clean-generic:
+clean-local:
 	$(ANT) clean
 
 check-local: all
diff --git a/lib/py/Makefile.am b/lib/py/Makefile.am
@@ -11,7 +11,7 @@ all:
 install-exec-hook:
 	$(PYTHON) setup.py install --root=$(DESTDIR) --prefix=$(PY_PREFIX) $(PYTHON_SETUPUTIL_ARGS)
 
-clean-generic:
-	rm -rf build
+clean-local:
+	$(RM) -r build
 
 check-local: all
diff --git a/test/Makefile.am b/test/Makefile.am
@@ -81,4 +81,4 @@ INCLUDES = \
 AM_CPPFLAGS = $(BOOST_CPPFLAGS)
 
 clean-local:
-	rm -rf gen-cpp
+	$(RM) -r gen-cpp
diff --git a/test/java/Makefile.am b/test/java/Makefile.am
@@ -1,6 +1,6 @@
 check:
 	$(ANT) test
 
-clean-generic:
+clean-local:
 	$(ANT) clean
 
diff --git a/test/perl/Makefile b/test/perl/Makefile
@@ -13,4 +13,4 @@ all: ../ThriftTest.thrift
 	$(THRIFT) -perl ../ThriftTest.thrift
 
 clean:
-	rm -fr gen-perl
+	$(RM) -r gen-perl
diff --git a/test/php/Makefile b/test/php/Makefile
@@ -22,4 +22,4 @@ stubs-inline: ../ThriftTest.thrift
 	$(THRIFT) --phpi ../ThriftTest.thrift
 
 clean:
-	rm -fr gen-php gen-phpi
+	$(RM) -r gen-php gen-phpi
diff --git a/test/py/Makefile.am b/test/py/Makefile.am
@@ -24,4 +24,4 @@ gen-py/ThriftTest/__init__.py: ../ThriftTest.thrift
 	$(THRIFT) -py $<
 
 clean-local:
-	-rm -rf gen-py
+	$(RM) -r gen-py
diff --git a/test/rb/Makefile b/test/rb/Makefile
@@ -19,4 +19,4 @@ tests: stubs
 	ruby TestSuite.rb
 
 clean:
-	rm -fr gen-rb
+	$(RM) -r gen-rb
diff --git a/test/threads/Makefile b/test/threads/Makefile
@@ -46,4 +46,4 @@ client: stubs
 	g++ -o ThreadsClient $(CFL) ThreadsClient.cpp ./gen-cpp/ThreadsTest.cpp ./gen-cpp/ThreadsTest_types.cpp
 
 clean:
-	rm -fr *.o ThreadsServer ThreadsClient gen-cpp gen-py
+	$(RM) -r *.o ThreadsServer ThreadsClient gen-cpp gen-py
diff --git a/tutorial/cpp/Makefile b/tutorial/cpp/Makefile
@@ -13,4 +13,4 @@ client: CppClient.cpp
 	g++ -o CppClient -I${THRIFT_DIR} -I${BOOST_DIR}  -I../gen-cpp -L${LIB_DIR} -lthrift CppClient.cpp ${GEN_SRC}
 
 clean:
-	rm -fr CppClient CppServer
+	$(RM) -r CppClient CppServer