Turns out the number of times you need to resume the process for /bin/sh depends on the

setting of the environment variable COMMAND_MODE.  Changed the Platform::GetResumeCountForShell
to Platform::GetResumeCountForLaunchInfo, and check both the shell and in the case of
/bin/sh the environment as well.

llvm-svn: 190538

lldb/include/lldb/Target/Platform.h

@@ -728,7 +728,7 @@ namespace lldb_private {
                       uint64_t &high);
         
         virtual int32_t
-        GetResumeCountForShell (const char *shell)
+        GetResumeCountForLaunchInfo (ProcessLaunchInfo &launch_info)
         {
             return 1;
         }

lldb/source/Plugins/Platform/Linux/PlatformLinux.cpp

@@ -415,7 +415,7 @@ PlatformLinux::LaunchProcess (ProcessLaunchInfo &launch_info)
             const bool is_localhost = true;
             const bool will_debug = launch_info.GetFlags().Test(eLaunchFlagDebug);
             const bool first_arg_is_full_shell_command = false;
-            uint32_t num_resumes = GetResumeCountForShell (launch_info.GetShell());
+            uint32_t num_resumes = GetResumeCountForLaunchInfo (launch_info);
             if (!launch_info.ConvertArgumentsForLaunchingInShell (error,
                                                                   is_localhost,
                                                                   will_debug,

lldb/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp

@@ -1226,8 +1226,12 @@ PlatformDarwin::GetEnvironment (StringList &env)
 }
 
 int32_t
-PlatformDarwin::GetResumeCountForShell (const char *shell)
+PlatformDarwin::GetResumeCountForLaunchInfo (ProcessLaunchInfo &launch_info)
 {
+    const char *shell = launch_info.GetShell();
+    if (shell == NULL)
+        return 1;
+        
     const char *shell_name = strrchr (shell, '/');
     if (shell_name == NULL)
         shell_name = shell;
@@ -1237,7 +1241,18 @@ PlatformDarwin::GetResumeCountForShell (const char *shell)
     if (strcmp (shell_name, "sh") == 0)
     {
         // /bin/sh re-exec's itself as /bin/bash requiring another resume.
-        return 2;
+        // But it only does this if the COMMAND_MODE environment variable
+        // is set to "legacy".
+        char * const *envp = (char * const*)launch_info.GetEnvironmentEntries().GetConstArgumentVector();
+        if (envp != NULL)
+        {
+            for (int i = 0; envp[i] != NULL; i++)
+            {
+                if (strcmp (envp[i], "COMMAND_MODE=legacy" ) == 0)
+                    return 2;
+            }
+        }
+        return 1;
     }
     else if (strcmp (shell_name, "csh") == 0
             || strcmp (shell_name, "tcsh") == 0

lldb/source/Plugins/Platform/MacOSX/PlatformDarwin.h

@@ -118,7 +118,7 @@ public:
     x86GetSupportedArchitectureAtIndex (uint32_t idx, lldb_private::ArchSpec &arch);
     
     virtual int32_t
-    GetResumeCountForShell (const char *shell);
+    GetResumeCountForLaunchInfo (lldb_private::ProcessLaunchInfo &launch_info);
 
 protected:
     virtual lldb_private::Error

lldb/source/Target/Platform.cpp

@@ -664,7 +664,7 @@ Platform::LaunchProcess (ProcessLaunchInfo &launch_info)
             const bool is_localhost = true;
             const bool will_debug = launch_info.GetFlags().Test(eLaunchFlagDebug);
             const bool first_arg_is_full_shell_command = false;
-            uint32_t num_resumes = GetResumeCountForShell (launch_info.GetShell());
+            uint32_t num_resumes = GetResumeCountForLaunchInfo (launch_info);
             if (!launch_info.ConvertArgumentsForLaunchingInShell (error,
                                                                   is_localhost,
                                                                   will_debug,