svn commit: r502794 - in head: graphics/py-gizeh www/py-graphite-api

[Jan Beich]

Mathieu Arnold <mat at FreeBSD.org> writes:

> On Mon, May 27, 2019 at 11:15:27PM +1000, Kubilay Kocak wrote:
>
>> On 27/05/2019 11:10 pm, Antoine Brodin wrote:
>> > Author: antoine
>> > Date: Mon May 27 13:10:06 2019
>> > New Revision: 502794
>> > URL: https://svnweb.freebsd.org/changeset/ports/502794
>> > 
>> > Log:
>> >    Unbreak bulk -a
>> >    Pointy hat:	koobs
>> > 
>> > Modified:
>> >    head/graphics/py-gizeh/Makefile
>> >    head/www/py-graphite-api/Makefile
>> > 
>> > Modified: head/graphics/py-gizeh/Makefile
>> > ==============================================================================
>> > --- head/graphics/py-gizeh/Makefile	Mon May 27 13:02:05 2019	(r502793)
>> > +++ head/graphics/py-gizeh/Makefile	Mon May 27 13:10:06 2019	(r502794)
>> > @@ -16,7 +16,7 @@ LICENSE_FILE=	${WRKSRC}/LICENCE.txt
>> >   RUN_DEPENDS=	${PYTHON_PKGNAMEPREFIX}cairocffi>0:graphics/py-cairocffi@${PY_FLAVOR} \
>> >   		${PYNUMPY}
>> > -USES=		python
>> > +USES=		python:3.5+
>> >   USE_PYTHON=	autoplist distutils
>> >   NO_ARCH=	yes
>> > 
>> > Modified: head/www/py-graphite-api/Makefile
>> > ==============================================================================
>> > --- head/www/py-graphite-api/Makefile	Mon May 27 13:02:05 2019	(r502793)
>> > +++ head/www/py-graphite-api/Makefile	Mon May 27 13:10:06 2019	(r502794)
>> > @@ -25,7 +25,7 @@ RUN_DEPENDS=	\
>> >   		${PYTHON_PKGNAMEPREFIX}gunicorn>=19.4.1:www/py-gunicorn@${PY_FLAVOR} \
>> >   		xorg-fonts-truetype>=0:x11-fonts/xorg-fonts-truetype
>> > -USES=		python
>> > +USES=		python:3.5+
>> >   USE_PYTHON=	distutils py3kplist
>> >   NO_ARCH=	yes
>> > 
>> 
>> Could you clarify why these don't just build their deps with the version
>> required by the dependency?
>
> As usual, the reason is that a dependency does not support all the
> flavors required.  In both case, graphics/py-cairocffi only supports
> 3.5+, so this breaks when you ask for the Python 2.7 flavor.  

Doesn't make sense. If a port fails to build then anything that depends
should be skipped, not break "bulk -a". For one, no one copies ONLY_FOR_ARCHS
to every consumer of a port.

Is this a bug in flavors implementation or poudriere?