Package: cdbs
Version: 0.4.51
Tags: patch
Severity: wishlist

I think it'd be useful if CDBS supported a DEB_DH_CLEAN_ARGS variable that would be passed to dh_clean, since I consider it cleaner than writing a new clean:: target.

I've attached a patch to do this.

It's probably also a good idea to use DEB_DH_CLEAN_ARGS for the clean:: target in the "Debhelper custom build rules" section of the documentation; I've not included that change in my patch.

        -Tim Abbott
Index: cdbs-0.4.51/1/rules/debhelper.mk.in
===================================================================
--- cdbs-0.4.51.orig/1/rules/debhelper.mk.in	2008-04-07 01:29:32.000000000 -0400
+++ cdbs-0.4.51/1/rules/debhelper.mk.in	2008-04-07 01:33:45.000000000 -0400
@@ -36,6 +36,8 @@
 #   Regular expressions matching files which should not have their permissions changed.
 # DEB_CLEAN_EXCLUDE
 #   Regular expressions matching files which should not be cleaned.
+# DEB_DH_CLEAN_ARGS
+#   Additional files that should be cleaned.
 # DEB_DH_ALWAYS_EXCLUDE
 #   Force builddeb to exclude files.  See the DH_ALWAYS_EXCLUDE section
 #   in debhelper(7) for more details.
@@ -134,11 +136,11 @@
 	$(if $(DEB_DH_COMPAT_DISABLE),,echo $(DH_COMPAT) >$@)
 
 clean::
-	dh_clean $(call cdbs_add_dashx,$(DEB_CLEAN_EXCLUDE))
+	dh_clean $(call cdbs_add_dashx,$(DEB_CLEAN_EXCLUDE)) $(DEB_DH_CLEAN_ARGS)
 
 common-install-prehook-arch common-install-prehook-indep:: common-install-prehook-impl
 common-install-prehook-impl::
-	dh_clean -k $(call cdbs_add_dashx,$(DEB_CLEAN_EXCLUDE))
+	dh_clean -k $(call cdbs_add_dashx,$(DEB_CLEAN_EXCLUDE)) $(DEB_DH_CLEAN_ARGS)
 	dh_installdirs -A $(DEB_INSTALL_DIRS_ALL)
 
 $(patsubst %,install/%,$(DEB_ALL_PACKAGES)) :: install/%:

Reply via email to