Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 6 May 2012 19:07:11 +0200
From:      Marcus von Appen <mva@FreeBSD.org>
To:        FreeBSD-gnats-submit@freebsd.org, freebsd-ports-bugs@FreeBSD.org
Cc:        python@freebsd.org, freebsd-ports@freebsd.org
Subject:   Re: ports/167368: Python version propagation breaks USE_PYTHON= usage for dependency builds
Message-ID:  <20120506170711.GA1570@medusa.sysfault.org>
In-Reply-To: <201204271900.q3RJ0OY7074360@freefall.freebsd.org>
References:  <201204271848.q3RImjd5090552@medusa.sysfault.org> <201204271900.q3RJ0OY7074360@freefall.freebsd.org>

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

--OROCMA9jn6tkzFBc
Content-Type: multipart/mixed; boundary="PuGuTyElPB9bOcsM"
Content-Disposition: inline


--PuGuTyElPB9bOcsM
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

In 2003 we introduced DEPENDS_ARGS to enforce the python version on
dependencies[0].
In 2005 we introduced th .MAKEFLAGS propagation to work around an issue
for Zope[1].

I doubt that both are still valid after we allowed parallel python
versions for quite some while now. In any case, the current
implementation _breaks_ ports, if they have non-matching sets of
supported python versions.

The attached patch removes both entries and a local tinderbox run did
not reveal any problems. However, the problem described here does
usually _not_ occur in a tinderbox, since every port is treated as a
single item, instead of a typical user workflow.
Thus more testing (especially with zope-related ports) is highly
appreciated.

[0] http://www.freebsd.org/cgi/cvsweb.cgi/ports/Mk/bsd.python.mk?annotate=1.41
[1] http://www.freebsd.org/cgi/cvsweb.cgi/ports/Mk/bsd.python.mk?annotate=1.62

Regards
Marcus

--PuGuTyElPB9bOcsM
Content-Type: text/x-diff; charset=us-ascii
Content-Disposition: attachment; filename="bsd.python.mk.patch"
Content-Transfer-Encoding: quoted-printable

Index: Mk/bsd.python.mk
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
RCS file: /home/pcvs/ports/Mk/bsd.python.mk,v
retrieving revision 1.143
diff -u -r1.143 bsd.python.mk
--- Mk/bsd.python.mk	11 Apr 2012 12:52:16 -0000	1.143
+++ Mk/bsd.python.mk	6 May 2012 17:01:48 -0000
@@ -339,7 +339,6 @@
 PYTHONBASE!=3D		(${PYTHON_CMD} -c 'import sys; print sys.prefix' \
 						2> /dev/null || ${ECHO_CMD} ${LOCALBASE}) | ${TAIL} -1
 .endif
-DEPENDS_ARGS+=3D		PYTHON_VERSION=3D${PYTHON_VERSION}
=20
 # We can only use the cached version if we are using the default python ve=
rsion.  Otherwise it
 # should point to some other version we have installed, according to the p=
ort USE_PYTHON
@@ -354,9 +353,6 @@
 PYTHON_PORTVERSION=3D	${PYTHON_DEFAULT_PORTVERSION}
 .endif
=20
-# Propagate the chosen python version to submakes.
-.MAKEFLAGS:	PYTHON_VERSION=3Dpython${_PYTHON_VERSION}
-
 # Python-3.2
 .if ${PYTHON_VERSION} =3D=3D "python3.2"
 PYTHON_PORTVERSION?=3D3.2.3

--PuGuTyElPB9bOcsM--

--OROCMA9jn6tkzFBc
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.19 (FreeBSD)

iEYEARECAAYFAk+mr78ACgkQi68/ErJnpkfXfQCfd/oAr457Rcohe4ai6+yu4ySi
OOMAnihREOWnftpVzffjzz/a9cGAAN0R
=HINZ
-----END PGP SIGNATURE-----

--OROCMA9jn6tkzFBc--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120506170711.GA1570>