Change to 1.4.15

In D28871#649595, @decke wrote:

I think we never had a legacy port for phpvirtualbox because there are not that many users and upstream is almost dead. It should be enough for now to change the dependencies to vbox-legacy. We can update it when the dust has settled.

In D28871#646753, @madpilot wrote:

I tested now virtualbox 6 modified to force clang 11 from ports on 12.2 and it works fine. I think this fix is cleaner than forcing DEBUG.

I'm almost sure this would work for 11.x too, but it's difficult to test because I'd have to build a full set of ports using SSL from ports, due to the QT/OpenSSL issue. Maybe I'll test that too, anyway I'll remove the restriction on 11 along with the one on 12.

In D28871#646424, @madpilot wrote:

In D28871#646261, @madpilot wrote:

In D28871#646258, @dereks_lifeofadishwasher.com wrote:

I was able to build, install, run, create/run VMs with this patch on 12.2-RELEASE-p3 host. Should the ignore 12 lines be removed?

If it works yes. Tomorrow I'll test myself, but you report looks promising.

I was able to build, install, run, create/run VMs with this patch on 12.2-RELEASE-p3 host. Should the ignore 12 lines be removed?

Remove copy/paste from testing

Updated patch for py-sphinxcontrib-autoprogram using USE_GITHUB

• devel/py-python-gitlab: Update to 2.6.0
• Switch to USE_GITHUB until sdist has doc/

Looks like this could use DOCS and do-test.

When setting the docs' html convert I chose lynx since it had the most overlapping depends with default mutt and built times. At the time that was www/lynx. Also assuming you're using poudriere to build so existing system checks aren't relevant there.

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.