From 817c9e38c38152ddbd87cf381c1502ba5b504aa7 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Fri, 10 Jul 2026 15:29:29 +0300 Subject: packages: fail fast on empty DTail version (review follow-up) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Makefile: new check-dtail-version target guards dtail-openbsd, dtail-freebsd, and the build-VM stamp recipe — a wrong DTAIL_SRC now aborts with a clear error before any VM start, /tmp cleanup, or remote work runs with an empty version in the file names (dtail-netbsd already had an equivalent inline guard) - Makefile: dtail-openbsd pre-cleans its stale staging tgz like the netbsd target, so a failed run can never silently upload an old package - pkg-dtail-openbsd.sh / pkg-dtail-freebsd.sh: reject a missing version argument like pkg-dtail-netbsd.sh (would otherwise build dtail-.tgz) Verified: make -n renders the guard first for both targets; make dtail-{openbsd,freebsd} DTAIL_SRC=/nonexistent fails fast with the version error before any rm/remote step. Guards only change failure paths — no package rebuild needed. Co-Authored-By: Claude Fable 5 --- packages/Makefile | 17 +++++++++++++---- packages/scripts/pkg-dtail-freebsd.sh | 4 ++++ packages/scripts/pkg-dtail-openbsd.sh | 4 ++++ 3 files changed, 21 insertions(+), 4 deletions(-) diff --git a/packages/Makefile b/packages/Makefile index 1c9a01a..9dd417a 100644 --- a/packages/Makefile +++ b/packages/Makefile @@ -92,7 +92,7 @@ DTAIL_NETBSD_VERSION = $(shell echo $(DTAIL_VERSION) | tr -d '-') CONF_FRONTENDS := $(abspath $(dir $(abspath $(lastword $(MAKEFILE_LIST))))../frontends) DTAIL_RPM_FILES := $(abspath $(dir $(abspath $(lastword $(MAKEFILE_LIST))))files/dtail-rocky) -.PHONY: pkg pkg-freebsd pkg-openbsd dtail-openbsd dtail-freebsd dtail-netbsd dtail-rocky check-vars clean +.PHONY: pkg pkg-freebsd pkg-openbsd dtail-openbsd dtail-freebsd dtail-netbsd dtail-rocky check-vars check-dtail-version clean .PHONY: buildvm-start buildvm-stop buildvm-ensure # --- Build VM management --- @@ -164,9 +164,15 @@ pkg: pkg-freebsd pkg-openbsd # --- DTail (multi-binary, native build on OpenBSD VM) --- +# Fail fast when the DTail version cannot be extracted (e.g. wrong DTAIL_SRC) +# — before any VM start, remote work, or /tmp cleanup runs with an empty +# version in the file names. +check-dtail-version: + @test -n "$(DTAIL_VERSION)" || { echo "Error: could not extract DTail version from $(DTAIL_SRC)"; exit 1; } + # Sync dtail source to build VM and compile natively. # Native build handles CGo dependencies (e.g. DataDog/zstd). -/tmp/dtail-binaries/.built: buildvm-ensure +/tmp/dtail-binaries/.built: check-dtail-version buildvm-ensure @echo "Building DTail $(DTAIL_VERSION) natively on OpenBSD build VM..." @mkdir -p /tmp/dtail-binaries @# Sync source to VM via git archive (excludes build artifacts, saves space) @@ -190,8 +196,11 @@ pkg: pkg-freebsd pkg-openbsd # Package, sign, and upload the OpenBSD dtail package. # Binaries are built on the local build VM; packaging and signing happen on fishfinger. -dtail-openbsd: /tmp/dtail-binaries/.built +dtail-openbsd: check-dtail-version /tmp/dtail-binaries/.built @echo "Packaging dtail $(DTAIL_VERSION) for OpenBSD..." + @# Remove stale staging artifacts from previous runs so a failed run can + @# never silently upload an old package. + rm -f /tmp/dtail-openbsd-$(DTAIL_VERSION).tgz $(OPENBSD_SCP) -r /tmp/dtail-binaries $(OPENBSD_HOST):/tmp/dtail-binaries $(OPENBSD_SCP) $(SCRIPTS)/pkg-dtail-openbsd.sh $(OPENBSD_HOST):/tmp/pkg-dtail-openbsd.sh $(OPENBSD_SSH) $(OPENBSD_HOST) "/bin/sh /tmp/pkg-dtail-openbsd.sh '$(DTAIL_VERSION)'" @@ -209,7 +218,7 @@ dtail-openbsd: /tmp/dtail-binaries/.built # Cross-compile dtail for FreeBSD/amd64 (CGO_ENABLED=0, nozstd), package on f0, upload to PV. # .zst log files are not supported in this binary; gzip still works. -dtail-freebsd: +dtail-freebsd: check-dtail-version @echo "Building DTail $(DTAIL_VERSION) for FreeBSD/amd64..." rm -rf /tmp/dtail-freebsd-binaries mkdir -p /tmp/dtail-freebsd-binaries diff --git a/packages/scripts/pkg-dtail-freebsd.sh b/packages/scripts/pkg-dtail-freebsd.sh index 1d42aaf..b17c3b6 100644 --- a/packages/scripts/pkg-dtail-freebsd.sh +++ b/packages/scripts/pkg-dtail-freebsd.sh @@ -9,6 +9,10 @@ set -e VERSION="$1" +if [ -z "$VERSION" ]; then + echo "Error: version argument missing (would build dtail-.tgz)" >&2 + exit 1 +fi PV_DEST="$2" NAME="dtail" COMMENT="Distributed log tail and grep tool" diff --git a/packages/scripts/pkg-dtail-openbsd.sh b/packages/scripts/pkg-dtail-openbsd.sh index 1206927..d193787 100755 --- a/packages/scripts/pkg-dtail-openbsd.sh +++ b/packages/scripts/pkg-dtail-openbsd.sh @@ -9,6 +9,10 @@ set -e VERSION="$1" +if [ -z "$VERSION" ]; then + echo "Error: version argument missing (would build dtail-.tgz)" >&2 + exit 1 +fi NAME="dtail" COMMENT="Distributed log tail and grep tool" DESC="DTail is a distributed DevOps tool for tailing, grepping, catting, and -- cgit v1.2.3