tensorlab / tensorfx

TensorFlow framework for training and serving machine learning models
Apache License 2.0
196 stars 41 forks source link

added test to build file for setuptools version #12

Closed brandondutra closed 7 years ago

brandondutra commented 7 years ago

@nikhilk

brandondutra commented 7 years ago

closing this for lack of interest: instead of doing a check, documentation will just be updated to recommend the use of some virtual environment to make sure the build tools are updated.

for historical note, the check proposed in this PR was:

# Check for setuptools >30. Otherwise, tar file will not have requirements.txt
# pip list --format=columns  (list all versions)
# grep 'setuptools' (gets setuptools x.y.z)
# awk  '{print $2}' (gets x.y.z)
# awk -F. '{print $1}' (gets x)
setuptools_version=`pip list --format=columns | grep 'setuptools' | awk  '{print $2}' | awk -F. '{print $1}'`
if [[ -z $setuptools_version || $setuptools_version -lt 30 ]]; then
  echo 'setuptools version 30 or higher is required.'
  echo "This system has version ${setuptools_version}"
  echo 'First upgrade: pip install --upgrade setuptools'
  exit 1
fi