From 620ac16e2628eb2ac764b9eda5b5e7df97e7e4f0 Mon Sep 17 00:00:00 2001 From: Christopher Baines Date: Tue, 26 Nov 2024 09:27:50 +0000 Subject: [PATCH] Don't crash when linters crash This is currently happening because some linters try to evaluate parts of packages for cross-building to aarch64-linux-gnu, but not all packages support that and some crash in this case. I'm not quite sure what the correct behaviour should be, but maybe the data service needs to try and handle these crashes rather than not processing the entire revision. --- guix-data-service/jobs/load-new-guix-revision.scm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/guix-data-service/jobs/load-new-guix-revision.scm b/guix-data-service/jobs/load-new-guix-revision.scm index a82afe7..9b7db0c 100644 --- a/guix-data-service/jobs/load-new-guix-revision.scm +++ b/guix-data-service/jobs/load-new-guix-revision.scm @@ -360,7 +360,8 @@ (simple-format (current-error-port) "exception checking ~A with ~A checker: ~A\n" package checker-name exn) - (raise-exception exn)) + ;; TODO Record and report this exception + '()) (lambda () (if (and lint-checker-requires-store?-defined? (lint-checker-requires-store? checker))