From: Dave Barach <dave@barachs.net>
Date: Mon, 4 Jan 2021 15:52:44 +0000 (-0500)
Subject: ebuild: follow symbolic links in find_newer_filtered_fn
X-Git-Tag: v21.10-rc0~750
X-Git-Url: https://gerrit.fd.io/r/gitweb?p=vpp.git;a=commitdiff_plain;h=1b576e0aa08c575a4409b50dc71e8f51d231f199

ebuild: follow symbolic links in find_newer_filtered_fn

Otherwise, changes in source code symbolically-linked into the
workspace will not be considered when deciding whether to invoke
e.g. Ninja.

This change is strongly preferable to asking people to remove
.mu_build_build_timestamp to force a Ninja run.

Deals reasonably gracefully with symbolic link loops, at least in
trivial cases: "find: File system loop detected"

Type: improvement

Signed-off-by: Dave Barach <dave@barachs.net>
Change-Id: I6666c84861de5e8b8aebcb94158f53526aded089
---

diff --git a/build-root/Makefile b/build-root/Makefile
index e5db8b8fe77..7f8d7752357 100644
--- a/build-root/Makefile
+++ b/build-root/Makefile
@@ -416,7 +416,7 @@ find_filter += -and -not -path '*/.mu_build_*'
 find_newer_filtered_fn =			\
   (! -f $(1)					\
     || -n $(call find_newer_files_fn,$(1),$(3))	\
-    || -n "`find -H $(2)			\
+    || -n "`find -L $(2) 			\
 	      -type f				\
               -and -newer $(1)			\
 	      -and \( $(4) \)			\