This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Patch] cygport: Allow to clear default_excludes


The cygport 0.3.8 "default_excludes" list is not suitable for all projects, in particular when aclocal.m4 is used in a non-automake project like bash (http://sourceware.org/ml/cygwin/2007-03/msg00613.html).

With the attached, the default_excludes list is cleared when when DIFF_EXCLUDES starts with '!' (Syntax borrowed from .cvsignore ;-)
Then all necessary excludes except CYGWIN-PATCHES must be specified, e.g:


DIFF_EXCLUDES="! config.* install-sh missing ..."

The patch also adds a missing blank for DIFF_EXCLUDES="-x foo ...".

Christian

--- cygport-0.3.8	2008-02-14 07:53:41.001000000 +0100
+++ cygport	2008-03-05 12:52:32.968750000 +0100
@@ -1770,14 +1770,24 @@ __pkg_diff() {
 		fi
 	fi
 
-	for exclude in ${default_excludes}
-	do
-		diff_excludes_x+=" -x ${exclude}";
-	done
+	# ignore defaults if DIFF_EXCLUDES starts with '!'
+	case ${DIFF_EXCLUDES} in
+		!*)
+			diff_excludes_x=" -x CYGWIN-PATCHES"
+			DIFF_EXCLUDES="${DIFF_EXCLUDES#!}"
+			DIFF_EXCLUDES="${DIFF_EXCLUDES# }"
+			;;
+		*)
+			for exclude in ${default_excludes}
+			do
+				diff_excludes_x+=" -x ${exclude}";
+			done
+			;;
+	esac
 
 	case ${DIFF_EXCLUDES} in
 		-x*)
-			diff_excludes_x+="${DIFF_EXCLUDES}";
+			diff_excludes_x+=" ${DIFF_EXCLUDES}";
 			;;
 		*)
 			for exclude in ${DIFF_EXCLUDES}

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]