Merge pull request #102 from EugenMayer/feature/upgrade-checks

implemented upgrade checks hooks, inform user about the unison rename…

lib/docker-sync/upgrade_check.rb

@@ -0,0 +1,59 @@
+require 'gem_update_checker'
+require 'thor/actions'
+require 'docker-sync/config'
+
+class UpgradeChecker
+  include Thor::Shell
+  @config
+  def initialize
+    @config = DockerSyncConfig::global_config
+  end
+
+  def run
+    unless should_run
+      return
+    end
+    check_and_warn
+  end
+
+  def last_upgraded_version
+    @config['upgrade_status'] || ''
+  end
+
+  def should_run
+    # get the update_status which is the version of the update hook which has been run already
+    upgrade_status = last_upgraded_version
+    if upgrade_status == '' || Gem::Version.new(upgrade_status) < Gem::Version.new(get_current_version) # thats how we compare the version
+      return true
+    end
+
+    return false
+  end
+
+
+  def get_current_version
+    path = File.expand_path('../../../', __FILE__)
+    return File.read("#{path}/VERSION")
+  end
+
+  def docker_sync_update_check
+    gem_name = 'docker-sync'
+    current_version = get_current_version
+    checker = GemUpdateChecker::Client.new(gem_name, current_version)
+    return checker
+  end
+
+  def check_and_warn
+    # this is the upgrade hook for the unison-unox introduction / rename of unison
+    if Gem::Version.new(last_upgraded_version) <  Gem::Version.new('0.1.0')
+      Thor::Shell::Basic.new.say_status 'warning', 'Please be aware that with the strategy "unison" is now called unison-onesided and you might need to migrate. See https://github.com/EugenMayer/docker-sync/wiki/Migration-Guide for more informations', :red
+      unless Thor::Shell::Basic.new.yes?('Shall we continue?')
+        exit 1
+      end
+    end
+
+    # update the upgrade_status
+    @config['upgrade_status'] = "#{get_current_version}"
+    DockerSyncConfig::global_config_save(@config)
+  end
+end

tasks/stack/stack.thor

@@ -2,6 +2,7 @@ require 'docker-sync/sync_manager'
 require 'docker-sync/config'
 require 'docker-sync/preconditions'
 require 'docker-sync/update_check'
+require 'docker-sync/upgrade_check'
 require 'docker/compose'
 require 'docker-sync/compose'
 class Stack < Thor
@@ -15,6 +16,9 @@ class Stack < Thor
     updates = UpdateChecker.new
     updates.run
 
+    upgrades = UpgradeChecker.new
+    upgrades.run
+
     begin
       Preconditions::check_all_preconditions
     rescue Exception => e

tasks/sync/sync.thor

@@ -2,6 +2,7 @@ require 'docker-sync/sync_manager'
 require 'docker-sync/config'
 require 'docker-sync/preconditions'
 require 'docker-sync/update_check'
+require 'docker-sync/upgrade_check'
 
 class Sync < Thor
 
@@ -13,7 +14,8 @@ class Sync < Thor
     # do run update check in the start command only
     updates = UpdateChecker.new
     updates.run
-
+    upgrades = UpgradeChecker.new
+    upgrades.run
     begin
       Preconditions::check_all_preconditions
     rescue Exception => e
@@ -112,5 +114,11 @@ class Sync < Thor
       print_table(config) if options['verbose']
     end
   end
+  desc 'start', 'Start all sync configurations in this project'
+  def upgrade
+    # do run update check in the start command only
+    upgrades = UpgradeChecker.new
+    upgrades.run
+  end
 
 end