From 852f732933d4ad7df476ecfe0cfa11abac18bc31 Mon Sep 17 00:00:00 2001
From: pat-s <patrick.schratz@gmail.com>
Date: Thu, 9 Apr 2026 22:59:46 +0200
Subject: [PATCH] fix: compute platform for all distros and add error handler
 to surface failures

platform was only set inside an alpine-specific if block, causing undefined variable errors for ubuntu/redhat builds that silently exited with code 1.
---
 local/packages-to-build.R | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/local/packages-to-build.R b/local/packages-to-build.R
index 0d1430f..4f8df44 100644
--- a/local/packages-to-build.R
+++ b/local/packages-to-build.R
@@ -1,3 +1,9 @@
+options(error = function() {
+  message(geterrmessage())
+  traceback(2)
+  q(status = 1)
+})
+
 library(bincraft, quietly = TRUE)
 suppressPackageStartupMessages(library(dplyr))
 library(DBI, quietly = TRUE)
@@ -10,10 +16,8 @@ suppressPackageStartupMessages(library(data.table))
 # Sys.setenv("ARCH" = "arm64")
 
 arch = Sys.getenv("ARCH")
-if (Sys.getenv("OS") == "alpine") {
-  # target: alpine-322
-  platform = paste(Sys.getenv("OS"), gsub("[.]", "", Sys.getenv("OS_VERSION")), sep = "-")
-}
+# target: alpine-322, ubuntu-2404, redhat-9, etc.
+platform = paste(Sys.getenv("OS"), gsub("[.]", "", Sys.getenv("OS_VERSION")), sep = "-")
 
 con <- DBI::dbConnect(
   RPostgres::Postgres(),