Commit 72bc6b4d authored by jcfr's avatar jcfr
Browse files

COMP: Fix install of "pyparsing" and "packaging" adding setuptools dependency

Following r27374 (ENH: Update pip from 10.0.1 to 18.0, and setuptools
from 39.2.0 to 40.2.0), pyparsing and packaging packages were removed
because they were vendorized in the new updated setuptools.

Then, they were re-introduced in r27279 (BUG: Ensure python-packaging is
available) because they were in fact directly used in script like
"" to parse version relying on "packaging.version".

This commit fixes regression introduced in r27279 by explicitly adding
setuptools as a dependency of "pyparsing" and "packaging".

It avoids error like the following:

cat /Volumes/Dashboards/Preview/Slicer-0-build/python-pyparsing-prefix/src/python-pyparsing-stamp/python-pyparsing-install-err.log
Traceback (most recent call last):
  File "", line 5, in <module>
    from setuptools import setup
ImportError: No module named setuptools

git-svn-id: 3bd1e089-480b-0410-8dfb-8563597acbee
parent cd74ede9
set(proj python-packaging)
# Set dependency list
set(${proj}_DEPENDENCIES python python-pyparsing)
set(${proj}_DEPENDENCIES python python-setuptools python-pyparsing)
if(NOT DEFINED Slicer_USE_SYSTEM_${proj})
set(Slicer_USE_SYSTEM_${proj} ${Slicer_USE_SYSTEM_python})
set(proj python-pyparsing)
# Set dependency list
set(${proj}_DEPENDENCIES python)
set(${proj}_DEPENDENCIES python python-setuptools)
if(NOT DEFINED Slicer_USE_SYSTEM_${proj})
set(Slicer_USE_SYSTEM_${proj} ${Slicer_USE_SYSTEM_python})
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment