search

ome / omero-install

OMERO installation scripts
https://www.openmicroscopy.org/omero
4 stars 24 forks source link

Remove Variable #189

Closed jburel closed 5 years ago

jburel commented 5 years ago

Remove variable no longer needed This was required for Python 2.7 support on CentOS 6

joshmoore commented 5 years ago

Will a similar variable be needed for PY3? (Not that it needs adding now; I'm just wondering if this will need adding back in soon)

jburel commented 5 years ago

Maybe so I will revisit the cleaning and if it is not required when we work on PY3 I can do a full cleaning

jburel commented 5 years ago

Thanks @mtbc It should be fixed now

mtbc commented 5 years ago

:+1: I don't see any other problems.

Outwith this PR, in the longer term we might want more elses that alert of unset/unexpected values.

jburel commented 5 years ago

This is mainly b/c that code is not really used. I might get rid of it since we install the apps differently now