The patch looks good to me. And I hope this was clear in the the discussion in PR 239576, though that was not stated explicitly there.

Accept corrections from @kai (slightly changed).

Address comments by Kai Knoblich. Thanks, Kai!

I discussed the tests failure with developers. The source of it is that Qt in FreeBSD uses three dots instead of the unicode ellipsis symbol (supposedly, elidedText() function).
The test has been changed to admit "..." as ellipsis. This will come with the next release.
We can simply ignore the test failure for now. I plan to find the exact place where ellipsis is inserted, and to try using the unicode ellipsis symbol (but don't hold your breath).

I tested the new version on my system. There are still some minor issues.

1. From portlint:

$ portlint -Ca
WARN: Makefile: When USES=pyqt:5 is defined, you must also define USE_PYQT=xxxx
0 fatal errors and 1 warning found.

It seems to me that options can be made more user friendly. Or completely abandoned for now. May be, until pyside is added.
In particular, for py36, the only possible toolkit is QT5, and it comes off by default. Logically, it can be unconditionally used in this flavor.
For py27, there is QT5 and WXGTK. None is selected by default, and I think, at least one is compulsory. Do they both work with traitsui? Or can we choose the better of the two? (I still cannot build it for py27, some deps fail to install with their py36 flavors installed.)
The DEMO option adds some dependencies. Is this useful without demos from examples directory? I cannot get any demo actually show anything, even with the examples...

A newer version committed.

I moved 'SVNROOT?=...' to the maintainer section.

Shell I submit a PR to Bugzilla? Or will it be committed directly from here? Provided no more questions arise, of course.

I'm sure I tried to remove flavors, and failed with dependence specification. Then I had problems with same name of pkgnames for flavors.
The first was rather long time ago, and the second, not so long, few months ago. Good progress!
Would be nice to have in Poter's Handbook up-to-date recommendations for flavors for Python...

Everything has been committed.

Sorry, forgot to "svn add" the new ports after getting a new svn working copy (due to OS upgrade).

Probably, better to say it explicitly:
I do not know about any issues with any of the five ports in this review. All five are ready to commit IMO.

Update to the current state.
All license information is filled.
Corresponding PRs in bugzilla are updated, too.

Update devel/py-pyface to 5.0.0.

Have you tested with the post-install: target? It helped me in one of my ports.
I'm travelling at the moment, can't test myself for a while.

I got the following warnings in poudriere:

Change USES=python:2 to USES=python:-2.7

mat, I agree, the change is very simple, and that's why I added ports commiters group (otherwise I'd use only Python group).
On the other hand, Julio Merino has a good point in his opinion that everything is to be reviewed.

My reasoning to create a PR immediately after a review was to put the QA logs to the PR and make cross-references in both of them. I have a feeling, probably false, that QA logs suit better a PR than a review.

Thanks, mat. A PR 205126 is filed as mentioned in my previous comment. Was it wrong to submit a PR before receiving a positive review?

See bug 205126 in Bugzilla for QA tests output.

Review 10830 contains graphics/py-mayavi/files/patch-* files, forgotten in the previous review. These patches fix the problem with plist.

Most of he issues are resolved in the new revision, with two exceptions.

1. LICENSE work is made only in commented out lines. I think, without ready documentation it's too complex job for these ports. Or, a least, I need more guidance in this area.
2. The dependence from a port with certain OPTION is reverted to what I had in the first revision. I'm still not ready to work on a new (sub)port of math/vtk6. On the other hand, the method I tried in my comment to graphycs/py-mayavi/Makefile (using pkg query) gives answer about packages installed on the system, but not in a jail of poudriere.

OK, I add the review via web interface.

I do not understand why all the files are empty. Is it related to the fact, that I use svn while 'arc diff --create' uses git in its commands:
diff --git a/graphics/py-traitsui/pkg-descr b/graphics/py-traitsui/pkg-descr