From fb9f2206a3a9b860480d2e9967561b47c1e86351 Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Thu, 16 Dec 2021 07:44:54 +0200
Subject: Don't consider implied existing targets in dyndep logic

While considering implied targets should be harmless, the result is racy.
---
 libbuild2/dyndep.hxx | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'libbuild2/dyndep.hxx')

diff --git a/libbuild2/dyndep.hxx b/libbuild2/dyndep.hxx
index cfd3c7e..ad95df1 100644
--- a/libbuild2/dyndep.hxx
+++ b/libbuild2/dyndep.hxx
@@ -198,7 +198,9 @@ namespace build2
                 const function<prefix_map_func>& = nullptr,
                 const srcout_map& = {});
 
-    // As above but do not insert the target if it doesn't already exist.
+    // As above but do not insert the target if it doesn't already exist. This
+    // function also returns NULL if the target exists but is implied (that
+    // is, not declared in a buildfile).
     //
     static pair<const file*, bool>
     find_file (tracer&, const char* what,
-- 
cgit v1.1