release: Add set -e to abort upon failure
ClosedPublic
Actions

Authored by emaste on Jun 6 2025, 2:18 AM.

Tags

None

Referenced Files

	F160786316: D50711.id157207.diff
	Sat, Jun 27, 8:41 PM

	Unknown Object (File)
	Sat, Jun 27, 4:09 AM

	Unknown Object (File)
	Thu, Jun 18, 6:16 PM

	Unknown Object (File)
	Thu, Jun 18, 4:44 PM

	Unknown Object (File)
	Thu, Jun 11, 2:46 AM

	Unknown Object (File)
	Wed, Jun 10, 10:59 PM

	Unknown Object (File)
	Fri, Jun 5, 10:01 PM

	Unknown Object (File)
	Fri, Jun 5, 8:33 PM

Subscribers

Details

Reviewers

cperciva

Group Reviewers

releng

Commits

rG611c1457ff32: release: Add set -e to abort upon failure

Summary

We don't want to blindly continue and produce broken images if something
goes wrong.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emaste created this revision.Jun 6 2025, 2:18 AM

Herald added a subscriber: jrtc27. · View Herald TranscriptJun 6 2025, 2:18 AM

emaste requested review of this revision.Jun 6 2025, 2:18 AM

This seems fine in principle, but it'll probably break some targets which rely on ignoring errors.

Please don't MFC until we've got everything fixed. But now is a good time to get this into the tree.

This revision is now accepted and ready to land.Jun 17 2025, 5:39 PM

Closed by commit rG611c1457ff32: release: Add set -e to abort upon failure (authored by emaste). · Explain WhyJun 17 2025, 5:41 PM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rG611c1457ff32: release: Add set -e to abort upon failure.

Herald added a subscriber: imp. · View Herald TranscriptJun 17 2025, 5:41 PM

Revision Contents
Changeset List

Path

Size

release/

scripts/

mk-vmimage.sh

1 line

Diff 157207

View Options

release/scripts/mk-vmimage.sh

release: Add set -e to abort upon failureClosedPublicActions