devel/py-robotframework-pabot: Update to 0.87

[Vladimir.Chukharev_gmail.com]

Summary

Update devel/py-robotframework-pabot from 0.86 to 0.87, see PR 240481

Add tests. Upstream neither put nor plan to put the tests to the distributive package, and I had to checkout them using svn (or git). I've chosen svn.
I asked upstream if an additional package with tests can be made available. Now awaiting an answer.

Test Plan

portlint -Ca : OK
port test : OK
poudriere : OK (120Ramd64 py36)
make test : OK (also in a poudriere interactive jail)

[kai]

devel/py-robotframework-pabot/Makefile

7 ↗

(On Diff #61918)

This line needs to removed because we switch to GitHub further down.

20 ↗

(On Diff #61918)

Should be removed, because the tests might work without it.

21 ↗

(On Diff #61918)

We don't need subversion to checkout upstream's repository. We simply switch from PyPi (= CHEESESHOP) to GitHub for a while until upstream decides to distribute the test suite in the sdist for PyPi.

25 ↗

(On Diff #61918)

By adding

USE_GITHUB=yes
GH_ACCOUNT=mkorpela
GH_PROJECT=pabot

you can use upstream's tarball that contains the test suite.

35 ↗

(On Diff #61918)

Here's no need for svn, so this should be removed.

By looking at the run_tests.sh maybe the only following line will work instantly (haven't tested it, yet):

@cd ${WRKSRC} && ${PYTHON_CMD} -m nose tests/

or if the locale needs to be set:

@cd ${WRKSRC} && ${SETENV} LC_ALL=en_US.UTF-8 ${PYTHON_CMD} -m nose tests/

[kai]

Added small updates to some of my previous inline comments.

devel/py-robotframework-pabot/Makefile

20 ↗

(On Diff #61918)

I meant only devel/py-virtualenv in the previous comment from above. It wasn't so clear after a second look.

35 ↗

(On Diff #61918)

Meanwhile I was able to run the test suite successfully. Following oneliner should be used because the locale needs to be set and I forgot to mention the -v parameter to make the tests more verbose:

@cd ${WRKSRC} && ${SETENV} LC_ALL=en_US.UTF-8 ${PYTHON_CMD} -m nose -v tests/

[Vladimir.Chukharev_gmail.com]

Address comments by Kai Knoblich. Thanks, Kai!

[kai]

The update will be ready soon to land. There's only one nitpick and a suggestion - both are added as inline comments.

devel/py-robotframework-pabot/Makefile

12 ↗

(On Diff #62026)

You can add LICENSE_FILE=LICENSE.txt if you like because the license is available in the tarball from upstream's repository at GitHub. That license should be also packaged for the sdist where it's still missing at moment.

22–24 ↗

(On Diff #62026)

Technically those lines are ok and do what they should. From the perspective regarding the style there's still a nitpick: Please add at least a tab after all the occurences of =.

[Vladimir.Chukharev_gmail.com]

Accept corrections from @kai (slightly changed).

[kai]

In D21600#471800, @Vladimir.Chukharev_gmail.com wrote:

Accept corrections from @kai (slightly changed).

Thanks, the diff looks good to me and I plan to commit it tonight.