From: Boris Kolpackov <boris@kolpackov.net>
Date: Fri, 29 Apr 2011 15:27:39 +0000 (+0000)
Subject: Delay caching of the file name in eval_makefile() until after all the
X-Git-Tag: moved-to-git~77
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=15a79d723de278d76737b5cbb8d7a1afbe72de7b;p=thirdparty%2Fmake.git

Delay caching of the file name in eval_makefile() until after all the
expansions and searches are complete. This fixes an assertion in a
situation where the MAKEFILES variable contains a file path that
contains tilde.
---

diff --git a/ChangeLog b/ChangeLog
index 7d2155ac..e365f6bb 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2011-04-29 Boris Kolpackov  <boris@codesynthesis.com>
+
+	* read.c (eval_makefile): Delay caching of the file name until after
+	all the expansions and searches.
+
 2011-04-17  David A. Wheeler  <dwheeler@dwheeler.com>
 
 	* doc/make.texi (Reading Makefiles): Document "!=".
diff --git a/read.c b/read.c
index c13ead87..299c2e5d 100644
--- a/read.c
+++ b/read.c
@@ -324,8 +324,7 @@ eval_makefile (const char *filename, int flags)
   char *expanded = 0;
   int makefile_errno;
 
-  filename = strcache_add (filename);
-  ebuf.floc.filenm = filename;
+  ebuf.floc.filenm = filename; /* Use the original file name.  */
   ebuf.floc.lineno = 1;
 
   if (ISDB (DB_VERBOSE))
@@ -369,13 +368,17 @@ eval_makefile (const char *filename, int flags)
                                          "/", filename);
 	  ebuf.fp = fopen (included, "r");
 	  if (ebuf.fp)
-	    {
-	      filename = strcache_add (included);
-	      break;
-	    }
+            {
+              filename = included;
+              break;
+            }
 	}
     }
 
+  /* Now we have the final name for this makefile. Enter it into
+     the cache.  */
+  filename = strcache_add (filename);
+
   /* Add FILENAME to the chain of read makefiles.  */
   deps = alloc_dep ();
   deps->next = read_makefiles;