Bug 181672 - [PATCH] devel/pear: [SUMMARIZE CHANGES]
Summary: [PATCH] devel/pear: [SUMMARIZE CHANGES]
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Antoine Brodin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-08-30 05:20 UTC by Gasol Wu
Modified: 2013-11-11 09:25 UTC (History)
1 user (show)

See Also:


Attachments
pear-1.9.4_3.patch (988 bytes, patch)
2013-08-30 05:20 UTC, Gasol Wu
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gasol Wu 2013-08-30 05:20:00 UTC
take care about directory that contains a space

Refer to http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/181668

Port maintainer (miwi@FreeBSD.org) is cc'd.

Generated with FreeBSD Port Tools 0.99_8 (mode: change, diff: ports)
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2013-08-30 05:20:16 UTC
Responsible Changed
From-To: freebsd-ports-bugs->miwi

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Bryan Drewery freebsd_committer freebsd_triage 2013-09-22 15:11:35 UTC
Responsible Changed
From-To: miwi->freebsd-ports-bugs

Back to the heap due to timeout
Comment 3 Antoine Brodin freebsd_committer freebsd_triage 2013-11-10 13:59:26 UTC
Responsible Changed
From-To: freebsd-ports-bugs->antoine

Take
Comment 4 dfilter service freebsd_committer freebsd_triage 2013-11-11 09:17:58 UTC
Author: antoine
Date: Mon Nov 11 09:17:51 2013
New Revision: 333480
URL: http://svnweb.freebsd.org/changeset/ports/333480

Log:
  Correctly handle directory names with space in PEAR_AUTOINSTALL
  This is needed for pear-Symfony_Component_Finder update
  
  PR:		ports/181672
  Reported by:	Gasol Wu

Modified:
  head/devel/pear/bsd.pear.mk

Modified: head/devel/pear/bsd.pear.mk
==============================================================================
--- head/devel/pear/bsd.pear.mk	Mon Nov 11 09:07:15 2013	(r333479)
+++ head/devel/pear/bsd.pear.mk	Mon Nov 11 09:17:51 2013	(r333480)
@@ -190,24 +190,24 @@ do-autogenerate-plist: patch
 	${ECHO_CMD} "Cannot generate packing list: package files outside PREFIX"; \
 	exit 1; fi;
 	@${ECHO_CMD} "${LPKGREGDIR}/package.xml" > ${PLIST}
-	# pkg_install needs to escape $ in directory name while pkg does not
+# pkg_install needs to escape $ in directory name while pkg does not
 .if defined(WITH_PKGNG)
 	@cd ${WRKDIR}/inst/${PREFIX} && ${FIND} . -type f | ${SORT} \
 	| ${CUT} -c 3- >> ${PLIST}
-	@DIRS=`cd ${WRKDIR}/inst/${PREFIX} && ${FIND} . -type d | ${SORT} -r | \
-	${CUT} -c 3-`; \
-	for d in $${DIRS}; do \
-		if [ ! -d ${LOCALBASE}/$${d} ]; then \
+	@cd ${WRKDIR}/inst/${PREFIX} && ${FIND} . -type d | ${SORT} -r | \
+	${CUT} -c 3- | \
+	while read d ; do \
+		if [ ! -d "${LOCALBASE}/$${d}" ]; then \
 			${ECHO_CMD} "@dirrmtry $${d}" >> ${PLIST}; \
 		fi; \
 	done
 .else
 	@cd ${WRKDIR}/inst/${PREFIX} && ${FIND} . -type f | ${SORT} \
 	| ${CUT} -c 3- >> ${PLIST}
-	@DIRS=`cd ${WRKDIR}/inst/${PREFIX} && ${FIND} . -type d | ${SORT} -r | \
-	${CUT} -c 3- | ${SED} -e 's,\\$$,\\\\$$,g'`; \
-	for d in $${DIRS}; do \
-		if [ ! -d ${LOCALBASE}/$${d} ]; then \
+	@cd ${WRKDIR}/inst/${PREFIX} && ${FIND} . -type d | ${SORT} -r | \
+	${CUT} -c 3- | ${SED} -e 's,\$$,\\\\$$,g' | \
+	while read d ; do \
+		if [ ! -d "${LOCALBASE}/$${d}" ]; then \
 			${ECHO_CMD} "@dirrmtry $${d}" >> ${PLIST}; \
 		fi; \
 	done
_______________________________________________
svn-ports-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-ports-all
To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"
Comment 5 Antoine Brodin freebsd_committer freebsd_triage 2013-11-11 09:24:47 UTC
State Changed
From-To: open->closed

Fixed, thanks for the report!