Most Alpine mirrors are in the format `alpine/vXX.YY`, but not all. I
hit this situation when upgrading to 23.12, the Alpine mirror I am using
is https://mirror.math.princeton.edu/pub/alpinelinux. After this script
ran it was still using the 3.18 repo when it should have been using
3.19. A quick survey of mirrors.alpinelinux.org shows that a few mirrors
are like this, and there are even some mirrors that are hosted on
subdomains with no path (e.g. http://alpine.mirror.far.fi)

Matching on the version will fix this for any mirror with a
"non-standard" repo path. It's also not great being less specific when
matching, but it seems reasonable to assume if there's a match for an
Alpine version on a line in /etc/apk/repositories then it's probably an
Alpine mirror and not a pmOS one.

Fix issue 11

As I just tried an upgrade, another source of error was having mrtest
packages installed. Make sure the user uninstalls them before attempting
an upgrade.

I just had an upgrade fail with a rather small boot partition size of
128 MiB. We changed the default to 256 MiB two years ago:
https://gitlab.com/postmarketOS/pmbootstrap/-/commit/03e9fb05709432407f1a9c4c646e742c592a4970

Print a warning if the boot partition size is smaller than expected.

Piraty authored 1 year ago and Oliver Smith committed 1 year ago

if coming from edge, this might pull packages with equal/greater
pkgver/pkgrel than what the release branch provides

Not yet implemented: install / remove packages as needed for the
upgrade, see issue 9.

According to GitLab we use 1.6 GiB for artifacts, don't keep them for
more than 1 week from now on. The old ones also need to be deleted via
the API.

Use -q with the last pmbootstrap command to avoid printing the "chroot
still active" and "done" lines twice. Remove the empty line at the end
of the file while at it.

(009828) [23:19:38] NOTE: chroot is still active (use 'pmbootstrap shutdown' as necessary)
(009828) [23:19:38] DONE!
(009088) [23:19:38] NOTE: chroot is still active (use 'pmbootstrap shutdown' as necessary)
(009088) [23:19:38] DONE!

When running "pmbootstrap ci upgrade-compat", use none as UI by default
as it is the fastest. The user can override it by setting the UI
environment variable.

Don't do the following again as it is already done by
install_pmbootstrap.sh (currently called from .gitlab-ci.yml):
* clone pmaports
* create pmbootstrap symlink
* run 'pmbootstrap init'

Just switch the git branch once after "pmbootstrap init" ran. Remove
related code that ensures and verifies that "pmbootstrap init" did not
switch the branch unexpectedly as it is not needed when we do it in this
order.

Move the PMAPORTS variable to the WORK variable and add an empty line to
separate these variables from the ones that the user can override.

Related: https://gitlab.com/postmarketOS/ci-common/-/blob/master/install_pmbootstrap.sh

Ignore the error for "In POSIX sh, local is undefined". Like in Alpine's
source code for abuild, we rather use local here and avoid related
mistakes even if it is not POSIX. The busybox sh supports it.

Add the ignore to .ci/shellcheck.sh and remove individual ignore lines
from upgrade.sh.

Related: https://www.shellcheck.net/wiki/SC3043

Recent versions of postmarketOS configure apk to ask for confirmation
(empty /etc/apk/interactive file). Make sure we always run it
non-interactively in that script for better usability, we already ask
once for confirmation before doing anything.