mail/managesieve3: Implements an RFC-5804 Manage Sieve client

[loader]

Summary

Proposed commit log message:

[NEW PORT]: mail/managesieve3: Implements an RFC-5804 Manage Sieve client

A pure Python client implementation of "A Protocol for Remotely
Managing Sieve Scripts", as defined in RFC-5804.

WWW: https://bitbucket.org/ericvsmith/managesieve3

Reviewed_by: koobs, mat
Approved by: koobs (mentor)
Differential_Revision: D16699

Test Plan

• portlint: OK (looks fine.)
• testport: OK (poudriere: 1200075, [armv7, aarch64], 'BUILD_ALL_PYTHON_FLAVORS=1', [py27, py34, py35, py36, py37] tested)
• unittest: OK (

Python2.7: pytest: Ran 13 tests in 0.015s OK
Python3.4: pytest: Ran 13 tests in 0.017s OK
Python3.5: pytest: Ran 13 tests in 0.017s OK
Python3.6: pytest: Ran 13 tests in 0.018s OK
Python3.7: pytest: Ran 13 tests in 0.017s OK
)

[koobs]

Accepted, with two minor review items that while aren't 'strictly' necessary, are good for consistency (both for individual ports, and against other similar ports)

Don't blame me for nit picking, your reviews are almost always perfect. :)

mail/py-managesieve3/Makefile

11

Try to match setup.py (pypi) metadata exactly where possible:

This projects (short description) 'description' field is

Implements an RFC-5804 Manage Sieve client

If a projects metadata can/should be improved (especially for pkg-descr), send suggestions for improvements upstream

22

Not a major issue/requirement, but use the ${PYDISTUTILS_SETUP} variable wherever possible. It is the explicitly setuptools-wrapped version of 'py.setup' that the framework implements.

[loader]

Update COMMENT and replace ${PYSETUP} with ${PYDISTUTILS_SETUP}

[loader]

Thanks so much for your help @koobs. :-)

[mat]

mail/py-managesieve3/Makefile

16

Wrong place in the Makefile. See Chapter 15. Order of Variables in Port Makefiles.

[koobs]

Accepted with port ordering change (it should go below USES)

[loader]

Move NO_ARCH to the section after USE and USE_x

[loader]

mail/py-managesieve3/Makefile

16

Thanks mat@.

I just submitted a patch for portlint:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=230594

[loader]

In D16699#354871, @koobs wrote:

Accepted with port ordering change (it should go below USES)

Thanks @koobs.