Merge pull request #982 from ychin/option-disable-sparkle

Make Sparkle updater an optional component of MacVim

Author: ychin

src/MacVim/English.lproj/MainMenu.nib/designable.nib

@@ -14,6 +14,7 @@
 
 @class MMWindowController;
 @class MMVimController;
+@class SUUpdater;
 
 
 @interface MMAppController : NSObject <MMAppProtocol> {
@@ -30,6 +31,8 @@
     int                 numChildProcesses;
     NSMutableDictionary *inputQueues;
     int                 processingFlag;
+    
+    SUUpdater           *updater;
 
     FSEventStreamRef    fsEventStream;
 }
@@ -52,6 +55,7 @@
 - (IBAction)orderFrontPreferencePanel:(id)sender;
 - (IBAction)openWebsite:(id)sender;
 - (IBAction)showVimHelp:(id)sender;
+- (IBAction)checkForUpdates:(id)sender;
 - (IBAction)zoomAll:(id)sender;
 - (IBAction)stayInFront:(id)sender;
 - (IBAction)stayInBack:(id)sender;

src/MacVim/English.lproj/MainMenu.nib/keyedobjects.nib

@@ -43,6 +43,7 @@
 #import "MMWindowController.h"
 #import "MMTextView.h"
 #import "Miscellaneous.h"
+#import "Sparkle.framework/Headers/Sparkle.h"
 #import <unistd.h>
 #import <CoreServices/CoreServices.h>
 // Need Carbon for TIS...() functions
@@ -298,6 +299,12 @@ - (id)init
         ASLogCrit(@"Failed to register connection with name '%@'", name);
         [connection release];  connection = nil;
     }
+    
+#if !DISABLE_SPARKLE
+    // Sparkle is enabled (this is the default). Initialize it. It will
+    // automatically check for update.
+    updater = [[SUUpdater alloc] init];
+#endif
 
     return self;
 }
@@ -315,6 +322,7 @@ - (void)dealloc
     [recentFilesMenuItem release];  recentFilesMenuItem = nil;
     [defaultMainMenu release];  defaultMainMenu = nil;
     [appMenuItemTemplate release];  appMenuItemTemplate = nil;
+    [updater release];  updater = nil;
 
     [super dealloc];
 }
@@ -892,6 +900,14 @@ - (void)setMainMenu:(NSMenu *)mainMenu
     // private so this will have to be considered a bit of a hack!)
     NSMenu *appMenu = [mainMenu findApplicationMenu];
     [NSApp performSelector:@selector(setAppleMenu:) withObject:appMenu];
+    
+#if DISABLE_SPARKLE
+    // If Sparkle is disabled, we want to remove the "Check for Updates" menu
+    // item since it's no longer useful.
+    NSMenuItem *checkForUpdatesItem = [appMenu itemAtIndex:
+                                       [appMenu indexOfItemWithAction:@selector(checkForUpdates:)]];
+    checkForUpdatesItem.hidden = true;
+#endif
 
     NSMenu *servicesMenu = [mainMenu findServicesMenu];
     [NSApp setServicesMenu:servicesMenu];
@@ -1195,6 +1211,15 @@ - (IBAction)showVimHelp:(id)sender
                                     @"-c", @":h gui_mac", @"-c", @":res", nil]
                        workingDirectory:nil];
 }
+    
+- (IBAction)checkForUpdates:(id)sender
+{
+#if !DISABLE_SPARKLE
+    // Check for updates for new versions manually.
+    ASLogDebug(@"Check for software updates");
+    [updater checkForUpdates:sender];
+#endif
+}
 
 - (IBAction)zoomAll:(id)sender
 {

src/MacVim/English.lproj/Preferences.nib/designable.nib

@@ -18,9 +18,11 @@
     // General pane
     IBOutlet NSPopUpButton *layoutPopUpButton;
     IBOutlet NSButton *autoInstallUpdateButton;
+    IBOutlet NSView *sparkleUpdaterPane;
 }
 
 // General pane
+- (IBAction)showWindow:(id)sender;
 - (IBAction)openInCurrentWindowSelectionChanged:(id)sender;
 - (IBAction)checkForUpdatesChanged:(id)sender;
 

src/MacVim/English.lproj/Preferences.nib/keyedobjects.nib

@@ -43,6 +43,16 @@ static void loadSymbols()
 
 @implementation MMPreferenceController
 
+- (IBAction)showWindow:(id)sender
+{
+    [super showWindow:sender];
+    #if DISABLE_SPARKLE
+        // If Sparkle is disabled in config, we don't want to show the preference pane
+        // which could be confusing as it won't do anything.
+        [sparkleUpdaterPane setHidden:YES];
+    #endif
+}
+
 - (void)setupToolbar
 {
     loadSymbols();

src/MacVim/MMAppController.h

@@ -66,7 +66,7 @@
 		1DFE25A50C527BC4003000F7 /* PSMTabBarControl.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = 1D493DB90C52533B00AB718C /* PSMTabBarControl.framework */; };
 		52818B031C1C08CE00F59085 /* QLStephen.qlgenerator in Copy QuickLookPlugin */ = {isa = PBXBuildFile; fileRef = 52818AFF1C1C075300F59085 /* QLStephen.qlgenerator */; settings = {ATTRIBUTES = (CodeSignOnCopy, ); }; };
 		528DA66A1426D4F9003380F1 /* macvim-askpass in Copy Scripts */ = {isa = PBXBuildFile; fileRef = 528DA6691426D4EB003380F1 /* macvim-askpass */; };
-		52A364731C4A5789005757EC /* Sparkle.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = 52A364721C4A5789005757EC /* Sparkle.framework */; };
+		52A364731C4A5789005757EC /* Sparkle.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = 52A364721C4A5789005757EC /* Sparkle.framework */; settings = {ATTRIBUTES = (Weak, ); }; };
 		52A364761C4A57C1005757EC /* Sparkle.framework in Copy Frameworks */ = {isa = PBXBuildFile; fileRef = 52A364721C4A5789005757EC /* Sparkle.framework */; settings = {ATTRIBUTES = (CodeSignOnCopy, RemoveHeadersOnCopy, ); }; };
 		52B7ED9B1C4A4D6900AFFF15 /* dsa_pub.pem in Resources */ = {isa = PBXBuildFile; fileRef = 52B7ED9A1C4A4D6900AFFF15 /* dsa_pub.pem */; };
 		8D11072A0486CEB800E47090 /* MainMenu.nib in Resources */ = {isa = PBXBuildFile; fileRef = 29B97318FDCFA39411CA2CEA /* MainMenu.nib */; };

src/MacVim/MMAppController.m

@@ -162,6 +162,16 @@ be in when executing these commands):
 1. Configure Vim (call "./configure --help" to see a list of flags)
     src/$ ./configure
 
+   MacVim-specific flags that `./configure` supports:
+   --disable-sparkle: Disable Sparkle auto-update. This is useful if you want
+                      to manually sync or building this as part of a package
+                      manager.
+   --with-macsdk:     Build MacVim against specific SDK versions.
+
+   You can also use environment variable `MACOSX_DEPLOYMENT_TARGET` to specify
+   minimum target macOS version to deploy (e.g. '10.14'), and `XCODEFLAGS` for
+   additional command-line arguments to pass to `xcodebuild`.
+
 2. Build
     src/$ make