Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 7 Jan 2012 18:23:54 -0800
From:      Garrett Cooper <yanegomi@gmail.com>
To:        Michael Scheidell <scheidell@freebsd.org>
Cc:        cvs-ports@freebsd.org, douglas@douglasthrift.net, cvs-all@freebsd.org, ports-committers@freebsd.org
Subject:   Re: cvs commit: ports/devel/py-setuptools Makefile pkg-plist ports/devel/py-setuptools/files pkg-install.in
Message-ID:  <CAGH67wS19BV=000RAgm=t=_Z0FhB%2Bgsqo2z=h2onKUHB9DUmvA@mail.gmail.com>
In-Reply-To: <201201061636.q06Ga3BI017302@repoman.freebsd.org>
References:  <201201061636.q06Ga3BI017302@repoman.freebsd.org>

index | next in thread | previous in thread | raw e-mail

On Fri, Jan 6, 2012 at 8:36 AM, Michael Scheidell <scheidell@freebsd.org> wrote:
> scheidell    2012-01-06 16:36:03 UTC
>
>  FreeBSD ports repository
>
>  Modified files:
>    devel/py-setuptools  Makefile pkg-plist
>  Added files:
>    devel/py-setuptools/files pkg-install.in
>  Log:
>  - Fixes: easy-install.pth contents lost on upgrade clobbering ports using it
>
>  PR:             ports/118301
>  Submitted by:   Douglas William Thrift <douglas@douglasthrift.net>
>  Approved by:    lars.eggert@nokia.com (maintainer, timeout 3 years), gabor (mentor)
>
>  Revision  Changes    Path
>  1.23      +11 -1     ports/devel/py-setuptools/Makefile
>  1.1       +24 -0     ports/devel/py-setuptools/files/pkg-install.in (new)
>  1.12      +4 -2      ports/devel/py-setuptools/pkg-plist

Some other issues with the patch:

1.

+	PKG_PREFIX=${PREFIX} ${SH} ${PKGINSTALL} ${PKGNAME} POST-INSTALL

This should be:

+	env PKG_PREFIX=${PREFIX} ${SH} ${PKGINSTALL} ${PKGNAME} POST-INSTALL

2. This line in the pkg_install.in disguises bugs with mv:

+exit 0

The overall script should be something like this:

ec=0
case $2 in
     PRE-INSTALL)
		if [ -f ${EASYINSTALL_PTH} ]; then
		    mv ${EASYINSTALL_PTH} ${EASYINSTALL_PTH}.tmp
		    ec=$?
		fi
		;;
     POST-INSTALL)
		if [ -f ${EASYINSTALL_PTH}.tmp ]; then
		    mv ${EASYINSTALL_PTH}.tmp ${EASYINSTALL_PTH}
		    ec=$?
		fi
		;;
esac

exit $ec

3. This doesn't fix parallel installation issue with easy-install
(see: http://mail.python.org/pipermail/distutils-sig/2009-March/011127.html
).

Thanks!
-Garrett


help

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGH67wS19BV=000RAgm=t=_Z0FhB%2Bgsqo2z=h2onKUHB9DUmvA>