Fixed problems with default paths and with shell-file-name.

1 files changed, 62 insertions, 5 deletions

diff --git a/src/w32.c b/src/w32.c
index 0fa5970124a..7d63c73eb18 100644
--- a/src/w32.c
+++ b/src/w32.c
@@ -65,6 +65,7 @@ along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.  */
 #undef localtime
 
 #include "lisp.h"
+#include "epaths.h"     /* for SHELL */
 
 #include <pwd.h>
 #include <grp.h>
@@ -2018,7 +2019,7 @@ init_environment (char ** argv)
       {"PRELOAD_WINSOCK", NULL},
       {"emacs_dir", "C:/emacs"},
       {"EMACSLOADPATH", NULL},
-      {"SHELL", "%emacs_dir%/bin/cmdproxy.exe"},
+      {"SHELL", "cmdproxy.exe"}, /* perhaps it is somewhere on PATH */
       {"EMACSDATA", NULL},
       {"EMACSPATH", NULL},
       {"INFOPATH", NULL},
@@ -2094,9 +2095,12 @@ init_environment (char ** argv)
         emacs_abort ();
       *p = 0;
 
-      if ((p = _mbsrchr (modname, '\\')) && xstrcasecmp (p, "\\bin") == 0)
+      if ((p = _mbsrchr (modname, '\\'))
+          /* From bin means installed Emacs, from src means uninstalled.  */
+          && (xstrcasecmp (p, "\\bin") == 0 || xstrcasecmp (p, "\\src") == 0))
         {
           char buf[SET_ENV_BUF_SIZE];
+          int within_build_tree = xstrcasecmp (p, "\\src") == 0;
 
           *p = 0;
           for (p = modname; *p; p = CharNext (p))
@@ -2104,6 +2108,15 @@ init_environment (char ** argv)
 
           _snprintf (buf, sizeof (buf)-1, "emacs_dir=%s", modname);
           _putenv (strdup (buf));
+          /* If we are running from the Posix-like build tree, define
+             SHELL to point to our own cmdproxy.  The loop below will
+             then disregard PATH_EXEC and the default value.  */
+          if (within_build_tree)
+            {
+              _snprintf (buf, sizeof (buf) - 1,
+                         "SHELL=%s/nt/cmdproxy.exe", modname);
+              _putenv (strdup (buf));
+            }
         }
       /* Handle running emacs from the build directory: src/oo-spd/i386/  */
 
@@ -2139,16 +2152,60 @@ init_environment (char ** argv)
         if (!getenv (env_vars[i].name))
           {
             int dont_free = 0;
+            char bufc[SET_ENV_BUF_SIZE];
 
             if ((lpval = w32_get_resource (env_vars[i].name, &dwType)) == NULL
                 /* Also ignore empty environment variables.  */
                 || *lpval == 0)
               {
                 xfree (lpval);
-                lpval = env_vars[i].def_value;
-                dwType = REG_EXPAND_SZ;
                 dont_free = 1;
-                if (!strcmp (env_vars[i].name, "HOME") && !appdata)
+                if (strcmp (env_vars[i].name, "SHELL") == 0)
+                  {
+                    /* Look for cmdproxy.exe in every directory in
+                       PATH_EXEC.  FIXME: This does not find cmdproxy
+                       in nt/ when we run uninstalled.  */
+                    char fname[MAX_PATH];
+                    const char *pstart = PATH_EXEC, *pend;
+
+                    do {
+                      pend = _mbschr (pstart, ';');
+                      if (!pend)
+                        pend = pstart + strlen (pstart);
+                      /* Be defensive against series of ;;; characters.  */
+                      if (pend > pstart)
+                        {
+                          strncpy (fname, pstart, pend - pstart);
+                          fname[pend - pstart] = '/';
+                          strcpy (&fname[pend - pstart + 1], "cmdproxy.exe");
+                          ExpandEnvironmentStrings ((LPSTR) fname, bufc,
+                                                    sizeof (bufc));
+                          if (check_existing (bufc))
+                            {
+                              lpval = bufc;
+                              dwType = REG_SZ;
+                              break;
+                            }
+                        }
+                      if (*pend)
+                        pstart = pend + 1;
+                      else
+                        pstart = pend;
+                      if (!*pstart)
+                        {
+                          /* If not found in any directory, use the
+                             default as the last resort.  */
+                          lpval = env_vars[i].def_value;
+                          dwType = REG_EXPAND_SZ;
+                        }
+                    } while (*pstart);
+                  }
+                else
+                  {
+                    lpval = env_vars[i].def_value;
+                    dwType = REG_EXPAND_SZ;
+                  }
+                if (strcmp (env_vars[i].name, "HOME") == 0 && !appdata)
                   Vdelayed_warnings_list
                     = Fcons (listn (CONSTYPE_HEAP, 2,
                                     intern ("initialization"),