Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 20 Jul 2014 23:18:58 +0200
From:      Marcus von Appen <mva@freebsd.org>
To:        Antoine Brodin <antoine@freebsd.org>, python@freebsd.org
Cc:        "svn-ports-head@freebsd.org" <svn-ports-head@freebsd.org>, "svn-ports-all@freebsd.org" <svn-ports-all@freebsd.org>, "ports-committers@freebsd.org" <ports-committers@freebsd.org>
Subject:   Re: svn commit: r361263 - in head: . Mk
Message-ID:  <20140720231858.Horde.hH8gtABZabgnNKjWAXT_Yw7@webmail.df.eu>
In-Reply-To: <CAALwa8mHv2dst-AyguonVdk%2BUEzmO9NT4UBx5LioGcy0KHGUew@mail.gmail.com>
References:  <201407081614.s68GEXP0055616@svn.freebsd.org> <CAALwa8mHv2dst-AyguonVdk%2BUEzmO9NT4UBx5LioGcy0KHGUew@mail.gmail.com>

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

Antoine Brodin <antoine@freebsd.org>:

> On Tue, Jul 8, 2014 at 6:14 PM, Marcus von Appen <mva@freebsd.org> wrote:
>> Author: mva
>> Date: Tue Jul  8 16:14:33 2014
>> New Revision: 361263
>> URL: http://svnweb.freebsd.org/changeset/ports/361263
>> QAT: https://qat.redports.org/buildarchive/r361263/
>>
>> Log:
>>   Support for installations based on the easy_install setup.py target has
>>   been removed from the Ports framework for Python software. The
>>   PYEASYINSTALL_* knobs and support for USE_PYDISTUTILS=easy_install have
>>   been removed.
>>
>>   Phabric:      D299
>>   With hat:     python@
>
> Hi,
>
> There is a regression after this commit,  more than 200 python modules
> that use USE_PYDISTUTILS=yes but don't use autoplist orphan their
> egginfo.

Ouch! I thought that I tested everything often enough...

>> @@ -574,10 +501,8 @@ PYDISTUTILS_EGGINFODIR?=${STAGEDIR}${PYT
>>  add-plist-egginfo:
>>  .if !defined(PYDISTUTILS_NOEGGINFO) && \
>>         !defined(PYDISTUTILS_AUTOPLIST) && \
>> -       (defined(INSTALLS_EGGINFO) ||   \
>> -               (defined(USE_PYDISTUTILS) && \
>> -                ${USE_PYDISTUTILS} != "easy_install")) && \
>> -        defined(PYTHON_REL)
>> +       defined(INSTALLS_EGGINFO) && \
>> +       defined(PYTHON_REL)
>>  . for egginfo in ${PYDISTUTILS_EGGINFO}
>>         if [ -d "${PYDISTUTILS_EGGINFODIR}/${egginfo}" ]; then \
>>                 ${LS} ${PYDISTUTILS_EGGINFODIR}/${egginfo} | while  
>> read f; do \

The mistake seems to be here. This should be

.if !defined(PYDISTUTILS_NOEGGINFO) && \
     !defined(PYDISTUTILS_AUTOPLIST) && \
     (defined(INSTALLS_EGGINFO) || defined(USE_PYDISTUTILS)) && \
     defined(PYTHON_REL)
...
.endif

I won't have the time to check and commit this in the next few days.
Can a python@'eer recheck and commit a fix please?

Thanks
Marcus




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