Snapshot 38

This is only a temporary measure until a proper fix is found.

src/MacVim/Info.plist

@@ -558,7 +558,7 @@
 		</dict>
 	</array>
 	<key>CFBundleVersion</key>
-	<string>37</string>
+	<string>38</string>
 	<key>NSMainNibFile</key>
 	<string>MainMenu</string>
 	<key>NSPrincipalClass</key>

src/MacVim/MMAppController.m

@@ -1954,6 +1954,11 @@ executeInLoginShell(NSString *path, NSArray *args)
         return -1;
     } else if (pid == 0) {
         // Child process
+
+        // We need to undo our zombie avoidance as Vim waits for and needs
+        // the exit statuses of processes it spawns.
+        signal(SIGCHLD, SIG_DFL);
+
         if (close(ds[1]) == -1) exit(255);
         if (dup2(ds[0], 0) == -1) exit(255);
 

src/MacVim/MMBackend.m

@@ -508,6 +508,14 @@ static NSString *MMSymlinkWarningString =
         }
         @catch (NSException *e) {
             NSLog(@"Exception caught when processing command queue: \"%@\"", e);
+            NSLog(@"outputQueue(len:%d)=%@", [outputQueue count]/2,
+                    outputQueue);
+            if (![connection isValid]) {
+                NSLog(@"WARNING! Connection is invalid, exit now!");
+                NSLog(@"waitForAck=%d got_int=%d isTerminating=%d",
+                        waitForAck, got_int, isTerminating);
+                mch_exit(-1);
+            }
         }
 
         [outputQueue removeAllObjects];
@@ -2390,7 +2398,7 @@ static NSString *MMSymlinkWarningString =
 
                 // Temporarily clear 'suffixes' so that the files are opened in
                 // the same order as they appear in the "filenames" array.
-                [self addInput:@":let t:mvim_oldsu=&su|set su=<CR>"];
+                [self addInput:@":let mvim_oldsu=&su|set su=<CR>"];
 
                 [self addInput:cmd];
 
@@ -2401,7 +2409,7 @@ static NSString *MMSymlinkWarningString =
                     [self addInput:@"|vert sall"];
 
                 // Restore the old value of 'suffixes'.
-                [self addInput:@"|let &su=t:mvim_oldsu|unlet t:mvim_oldsu"];
+                [self addInput:@"|let &su=mvim_oldsu|unlet mvim_oldsu"];
 
                 // Adding "|redr|f" ensures a "Hit ENTER" prompt is not shown.
                 [self addInput:@"|redr|f<CR>"];

src/MacVim/MacVim.xcodeproj/project.pbxproj

@@ -766,7 +766,7 @@
 					i386,
 				);
 				COPY_PHASE_STRIP = YES;
-				CURRENT_PROJECT_VERSION = 37;
+				CURRENT_PROJECT_VERSION = 38;
 				FRAMEWORK_SEARCH_PATHS = (
 					"$(inherited)",
 					"$(FRAMEWORK_SEARCH_PATHS_QUOTED_1)",
@@ -807,7 +807,7 @@
 			buildSettings = {
 				ARCHS = "$(NATIVE_ARCH)";
 				COPY_PHASE_STRIP = NO;
-				CURRENT_PROJECT_VERSION = 37;
+				CURRENT_PROJECT_VERSION = 38;
 				FRAMEWORK_SEARCH_PATHS = (
 					"$(inherited)",
 					"$(FRAMEWORK_SEARCH_PATHS_QUOTED_1)",
@@ -839,7 +839,7 @@
 			buildSettings = {
 				ARCHS = "$(NATIVE_ARCH)";
 				COPY_PHASE_STRIP = YES;
-				CURRENT_PROJECT_VERSION = 37;
+				CURRENT_PROJECT_VERSION = 38;
 				FRAMEWORK_SEARCH_PATHS = (
 					"$(inherited)",
 					"$(FRAMEWORK_SEARCH_PATHS_QUOTED_1)",

src/MacVim/macvim_latest.xml

@@ -40,6 +40,31 @@
     Sparkle supports updates in zip, tar, tbz, tgz, or dmg format.
     -->
 
+    <item>
+      <title>Snapshot 38 released</title>
+      <description><![CDATA[
+<h1>MacVim snapshot 38 released</h1>
+
+<p> Snapshot 37 had a problem with the "Login shell" option, so I've built a new snapshot containing a fix (by Ben Schmidt).</p>
+
+<p> Changes since snapshot 37:
+  <ul>
+    <li> Dropping multiple files on a window no longer results in an error </li>
+    <li> Fix "Login shell" problems (Ben Schmidt) </li>
+    <li> Exit Vim process if connection becomes invalid -- this should avoid the system log filling up with error messages, but please let me know when it happens (and send me the output from Console.app) </li>
+  </ul>
+</p>
+      ]]></description>
+      <pubDate>Sat, 21 Nov 2008 16:26 CET</pubDate>
+      <enclosure type="application/octet-stream"
+        url="http://newmacvim.muskokamug.org/mirror/files/MacVim-snapshot-38.tbz"
+        length="8007082"
+        sparkle:version="38"
+        sparkle:shortVersionString="7.2"
+        />
+    </item>
+
+
     <item>
       <title>Snapshot 37 released</title>
       <description><![CDATA[