From 4e547cad02a41d020895eda83088865fecef069a Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Wed, 9 Nov 2022 07:42:38 +0200
Subject: Make process exit diagnostics in executable metadata extraction
 consistent

---
 libbuild2/file.cxx | 41 ++++++++++++++++++++++++++++++-----------
 1 file changed, 30 insertions(+), 11 deletions(-)

(limited to 'libbuild2/file.cxx')

diff --git a/libbuild2/file.cxx b/libbuild2/file.cxx
index a3039a2..8666520 100644
--- a/libbuild2/file.cxx
+++ b/libbuild2/file.cxx
@@ -1862,13 +1862,13 @@ namespace build2
     try
     {
       // Note: not using run_*() functions since need to be able to suppress
-      // all errors, including inability to exec.
+      // all errors, including abnormal, inability to exec, etc., in case of
+      // optional import. Also, no need to buffer diagnostics since in the
+      // serial load.
       //
       if (verb >= 3)
         print_process (args);
 
-      // @@ DBUF: diag
-
       process pr (pp,
                   args,
                   -2           /* stdin  to /dev/null                 */,
@@ -1924,10 +1924,19 @@ namespace build2
             return r;
 
           if (!opt)
-            error (loc) << "invalid metadata signature in " << args[0]
-                        << " output" <<
+          {
+            diag_record dr;
+            dr << error (loc) << "invalid metadata signature in " << args[0]
+               << " output" <<
               info << "expected '" << s << "'";
 
+            if (verb >= 1 && verb <= 2)
+            {
+              dr << info << "command line: ";
+              print_process (dr, args);
+            }
+          }
+
           goto fail;
         }
 
@@ -1943,16 +1952,27 @@ namespace build2
       if (pr.wait ())
       {
         if (!opt)
-          error (loc) << "unable to read metadata from " << args[0];
+          error (loc) << "io error reading metadata from " << args[0];
       }
       else
       {
         // The child process presumably issued diagnostics but if it didn't,
-        // the result will be very confusing. So let's issue something
-        // generic for good measure.
+        // the result will be very confusing. So let's issue something generic
+        // for good measure. But also make it consistent with diagnostics
+        // issued by run_finish().
         //
         if (!opt)
-          error (loc) << "unable to extract metadata from " << args[0];
+        {
+          diag_record dr;
+          dr << error (loc) << "unable to extract metadata from " << args[0] <<
+            info << "process " << args[0] << " " << *pr.exit;
+
+          if (verb >= 1 && verb <= 2)
+          {
+            dr << info << "command line: ";
+            print_process (dr, args);
+          }
+        }
       }
 
       goto fail;
@@ -1968,8 +1988,7 @@ namespace build2
       goto fail;
     }
 
-    fail:
-
+  fail:
     if (opt)
     {
       metadata_cache.insert (pp.effect_string (), true);
-- 
cgit v1.1