motion-sparkle-sandbox 2.0.0 → 2.0.1

data/lib/motion/project/rake_tasks.rb

@@ -1,18 +1,27 @@
-# Rake tasks
+# frozen_string_literal: true
+
+# Sparkle specific rake tasks
 namespace :sparkle do
+  desc 'Sparkle Help'
+  task :help do
+    puts <<~HELP
+      During initial Sparkle setup, run these rake tasks:
 
-  task :install do
-    sparkle = App.config.sparkle
-    sparkle.install
+      1. `rake sparkle:setup_certificates`
+      2. `rake sparkle:setup`
+
+      Then after running `rake build:release`, you can run
+      `rake sparkle:package`
+    HELP
   end
 
-  desc "Setup Sparkle configuration"
+  desc 'Setup Sparkle configuration'
   task :setup do
     sparkle = App.config.sparkle
     sparkle.setup
   end
 
-  desc "Create a ZIP file with you application .app release build"
+  desc 'Create a ZIP file with your application .app release build'
   task :package do
     App.config_without_setup.build_mode = :release
     sparkle = App.config.sparkle
@@ -24,50 +33,37 @@ namespace :sparkle do
     sparkle.generate_keys
   end
 
-  desc "Sign the ZIP file with appropriate certificates"
+  desc 'Generate the EdDSA signature for a package'
   task :sign do
     App.config_without_setup.build_mode = :release
     sparkle = App.config.sparkle
     sparkle.sign_package
   end
 
-  task :recreate_public_key do
-    sparkle = App.config.sparkle
-    sparkle.generate_public_key
-  end
-
-  task :copy_release_notes_templates do
-    App.config_without_setup.build_mode = :release
-    sparkle = App.config.sparkle
-    sparkle.copy_templates(force = true)
-  end
-
-  desc "Generate the appcast xml feed"
-  task :feed do
+  desc "Generate the appcast xml feed using Sparkle's `generate_appcast`"
+  task :generate_appcast do
     App.config_without_setup.build_mode = :release
     sparkle = App.config.sparkle
-    sparkle.create_appcast
+    sparkle.generate_appcast
   end
 
-  desc "Generate the appcast using Sparkle's `generate_appcast`"
-  task :generate_appcast do
-    App.config_without_setup.build_mode = :release
-    sparkle = App.config.sparkle
-    results = `#{sparkle.sparkle_vendor_path}/generate_appcast "#{sparkle.private_key_path}" "#{sparkle.archive_folder}"`
+  namespace :generate_appcast do
+    desc "Show help for Sparkle's `generate_appcast`"
+    task :help do
+      App.config_without_setup.build_mode = :release
+      sparkle = App.config.sparkle
+      sparkle.generate_appcast_help
+    end
   end
 
-  desc "Update the release notes of this build"
+  desc 'Update the release notes of this build'
   task :release_notes do
     App.config_without_setup.build_mode = :release
     sparkle = App.config.sparkle
     sparkle.create_release_notes
   end
 
-  desc "Upload to configured location"
-  task :upload do
-  end
-
-  desc "Clean the Sparkle release folder"
+  desc 'Clean the Sparkle release folder'
   task :clean do
     dir = Motion::Project::Sparkle::RELEASE_PATH
     if File.exist?("./#{dir}")

data/lib/motion/project/setup.rb

@@ -1,96 +1,90 @@
-module Motion::Project
-  class Sparkle
+# frozen_string_literal: true
 
-    def setup_ok?
-      config_ok?
-      certificates_ok?
-    end
+module Motion
+  module Project
+    class Sparkle
+      def setup_ok?
+        config_ok?
+        certificates_ok?
+      end
 
-    def config_ok?
-      check_base_url
-      check_feed_url
-      check_public_key
-    end
+      def config_ok?
+        check_base_url
+        check_feed_url
+        check_public_key
+      end
+
+      def check_base_url
+        return true if appcast.base_url.present?
 
-    def check_base_url
-      base_url_check = appcast.base_url.to_s
-      if base_url_check.nil? or base_url_check.empty?
         App.fail "Sparkle :base_url missing. Use `release :base_url, 'http://example.com/your_app_folder'` in your Rakefile's `app.sparkle` block"
       end
-      true
-    end
 
