Page MenuHomeFreeBSD

Allow upgrade from -BETA and -RC releases

Authored by on Jan 17 2019, 10:50 PM.



Update to Revision 343122 to include -BETA and -RC releases
as upgrade-able via freebsd-update

Test Plan


Diff Detail

rS FreeBSD src repository - subversion
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

671 ↗(On Diff #52952)

The case body is executed upon a match, so is this not backwards? Also typical FreeBSD style puts the cases on a separate line, so something like:

case ${RELNUM} in
*-RELEASE | *-ALPHA* | *-BETA* | *-RC*)
        # error and exit

(also adding *-ALPHA* as noted on the svn mailing list)

674 ↗(On Diff #52952)

Since we're going to add -ALPHA in here too I might drop the explicit -RELEASE form and make the text read more naturally with something along the lines of "is not a release (or alpha, beta or release candidate)." or "is not a release (including alpha, beta and release candidates)."

  • Include support for upgrading from alpha releases

Also fix reversed check for illegal releases

LGTM in principal (except the style issue raised by @emaste which is minor and I think he would take care of it when committing).

The two "NO ACTION REQUESTED FOR THIS CHANGESET" comments are issues that beyond the scope of this changeset, but something that we should do.

665 ↗(On Diff #52994)

[NO ACTION REQUESTED FOR THIS CHANGESET] We could potentially remove this compatibility shim now.

684 ↗(On Diff #52994)

[NO ACTION REQUESTED FOR THIS CHANGESET] We should consider adding a --force option to bypass this kind of checks in a separate changeset.

This revision is now accepted and ready to land.Jan 18 2019, 6:08 PM

Need to add export UNAME_r to config_SourceRelease. Will create new review with fix.

This revision was automatically updated to reflect the committed changes.