Antoine Brodin <anto...@freebsd.org>:

On Tue, Jul 8, 2014 at 6:14 PM, Marcus von Appen <m...@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

_______________________________________________
freebsd-python@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-python
To unsubscribe, send any mail to "freebsd-python-unsubscr...@freebsd.org"

Reply via email to