PERFORCE change 118482 for review

Gabor Kovesdan gabor at FreeBSD.org
Fri Apr 20 15:25:56 UTC 2007


http://perforce.freebsd.org/chv.cgi?CH=118482

Change 118482 by gabor at gabor_server on 2007/04/20 15:25:18

	Style and whitespace

Affected files ...

.. //depot/projects/soc2006/gabor_destdir/Mk/bsd.port.mk#13 edit

Differences ...

==== //depot/projects/soc2006/gabor_destdir/Mk/bsd.port.mk#13 (text+ko) ====

@@ -3170,8 +3170,8 @@
 do-fetch:
 	@${MKDIR} ${_DISTDIR}
 	@cd ${_DISTDIR};\
-	 ${_MASTER_SITES_ENV} ; \
-	 for _file in ${DISTFILES}; do \
+	${_MASTER_SITES_ENV} ; \
+	for _file in ${DISTFILES}; do \
 		file=$${_file%%:*}; \
 		if [ $$_file = $$file ];	then	\
 			select='';	\
@@ -3192,7 +3192,7 @@
 				${ECHO_MSG} "=> Perhaps a filesystem (most likely a CD) isn't mounted?"; \
 				${ECHO_MSG} "=> Please correct this problem and try again."; \
 				exit 1; \
-			fi ; \
+			fi; \
 			if [ -f ${MD5_FILE} -a "x${NO_CHECKSUM}" = "x" ]; then \
 				_md5sum=`alg=MD5; ${DISTINFO_DATA}`; \
 				if [ -z "$$_md5sum" ]; then \
@@ -3213,13 +3213,13 @@
 					if [ ! -z \$${_MASTER_SITES_$${group}} ] ; then \
 						eval ___MASTER_SITES_TMP="\$${_MASTER_SITES_$${group}}" ; \
 						__MASTER_SITES_TMP="$${__MASTER_SITES_TMP} $${___MASTER_SITES_TMP}" ; \
-					fi \
+					fi; \
 				done; \
 				___MASTER_SITES_TMP= ; \
 				SORTED_MASTER_SITES_CMD_TMP="${ECHO_CMD} ${_MASTER_SITE_OVERRIDE} `${ECHO_CMD} $${__MASTER_SITES_TMP} | ${AWK} '${MASTER_SORT_AWK:S|\\|\\\\|g}'` ${_MASTER_SITE_BACKUP}" ; \
 			else \
 				SORTED_MASTER_SITES_CMD_TMP="${SORTED_MASTER_SITES_DEFAULT_CMD}" ; \
-			fi ; \
+			fi; \
 			for site in `eval $$SORTED_MASTER_SITES_CMD_TMP ${_RANDOMIZE_SITES}`; do \
 			    ${ECHO_MSG} "=> Attempting to fetch from $${site}."; \
 				CKSIZE=`alg=SIZE; ${DISTINFO_DATA}`; \
@@ -3230,17 +3230,17 @@
 				esac; \
 				if ${SETENV} ${FETCH_ENV} ${FETCH_CMD} ${FETCH_BEFORE_ARGS} $${args} ${FETCH_AFTER_ARGS}; then \
 					continue 2; \
-				fi \
+				fi; \
 			done; \
 			${ECHO_MSG} "=> Couldn't fetch it - please try to retrieve this";\
 			${ECHO_MSG} "=> port manually into ${_DISTDIR} and try again."; \
 			exit 1; \
-	    fi \
+	    fi; \
 	 done
 .if defined(PATCHFILES)
 	@cd ${_DISTDIR};\
 	${_PATCH_SITES_ENV} ; \
-	 for _file in ${PATCHFILES}; do \
+	for _file in ${PATCHFILES}; do \
 		file=`${ECHO_CMD} $$_file | ${SED} -E -e 's/:[^:]+$$//'` ; \
 		select=`${ECHO_CMD} $${_file#$${file}} | ${SED} -e 's/^://' -e 's/,/ /g'` ; \
 		force_fetch=false; \
@@ -3257,7 +3257,7 @@
 				${ECHO_MSG} "=> Perhaps a filesystem (most likely a CD) isn't mounted?"; \
 				${ECHO_MSG} "=> Please correct this problem and try again."; \
 				exit 1; \
-			fi ; \
+			fi; \
 			${ECHO_MSG} "=> $$file doesn't seem to exist in ${_DISTDIR}."; \
 			if [ ! -z "$$select" ] ; then \
 				__PATCH_SITES_TMP= ; \
