From 31501c88e76c44eb14cd8aba9a1ad287b3e175ba Mon Sep 17 00:00:00 2001 From: James M Snell Date: Sun, 10 Aug 2025 18:16:49 -0700 Subject: [PATCH] tools: improve error handling in node_mksnapshot --- tools/snapshot/node_mksnapshot.cc | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/tools/snapshot/node_mksnapshot.cc b/tools/snapshot/node_mksnapshot.cc index e446dd1668d61d..0842cba257d0ca 100644 --- a/tools/snapshot/node_mksnapshot.cc +++ b/tools/snapshot/node_mksnapshot.cc @@ -66,8 +66,16 @@ int BuildSnapshot(int argc, char* argv[]) { std::vector(argv, argv + argc), node::ProcessInitializationFlags::kGeneratePredictableSnapshot); + if (result->exit_code() != 0) { + for (const std::string& error : result->errors()) { + std::cerr << error << "\n"; + } + std::cerr << "node_mksnapshot failed with exit code " << result->exit_code() + << "\n"; + return static_cast(result->exit_code()); + } + CHECK(!result->early_return()); - CHECK_EQ(result->exit_code(), 0); std::string out_path; std::optional builder_script_path = std::nullopt;