Looks good to me, but would it be cleaner to just add the two lines below to the list in the [IMPORTANT] section above rather than having two consecutive colour blocks?

Thanks.

In D40014#912676, @bz wrote:

Can you "Approve" this change as well given I am not a ports committer?

One minor nit. portclippy/portfmt have a few suggestions.

I initially thought an entry in MOVED would be appropriate, but that's probably not the case, since net/wifi-firmware-base will never be installed.

Thanks for working on this and sorry for the late feedback.

(With a few minor tweaks discussed outside of the review.)

Return CFLAGS="${CFLAGS}" to MAKE_ARGS

Add entry to x11-clocks/Makfile and remove commented line from pkg-descr

As we discussed privately, we don't usually strip static libraries, so either not stripping or --strip-uneeeded would be fine.

Adding PowerPC here to determine whether USES=compiler:c++11-lang is still required for building.

Address comments from grahamperrin and salvadore

No problem. I'll wait for the framework fix and ignore poudriere testport failures for now.

We now have an rc script that generates /var/db/machine-id and an /etc/machine-id symlink is created pointing to /var/db/machine-id.

A more general fix was made.

Err.. no. There shouldn't be a period at the end fo the COMMENT.

Add missing period at the end of the COMMENT.

Sergio, thanks again for the quick review. As I said in our private conversation, I requested a quick review because we would like to have this page updated before the office hours tomorrow. If there are tweaks to be made, I can make them starting Thursday.

The other two executables are also wrappers.

Add a comment that refers to the PR.