@@ -3265,13 +3265,13 @@
 					if [ ! -z \$${_PATCH_SITES_$${group}} ] ; then \
 						eval ___PATCH_SITES_TMP="\$${_PATCH_SITES_$${group}}" ; \
 						__PATCH_SITES_TMP="$${__PATCH_SITES_TMP} $${___PATCH_SITES_TMP}" ; \
-					fi \
+					fi; \
 				done; \
 				___PATCH_SITES_TMP= ; \
 				SORTED_PATCH_SITES_CMD_TMP="${ECHO_CMD} ${_MASTER_SITE_OVERRIDE} `${ECHO_CMD} $${__PATCH_SITES_TMP} | ${AWK} '${MASTER_SORT_AWK:S|\\|\\\\|g}'` ${_MASTER_SITE_BACKUP}" ; \
 			else \
 				SORTED_PATCH_SITES_CMD_TMP="${SORTED_PATCH_SITES_DEFAULT_CMD}" ; \
-			fi ; \
+			fi; \
 			for site in `eval $$SORTED_PATCH_SITES_CMD_TMP`; do \
 			    ${ECHO_MSG} "=> Attempting to fetch from $${site}."; \
 				CKSIZE=`alg=SIZE; ${DISTINFO_DATA}`; \
@@ -3282,12 +3282,12 @@
 				esac; \
 				if ${SETENV} ${FETCH_ENV} ${FETCH_CMD} ${FETCH_BEFORE_ARGS} $${args} ${FETCH_AFTER_ARGS}; then \
 					continue 2; \
-				fi \
+				fi; \
 			done; \
 			${ECHO_MSG} "=> Couldn't fetch it - please try to retrieve this";\
 			${ECHO_MSG} "=> port manually into ${_DISTDIR} and try again."; \
 			exit 1; \
-	    fi \
+		fi; \
 	 done
 .endif
 .endif
@@ -3302,7 +3302,7 @@
 		if ! (cd ${WRKDIR} && ${EXTRACT_CMD} ${EXTRACT_BEFORE_ARGS} ${_DISTDIR}/$$file ${EXTRACT_AFTER_ARGS});\
 		then \
 			exit 1; \
-		fi \
+		fi; \
 	done
 .if !defined(EXTRACT_PRESERVE_OWNERSHIP)
 	@if [ `${ID} -u` = 0 ]; then \
@@ -3622,7 +3622,7 @@
 								fi; \
 						fi; \
 				done; \
-		fi ; \
+		fi; \
 		if [ -d ${PKG_DBDIR}/${PKGNAME} -o -n "$${found_package}" ]; then \
 				if [ -d ${PKG_DBDIR}/${PKGNAME} ]; then \
 					${ECHO_CMD} "===>   ${PKGNAME} is already installed"; \
@@ -4219,13 +4219,13 @@
 					if [ ! -z \$${_MASTER_SITES_$${group}} ] ; then \
 						eval ___MASTER_SITES_TMP=\$${_MASTER_SITES_$${group}} ; \
 						__MASTER_SITES_TMP="$${__MASTER_SITES_TMP} $${___MASTER_SITES_TMP}" ; \
-					fi \
+					fi; \
 				done; \
 				___MASTER_SITES_TMP= ; \
 				SORTED_MASTER_SITES_CMD_TMP="${ECHO_CMD} ${_MASTER_SITE_OVERRIDE} `${ECHO_CMD} $${__MASTER_SITES_TMP} | ${AWK} '${MASTER_SORT_AWK:S|\\|\\\\|g}'` ${_MASTER_SITE_BACKUP}" ; \
 			else \
 				SORTED_MASTER_SITES_CMD_TMP="${SORTED_MASTER_SITES_DEFAULT_CMD}" ; \
-			fi ; \
+			fi; \
 			for site in `eval $$SORTED_MASTER_SITES_CMD_TMP ${_RANDOMIZE_SITES}`; do \
 				if [ ! -z "`${ECHO_CMD} ${NOFETCHFILES} | ${GREP} -w $${file}`" ]; then \
 					if [ -z "`${ECHO_CMD} ${MASTER_SITE_OVERRIDE} | ${GREP} -w $${site}`" ]; then \
@@ -4241,7 +4241,7 @@
 				${ECHO_CMD} -n ${SETENV} ${FETCH_ENV} ${FETCH_CMD} ${FETCH_BEFORE_ARGS} $${args} "${FETCH_AFTER_ARGS}" '|| ' ; \
 			done; \
 			${ECHO_CMD} "${ECHO_CMD} $${file} not fetched" ; \
