I'd also suggest to bump PORTREVISION to also reflect the future binary packages (this might not be necessary if you plan to do a quick update).

Adding cy@ as he might also want have a look (as it's src component).

LGTM

LGTM.

Although this change looks good to me, we also should consider making -d default to $(-b)/var/db/freebsd-update. *THAT* left me with broken boot environment :-/ (I've used -b /path/to/be without -d /path/to/be/var/db/freebsd-update).

LGTM as well.

It was some problem on my end (I had globally disabled X11 option).

Ship it.

In D42719#978596, @michaelo wrote:

In D42719#978591, @otis wrote:

Does this (set -e) have any consequences on existing flow?

I'd say yes. I will fail immediately (fail-fast) when a command exits with non-zero. That's the point. Better be fail than be sorry. My initial problem was a bad NIS setup which I cannot fix at the moment, but the currect create script did not fail, thus left me with a halfbroken installation. I prefer no installation instead of a broken one. I think this should also be on 14 and 13 since it will improve realibility and will affect only a few users.

Does this (set -e) have any consequences on existing flow?