From 6c112f9860c1730237029f4e23d40bf658d73704 Mon Sep 17 00:00:00 2001
From: Julian Andres Klode <julian.klode@canonical.com>
Date: Sun, 9 Mar 2025 13:55:23 +0100
Subject: Print --solver 3.0 explanation if both internal and it failed

This can provide useful additional context.

To avoid updating the whole test suite for it, introduce a new
option `quiet::NoSolver3Explanation` and set it by default. All
the other tests that assert output will already have matching
tests for --solver 3.0 with the correct messages asserted, it
makes no sense to duplicate them.
---
 apt-private/private-install.cc | 3 +++
 1 file changed, 3 insertions(+)

(limited to 'apt-private')

diff --git a/apt-private/private-install.cc b/apt-private/private-install.cc
index cb9f93e9e..29e23a57e 100644
--- a/apt-private/private-install.cc
+++ b/apt-private/private-install.cc
@@ -1024,6 +1024,9 @@ bool DoCacheManipulationFromCommandLine(CommandLine &CmdL, std::vector<PseudoPkg
 	   transaction.temporaryRollback(); // Rollback to before internal solver made changes so we can dump the OG request
 	   WriteApportReport(Cache, error, errors, UpgradeMode, distUpgradeMode);
 	}
+
+	if (not res && not errors.empty() && not _config->FindB("quiet::NoSolver3Explanation", false))
+	   _error->Error("The following information from --solver 3.0 may provide additional context:\n%s", APT::String::Join(errors, "\n").c_str());
      }
      // If the 3.0 solver could not recover either, abort.
      _error->MergeWithStack();
-- 
cgit v1.2.3-70-g09d2