-		fi \
+		fi; \
 	done)
 .if defined(PATCHFILES)
 	@(cd ${_DISTDIR}; \
@@ -4256,13 +4256,13 @@
 					if [ ! -z \$${_PATCH_SITES_$${group}} ] ; then \
 						eval ___PATCH_SITES_TMP=\$${_PATCH_SITES_$${group}} ; \
 						__PATCH_SITES_TMP="$${__PATCH_SITES_TMP} $${___PATCH_SITES_TMP}" ; \
-					fi \
+				fi; \
 				done; \
 				___PATCH_SITES_TMP= ; \
 				SORTED_PATCH_SITES_CMD_TMP="${ECHO_CMD} ${_MASTER_SITE_OVERRIDE} `${ECHO_CMD} $${__PATCH_SITES_TMP} | ${AWK} '${MASTER_SORT_AWK:S|\\|\\\\|g}'` ${_MASTER_SITE_BACKUP}" ; \
 			else \
 				SORTED_PATCH_SITES_CMD_TMP="${SORTED_PATCH_SITES_DEFAULT_CMD}" ; \
-			fi ; \
+			fi; \
 			for site in `eval $$SORTED_PATCH_SITES_CMD_TMP ${_RANDOMIZE_SITES}`; do \
 				CKSIZE=`alg=SIZE; ${DISTINFO_DATA}`; \
 				case $${file} in \
@@ -4272,7 +4272,7 @@
 				${ECHO_CMD} -n ${SETENV} ${FETCH_ENV} ${FETCH_CMD} ${FETCH_BEFORE_ARGS} $${args} "${FETCH_AFTER_ARGS}" '|| ' ; \
 			done; \
 			${ECHO_CMD} "${ECHO_CMD} $${file} not fetched" ; \
-		fi \
+		fi; \
 	 done)
 .endif
 .endif
@@ -4442,7 +4442,7 @@
 				      OK="true"; \
 				  fi; \
 			fi; \
-		fi ; \
+		fi; \
 		\
 		if [ "$$OK" != "true" -a ${FETCH_REGET} -eq 0 ]; then \
 			${ECHO_MSG} "===>  Giving up on fetching files: $$refetchlist"; \
@@ -4453,7 +4453,7 @@
 		fi; \
 		if [ "$$OK" != "true" ]; then \
 			exit 1; \
-		fi \
+		fi; \
 	elif [ -n "${_CKSUMFILES:M*}" ]; then \
 		${ECHO_MSG} "=> No checksum file (${MD5_FILE})."; \
 	fi
@@ -4580,7 +4580,7 @@
 					fi; \
 				fi; \
 			elif ${WHICH} "$$prog" > /dev/null 2>&1 ; then \
-					${ECHO_MSG} "===>   ${PKGNAME} depends on executable: $$prog - found"; \
+				${ECHO_MSG} "===>   ${PKGNAME} depends on executable: $$prog - found"; \
 				if [ ${_DEPEND_ALWAYS} = 1 ]; then \
 					${ECHO_MSG} "       (but building it anyway)"; \
 					notfound=1; \
@@ -4592,6 +4592,7 @@
 				notfound=1; \
 			fi; \
 		fi; \
+### FIXME: somehow it breaks after the two fi statements?
 		if [ $$notfound != 0 ]; then \
 			${ECHO_MSG} "===>    Verifying $$target for $$prog in $$dir"; \
 			if [ ! -d "$$dir" ]; then \
@@ -4854,7 +4855,7 @@
 				shift 3; \
 			done; \
 			checked="$$dir $$childdir $$checked"; \
-		else \\
+		else \
 			${ECHO_MSG} "${PKGNAME}: \"$$dir\" non-existent -- dependency list incomplete" >&2; \
 		fi; \
 	done
@@ -4878,7 +4879,7 @@
 		installed=$$(${PKG_INFO} -qO $${THISORIGIN}); \
 		if [ -z "$$installed" ]; then \
 			${ECHO_CMD} $$THISORIGIN; \
-		fi \
+		fi; \
 	done
 
 ################################################################
@@ -5007,7 +5008,7 @@
 		${ECHO_MSG} "${.TARGET} requires an INDEX file (${INDEXFILE}). Please run make index or make fetchindex."; \
 	else \
 		target=${.TARGET:C/pretty-print-(.*)-depends-list/\1/} ; \
-		if [ "$$target" = "build" ] ; then fldnum=8 ; else fldnum=9 ; fi ; \
+		if [ "$$target" = "build" ] ; then fldnum=8 ; else fldnum=9 ; fi; \
 		${ECHO_MSG} -n 'This port requires package(s) "' ; \
 		${ECHO_MSG} -n `${AWK} -F\| "\\$$1 ~ /^${PKGNAME}/ {print \\$$$${fldnum};}" ${INDEXDIR}/${INDEXFILE}` ; \
 		${ECHO_MSG} "\" to $$target."; \


More information about the p4-projects mailing list