Re: Build failure on M$ (using MSVC): Patch enclosed

[dhruva <dhruvakm@gmail.com>]

The below patch fixes the bug of infinite looping.

diff --git a/lisp/files.el b/lisp/files.el
index ee5efe4..c680d32 100644
--- a/lisp/files.el
+++ b/lisp/files.el
@@ -3120,8 +3120,9 @@ If the file is in a registered project, a cons from
 `project-directory-alist' is returned.
 Otherwise this returns nil."
   (let ((dir (file-name-directory file))
+       (root (expand-file-name "/"))
        (result nil))
-    (while (and (not (string= dir "/"))
+    (while (and (not (string= dir root))
                (not result))
       (cond
        ((setq result (assoc dir project-directory-alist))

-dk

On Tue, May 20, 2008 at 1:56 PM, dhruva <dhruvakm@gmail.com> wrote:
> Hi,
>  I figured out the problem. A call to find-file calls
> hack-project-variables which calls project-find-settings-file which
> tries to search for the file from the root (C: for me). It keeps
> looping on "C:\.dir-settings.el". I saw that in a call to stat. I just
> created an empty .dir-settings.el under the "C:" and the build is
> going on fine (with my earlier path to fix the compilation error -
> that needs to be made MSVC specific as it works with GCC/MinGW). I
> will try to find the real cause for this looping once I am done with
> the build.
>
> -dhruva
>
> On Tue, May 20, 2008 at 11:40 AM, dhruva <dhruvakm@gmail.com> wrote:
>> Hi,
>>  I tried doing a 'nmake bootstrap' and notice and infinite recursion
>> (or loop) in garbage collection. The call to 'mark_object' never comes
>> out. The following command (for emacs built with MSVC on WXP)
>> reproduces the problem consistently. I will try getting a proper back
>> trace and mail it.
>>
>> "C:\users\dhruva\stub\repo\git\emacs\lisp/../bin/emacs.exe" -batch
>> --no-init-file --no-site-file --multibyte  -l autoload  --eval "(setq
>> generate-autoload-cookie \";;;###mh-autoload\")"  --eval "(setq
>> find-file-suppress-same-file-warnings t)"  --eval "(setq
>> make-backup-files nil)"  -f w32-batch-update-autoloads
>> "C:/users/dhruva/stub/repo/git/emacs/lisp/mh-e/mh-loaddefs.el" ./mh-e
>>
>> -dhruva
>>
>> --
>> Contents reflect my personal views only!
>>
>
>
>
> --
> Contents reflect my personal views only!
>



-- 
Contents reflect my personal views only!