From 442cf4e38a4661f77b413fc2a90646515d2bee2f Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Thu, 12 Dec 2024 10:59:37 +0200
Subject: Fix incorrect logic based on dry_run_option value

---
 libbuild2/build/script/parser.cxx | 4 ++++
 1 file changed, 4 insertions(+)

(limited to 'libbuild2/build/script')

diff --git a/libbuild2/build/script/parser.cxx b/libbuild2/build/script/parser.cxx
index 3ecf23d..c9193ff 100644
--- a/libbuild2/build/script/parser.cxx
+++ b/libbuild2/build/script/parser.cxx
@@ -2501,6 +2501,10 @@ namespace build2
           //
           auto fail = [this, what, &ctx] (const auto& f) -> optional<bool>
           {
+            // Note that this test will give a false negative if this target
+            // ends up being updated during load or match. At least it's
+            // conservative.
+            //
             bool df (!ctx.match_only && !ctx.dry_run_option);
 
             diag_record dr;
-- 
cgit v1.1