On Sat, Apr 27, 2019 at 12:30:52PM +0200, Martin Reindl wrote:
> > Hello ports,
> > Attached is a new port of net/py-siphon:
> > Siphon is a collection of Python utilities for downloading data from
> > remote data services. Much of Siphon's current functionality focuses
> > on access to data hosted on a THREDDS Data Server. It also provides
> > clients to a variety of simple web services.
> Resending. Anyone wants to take a look at this?
The ports tree has a wee bit of chaos going on at the moment, so this
will be an incomplete review, but I figure I'll give you the feedback
I have so far.
1) *_DEPENDS should be sorted alphabetically.
2) The setup.py indicates needing py-enum34 for versions < 3.4. So you should
include that for the python 2 flavor.
3) Is there an advantage to pulling from github rather than PyPi?
--Kurt
No comments:
Post a Comment