devel/py-botocore: Update to 1.12.142

[koobs]

Summary

www/py-boto3 update requires botocore >= 1.12.142

devel/py-botocore: Update to 1.12.142

 - Update RUN_DEPENDS to match setup.py versions
 - Document RUN_DEPENDS max versions to ease future QA

Reviewed_by: bhughes
Approved by: bhughes (maintainer)
Differential_Revision: D20161

Test Plan

• portlint: OK (looks fine.)
• testport: OK (poudriere: 12amd64py{36,27})
• maketest: Ran 5989 tests in 785.902s OK (SKIP=7)

[bhughes]

LGTM :)

[mat]

head/devel/py-botocore/Makefile

16–19

This change is awful and should be reverted. Adding version requirements in comment is completely useless, especially as they can be expressed just fine in the dependency line:

${PYTHON_PKGNAMEPREFIX}dateutil>=2.1<3.0.0:devel/py-dateutil@${PY_FLAVOR}

[koobs]

What, multiple version specs (AND'd, not OR'd) are supported? If so, WOW.

I've recently been specifying max pinned versions to make future QA easier (grepping the tree), as it is a ridiculously laborious task to do manually, particularly for python ports with a large set of dependencies

[koobs]

In D20161#434772, @koobs wrote:

What, multiple version specs (AND'd, not OR'd) are supported? If so, WOW.

I've recently been specifying max pinned versions to make future QA easier (grepping the tree), as it is a ridiculously laborious task to do manually, particularly for python ports with a large set of dependencies

If it is supported, can we make them support comma separation?

[mat]

It has been supported ever since we switched to pkg years ago.

And sure I understand what you are saying about commas, but no, we're not changing how it works.

Under the hood, it runs pkg info, you can test for yourself, like, pkg info 'perl5<5.29>5.28'. Empiric testing, pkg info 'perl5<5.29,>5.28' says that this also works, if this is what you meant by commas.

[mat]

It only supports AND, all versions comparison you put must match.