good call, fixed


On Tue, Oct 8, 2013 at 10:05 AM, Jai Ram Rideout
<notifications@github.com>wrote:

> In setup.py:
>
> > +    License :: OSI Approved :: BSD License
> > +    Topic :: Software Development :: Libraries :: Application Frameworks
> > +    Topic :: Software Development :: User Interfaces
> > +    Programming Language :: Python
> > +    Programming Language :: Python :: 2.7
> > +    Programming Language :: Python :: Implementation :: CPython
> > +    Operating System :: OS Independent
> > +    Operating System :: POSIX
> > +    Operating System :: MacOS :: MacOS X
> > +"""
> > +classifiers = [s.strip() for s in classes.split('\n') if s]
> > +
> > +# Verify Python version
> > +ver = '.'.join(map(str, [sys.version_info.major, sys.version_info.minor]))
> > +if ver not in ['2.7']:
> > +    raise ValueError, "Python %s is not supported" % ver
>
> Instead of raising an error here, do you think it'd be better to print the
> message to stderr and then sys.exit(1)? That way users will get a nice
> error message instead of a traceback.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bipy/pyqi/pull/197/files#r6830099>
> .
>