-    def check_feed_url
-      feed_url_check = @config.info_plist['SUFeedURL']
-      feed_filename_check = appcast.feed_filename
-      if feed_url_check.nil? or feed_url_check.empty? or feed_filename_check.nil? or feed_filename_check.empty?
-        App.fail "Sparkle :feed_filename is nil or blank. Please check your Rakefile."
+      def check_feed_url
+        return true if feed_url.present? && appcast.feed_filename.present?
+
+        App.fail 'Sparkle :feed_filename is nil or blank. Please check your Rakefile.'
       end
-      true
-    end
 
-    def check_public_key
-      public_key_check = @config.info_plist['SUPublicDSAKeyFile'].to_s
-      if public_key_check.nil? or public_key_check.empty?
-        App.fail "Sparkle :public_key is nil or blank. Please check your Rakefile."
+      def check_public_key
+        return true if public_EdDSA_key.present?
+
+        App.fail 'Sparkle :public_key is nil or blank. Please check your Rakefile.'
       end
-      true
-    end
 
-    def certificates_ok?(silence=false)
-      unless File.exist?("./#{Sparkle::CONFIG_PATH}")
-        if silence
-          return false
-        else
-          App.fail "Missing `#{Sparkle::CONFIG_PATH}`. Run `rake sparkle:setup` to get started" 
+      def certificates_ok?(silence = false)
+        unless File.exist?("./#{Sparkle::CONFIG_PATH}")
+          return false if silence
+
+          App.fail "Missing `#{Sparkle::CONFIG_PATH}`. Run `rake sparkle:setup` to get started"
         end
-      end
-      unless File.exist?(private_key_path)
-        if silence
-          return false
-        else
+
+        if appcast.use_exported_private_key && !File.exist?(private_key_path)
+          return false if silence
+
           App.fail "Missing `#{private_key_path}`. Please run `rake sparkle:setup_certificates` or check the docs to know where to put them."
         end
-      end
-      unless File.exist?(public_key_path)
-        if silence
-          return false
-        else
+
+        unless public_EdDSA_key.present?
+          return false if silence
+
           App.fail "Missing `#{public_key_path}`. Did you configure `release :public_key` correctly in the Rakefile? Advanced: recreate your public key with `rake sparkle:recreate_public_key`"
         end
+
+        true
       end
-      true
-    end
 
-    def setup
-      verify_installation
-      create_sparkle_folder
-      add_to_gitignore
-      copy_templates
+      def setup
+        verify_installation
+        create_sparkle_folder
+        add_to_gitignore
+        copy_templates
 
-      if config_ok?
-        App.info "Sparkle", "Config found"
-      else
-        return false
-      end
+        return false unless config_ok?
+
+        App.info 'Sparkle', 'Config found'
 
-      silence = true
-      if certificates_ok?(silence)
-        App.info "Sparkle", "Certificates found"
-      else
-        App.info "Sparkle", "Certificates not found
-Please generate your private and public keys with
-    `rake sparkle:setup_certificates`
-If you already have your certificates and only need to include them in the project, follow these steps:
-    1. Rename your private key to `./#{private_key_path}`
-    2. Place your public key in `./#{public_key_path}`
-    3. If you wish to use a different name or location for your public key within the resources dir, 
-       make sure you add `publish :public_key, 'folder/new_name.pem'` to the sparkle config in your Rakefile
-        "
-        return false
+        silence = true
+        unless certificates_ok?(silence)
+          App.info 'Sparkle', <<~CERTIFICATES
+            Certificates not found
+
+                        Please generate your private and public keys with
+                           `rake sparkle:setup_certificates`
+
+                        If you already have your certificates and only need to include them in the project, follow these steps:
+                           1. Rename your private key to `./#{private_key_path}`
+                              and make sure you've added it to your `.gitignore` file - it should NEVER be
+                              stored in your repository
+                           2. Add `publish :public_key, 'PUBLIC_KEY'` to the Sparkle config in your Rakefile
+          CERTIFICATES
+
+          return false
+        end
+
+        App.info 'Sparkle', 'Certificates found'
+        App.info 'Sparkle', 'Setup OK. After `rake build:release`, you can now run `rake sparkle:package`.'
       end
-      App.info "Sparkle", "Setup OK. After `rake build:release`, you can now run `rake sparkle:package`."
     end
-
   end
 end