Do you have any way to verify mu4e? I do not use it.

I've been using 1.4.13 for a couple days now without issues.

Turn this into an optional patch with an addition to pkg-message

In D26061#579331, @uqs wrote:

Looks good, do you want me to go ahead and commit this on your behalf?

Reworded ttrssd.

Reword ttrssd instructions required for both web and manual installs

Make the base directory root:www owned 755

Missing % in pkg-message rm command

Tested this with uninstall/reinstalls confirming the permissions were what we expect.
Ran thru the web installer without errors.
Switched options to confirm the proper depends were being added.
Reviewed ttrssd was be commented properly with the new OPT_VARS= method.

I still believe having a config.php is the right thing to do here. Looking at other packaged tt-rss they all include a config.php so I think the same should still appear here.
config.php-dist is now there so opting in to the web installer is simple enough (rm config.php) and the web installer just works with writing the config.

As for (2), I think we should fix it, especially as:

• the fix is trivial, just don't rename the file, and
• the web-based setup seems more documented online and apparently is what the developers of tt-rss support, and
• that web-based setup correctly flagged the missing iconv module, so it's a good thing :) It was also easier to setup and init the database. I rarely need to use the psql or mysql CLIs so not having to deal with them is a plus :)

Is there a reason the file is being renamed in the first place? That seems a bit gratuitous. If the changed name is really required, I would suggest to copy/link instead of renaming it, so both dist or sample files are present (and updated).

• Renamed to moin-theme-hal
• Changed RUN_DEPEND to be vesion-specish

See example of wiki running here https://pasteboard.co/JewhwmC.jpg

Update MAINTAINER at a later time.
Turn pytest-running int a straight patch

Update 20.3.0 requirments. The original commit was from 19.10.0.

Looking for any additional feedback on testing py-twisted.

alphabetical order correction for devel/Makefile

Based on comments from https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=234929
the last patch disabled some functionality with libvncserver. Also added SASL
support with patch to fix a build error.

Looks like there were a couple scenarios that might cause the port to fail due to the new cmake build system and the defaults it sets.

Rebuilt net/x11vnc and emulators/virtualbox-ose.
Also confirmed 0.9.12 fixes emulators/virtualbox-ose virtualbox.org ticket.

Fixed via 467561

Fixed via 467557

Fixed via 467558

Add tests and DOCS option

Back to the original commit for www/py-httmock

Added test and DOCS options

Add tests and DOCS option

Remove concurrent. Not needed for this port

Wrong comparision for py-requests should be >=

• Convert to CHEESESHOP from USE_GITHUB
• LICENSE was incorrect should be LGPL3
• Add concurrent support
• Change COMMENT to match
• Small reword in pkg-message.in for CLI config

Fixed by https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=220461

Yes, I agree.

I believe this review can be closed because of https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=222244

vvv was updated for 1.9.0

Anyone mind commenting on removing NNTP patch?

Fixed by https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=219786 and commit https://svnweb.freebsd.org/changeset/ports/443855

• Update COMMENT removing ending .
• Add support for XML and YAML based lanuages (ctags --list-lanuages and --list-subparsers=all)
• Add support for JSON output
• Install man pages
• Move "small" plist to Makefile
• Don't run autogen.sh
• Disable test-cases.mak

Latest release is 2.2.9 not 2.2.6

Fix botched pkg-plist update

Based on https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=218899 we shouldn't be looking at GSSAPI_* for SASL GSSAPI plugins. Looking at something different.

Forgot to dump port revision number

doh. The inline "I don't know if I fully understand..." was held waiting for me to submit "Leap Into Action!" This was initially entered Sun 2/27. I think that would explain why I didn't see any reply about it.

It seems possible that pre-configure:: could be converted to a : if "CONFIGURE_SED" is moved to a _VAR helper.

• Updated to use the COMPRESSED_FOLDER_PLIST_FILES helper.
• post-patch:: can be changed to a : since there's a single post-patch creation script.

Excluded previous commit - included here.

Update to 454292b29 to include more window.opener fixes

Looks like there was another update to that fixes more window.opener issues. Going to update this commit to reflect that.

Reviewing the way Debian handled this same issue maybe it's best left to the user to handle this. https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=837761

Good point I was only thinking about vim via terminal.