if zip is missing, test-setup.sh summarily deletes undeclared outputs.

[rpoyner-tri]

Description of the bug:

If zip is missing, test-setup.sh summarily deletes undeclared outputs. See also #13230.

Since commit 8e639df , test-setup.sh will delete all undeclared outputs when zip is unavailable.

Which category does this issue belong to?

Core

What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

On ubuntu, just apt remove zip, and run any test that emits undeclared outputs. After the test completes, the outputs will be gone.

Which operating system are you running Bazel on?

Ubuntu 22 and 24.

What is the output of bazel info release?

release 7.2.1

If bazel info release returns development version or (@non-git), tell us how you built Bazel.

No response

What's the output of git remote get-url origin; git rev-parse HEAD ?

No response

If this is a regression, please try to identify the Bazel commit where the bug was introduced with bazelisk --bisect.

8e639df

Have you found anything relevant by searching the web?

No response

Any other information, logs, or outputs that you want to share?

Not directly relevant, but I find it somewhat troubling that a long and complicated script like test-setup.sh doesn't use basic bash error checking, such as set -eu -o pipefail.

[rpoyner-tri]

Thanks for taking care of this!

[tjgq]

@bazel-io fork 7.4.0

[iancha1992]

A fix for this issue has been included in Bazel 7.4.0 RC1. Please test out the release candidate and report any issues as soon as possible.
If you're using Bazelisk, you can point to the latest RC by setting USE_BAZEL_VERSION=7.4.0rc1. Thanks!