ome / omero-mapr

An OMERO.web app allowing to browse the data through attributes linked to the image
https://pypi.org/project/omero-mapr/
GNU Affero General Public License v3.0
5 stars 12 forks source link

Remove setup_requires declaration from setup.py #35

Closed sbesson closed 6 years ago

sbesson commented 6 years ago

The pytest-runner is already declared in requirements-test.txt. This change should fix the installation of omero-mapr on old Python/pip (e.g. CentOS) following the PyPI TLS v1.0/1.1 deprecation

See also https://github.com/openmicroscopy/omero-marshal/pull/50

To test this PR,

$ docker run --rm -it centos:centos7
[root@985cea0418f9 /]# yum install -y python-virtualenv && virtualenv /tmp/venv
[root@985cea0418f9 /]# /tmp/venv/bin/pip install omero-mapr # should fail
[root@985cea0418f9 /]# /tmp/venv/bin/pip install https://github.com/sbesson/omero-mapr/archive/pypi_tls_1.0.zip # should work
joshmoore commented 6 years ago

:+1: for green travis. Local testing in docker as described passes.

Note: pip install -U setuptools pip also fixes the issue.

Propose to release ASAP as 0.2.2.