Can we merge this?

Limit scope to HEAD^

Address additional comments.

Address reviewer comments.

In D43425#990303, @rew wrote:

In D43425#990237, @delphij wrote:

In D43425#990231, @rew wrote:

For ZFS, the argument passed to the shell script is the
pool name to import.

Ah you are right, but the name is not guaranteed to be identical to the filename. Probably zpool import -a -d /dev/$(mdconfig $0).uzip instead? (and maybe -R $1)

yea, that would work - it bumps it over the limit though, so if we want zpool import -d ... then [ -z $1 ] && exit 1 would need to be removed.

I'm fine either way, I had almost considered removing the "run uzip image as a script" functionality altogether since it has been broken so long anyways.

In D43425#990231, @rew wrote:

For ZFS, the argument passed to the shell script is the
pool name to import.

Not sure if the length of magic allows you to squeeze in something like [ -n "$1" ] && before mounts (if the user didn't specify a mountpoint, the image should work like before the change).

In D43411#989745, @cperciva wrote:

Just to confirm, this will land in stable/13 and stable/14? It looks like we have the gve driver in both.

LGTM. Do we want to extract that catmsgs out to somewhere? (Not necessarily as part of this commit)

I don't think these whole case..esac blocks are needed: modern enough zcat(1) is capable of handling compressed and uncompressed text files, so you can simply do: