Summary: | [NEW PORT] net/py-IPy: Handling of IPv4 and IPv6 addresses in Python | ||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Yuri Victorovich <yuri> | ||||||||||||||||
Component: | Individual Port(s) | Assignee: | freebsd-ports-bugs (Nobody) <ports-bugs> | ||||||||||||||||
Status: | Closed FIXED | ||||||||||||||||||
Severity: | Affects Only Me | CC: | koobs, python, wen | ||||||||||||||||
Priority: | --- | Keywords: | feature, needs-patch, patch | ||||||||||||||||
Version: | Latest | ||||||||||||||||||
Hardware: | Any | ||||||||||||||||||
OS: | Any | ||||||||||||||||||
Attachments: |
|
Created attachment 158440 [details]
shar archive
Created attachment 158441 [details]
poudriere log
Created attachment 158442 [details]
shar archive
Added NOARCH
Thanks Yuri. On initial review: * Lowercase PORTNAME * Use DISTNAME instead of GH_PROJECT/GH_TAGNAME * Sort USE_PYTHON values * LICENSE appears to be BSD (3CLAUSE) as per setup.py and COPYING file * Remove trailing slash from pkg-descr: WWW url * I would probably remove "Python class and " from COMMENT By the way, our reviews system (Phabricator) is now configured to accept outside contributor accounts. I highly recommend creating an account there to obtain future reviews for your contributions > * Lowercase PORTNAME
It is branded as IPy, making it ipy will decrease its recognizability
> By the way, our reviews system (Phabricator) is now configured to accept outside contributor accounts. I highly recommend creating an account there to obtain future reviews for your contributions
It will likely make such submission much more work. Need to submit at least twice then.
> * LICENSE appears to be BSD (3CLAUSE) as per setup.py and COPYING file No, COPYING file doesn't say tis is BSD license. I asked upstream to clarify: https://github.com/autocracy/python-ipy/issues/39 Until then we should just use it as-is. Created attachment 158465 [details] Updated shar > * Use DISTNAME instead of GH_PROJECT/GH_TAGNAME DISTNAME evaluates to different thing for GH-based project. So I left this one. (In reply to yuri from comment #7) See: https://github.com/autocracy/python-ipy/blob/master/setup.py#L67 Created attachment 158479 [details]
Updated shar
Created attachment 158514 [details]
Updated shar
Hi: There is net-mgmt/py-ipy in portstree :) wen Ah, I see, thanks. Because I searched for IPy, but ity is spelled in lowercase, and PYPI also seems to be case insensitive for some reason. Wen, could you please update net-mgmt/py-ipy to the latest version please? I updated it to 0.83 and reset maintainer to ports@. Yuri, please feel free to take maintainership of it now. wen Thanks Wen! I am not particularly into this package though, I was combing through the dependencies of another, larger package that I am creating and came across py-IPy. Yuri |
Created attachment 158439 [details] shar archive