search

tahoe-lafs / zfec

zfec -- an efficient, portable erasure coding tool
Other
374 stars 44 forks source link

Remove "python_version <= 2.7" install requirement #24

Closed sajith closed 3 years ago

sajith commented 3 years ago

I'm pretty sure python_version <= '2.7'" is meaningless here, certainly at this time, because zfec builds just fine with Python 2.7 and and a bunch of Python 3+ versions. This was added in 45e5dab579, probably by mistake.

exarkun commented 2 years ago

It meant "argparse > 0.8 is a dependency only on Python 2.7 and older". This is reasonable since argparse is part of the standard library on any but the oldest Python 3 releases (like, 3.0).

With the constraint removed, argparse is a dependency even on newest Python 3, where there is a stdlib module of the same name. It probably doesn't hurt much, especially given how simple and little used the zfec command line is... but the constraint was probably correct.