tags 504108 +patch

thanks



Would it be possible to get this bug fixed? preferably in the Lenny release if 
you can get the release managers to accept it.


Currently the DEB_BUILD_OPTIONS checks in debian/rules do not work.  If 
someone specifies more than one option, the options handles directly in 
debian/rules will not work.  (ex nostrip and nocheck together)

nostrip will be handled by debhelper properly, but nocheck will not.

Attached is a patch to fix the bug, sorry it is applied to the rules file, and 
not the debian directory.




-- 
Stephen Depooter
<[email protected]>
Index: rules
===================================================================
--- rules	(revision 1391)
+++ rules	(working copy)
@@ -28,7 +28,7 @@
 endif
 
 MAKE_TEST_TARGET=test-force
-ifneq ($(findstring $(DEB_BUILD_OPTIONS),fulltest),)
+ifneq ($(findstring fulltest,$(DEB_BUILD_OPTIONS)),)
 # make test-bt is the testsuite run by the MySQL build team 
 # before a release, but it is long
     MAKE_TEST_TARGET=test-bt
@@ -131,7 +131,7 @@
 
 	$(MAKE) $(MAKE_J)
 
-ifeq ($(findstring $(DEB_BUILD_OPTIONS),nocheck),)
+ifeq ($(findstring nocheck,$(DEB_BUILD_OPTIONS)),)
 	if [ ! -f testsuite-stamp ] ; then \
 		$(MAKE) $(MAKE_TEST_TARGET) || $(TESTSUITE_FAIL_CMD) ; \
 	fi

Reply via email to