ppgit 0.6.0 → 0.7.0

data/CHANGELOG

@@ -1,3 +1,9 @@
+0.7.0
+  - added a --names_separator option, if you don't like the default '+'
+    * `ppgit ara nva`                             ==> user.name = 'ara+nva'
+    * `ppgit ara nva  --names_separator _and`     ==> user.name = 'ara_and_nva'
+  - improved (coloured) the result of `ppgit info`
+  - when a new version is available, display the changelog.
 0.6.0
   - use '+' in the pair name, instead of '_'
 0.5.0
@@ -6,7 +12,7 @@
 0.4.0
   - use a bash & platform-agnostic ruby location
   - fixed that ppgit.emailroot would be stored in the local config file, instead of ~/.gitconfig.
-  - 'ppgit clear' does not restore user.email/user.name values if they are already present in ~/.gitconfig.
+  - `ppgit clear` does not restore user.email/user.name values if they are already present in ~/.gitconfig.
   - print Usage text when user enters invalid command
 0.3.0
  - display the info message after each command

data/VERSION

@@ -1 +1 @@
-0.6.0
+0.7.0

data/bin/git-pp

@@ -15,12 +15,20 @@ else
   LOCAL_CONFIG_FILE  = in_test_mode ? config_file_part('--file'       ) : '--file .git/config'
   GLOBAL_CONFIG_FILE = in_test_mode ? config_file_part('--global_file') : '--global'
 
+  if names_separator = argv_value_of('--names_separator')
+    set_global_git_value 'ppgit.namesseparator', names_separator
+  end
+
+  if get_global_git_value('ppgit.namesseparator').blank? && !ARGV.include?('clear')
+    # write the default value in the config file, as hint/documentation
+    set_global_git_value 'ppgit.namesseparator', Ppgit::DEFAULT_PAIR_NAME_SEPARATOR
+  end
+
   if emailroot = argv_value_of('--email_root')
-    do_ppgit_emailroot(emailroot)
-    do_ppgit_info() if ARGV.empty?
+    set_global_git_value 'ppgit.emailroot', emailroot
   end
 
-  if ARGV == %w(info)
+  if ARGV.empty? || ARGV == %w(info)
     do_ppgit_info()
 
   elsif ARGV == %w(clear)
@@ -28,7 +36,7 @@ else
     do_ppgit_info()
 
   elsif [2, 3].include?(ARGV.length)
-    do_ppgit_set_pair_as_a_user(user_1=ARGV[0], user_2=ARGV[1], pair_email=ARGV[2])
+    do_ppgit_set_pair_as_a_user(user_1=ARGV[0], user_2=ARGV[1], pair_email=ARGV[2], names_separator=get_global_git_value('ppgit.namesseparator'))
     do_ppgit_info()
 
   elsif 0 != ARGV.length
@@ -39,7 +47,10 @@ else
     puts "\n----------------------------\n\n"
   end
 
-  unless in_test_mode
-    print_message_if_new_version_available('alainravet', 'ppgit')
+  unless in_test_mode || 'true' == ENV['PPGIT_NO_VERSION_CHECK']
+    print_message_if_new_version_available
+    puts "Note: you can prevent the version check with :"
+    puts "   PPGIT_NO_VERSION_CHECK=true ppgit <command>"
+    puts ''
   end
 end

data/bin/ppgit

@@ -15,12 +15,20 @@ else
   LOCAL_CONFIG_FILE  = in_test_mode ? config_file_part('--file'       ) : '--file .git/config'
   GLOBAL_CONFIG_FILE = in_test_mode ? config_file_part('--global_file') : '--global'
 
+  if names_separator = argv_value_of('--names_separator')
+    set_global_git_value 'ppgit.namesseparator', names_separator
+  end
+
+  if get_global_git_value('ppgit.namesseparator').blank? && !ARGV.include?('clear')
+    # write the default value in the config file, as hint/documentation
+    set_global_git_value 'ppgit.namesseparator', Ppgit::DEFAULT_PAIR_NAME_SEPARATOR
+  end
+
   if emailroot = argv_value_of('--email_root')
-    do_ppgit_emailroot(emailroot)
-    do_ppgit_info() if ARGV.empty?
+    set_global_git_value 'ppgit.emailroot', emailroot
   end
 
-  if ARGV == %w(info)
+  if ARGV.empty? || ARGV == %w(info)
     do_ppgit_info()
 
   elsif ARGV == %w(clear)
@@ -28,7 +36,7 @@ else
     do_ppgit_info()
 
   elsif [2, 3].include?(ARGV.length)
-    do_ppgit_set_pair_as_a_user(user_1=ARGV[0], user_2=ARGV[1], pair_email=ARGV[2])
+    do_ppgit_set_pair_as_a_user(user_1=ARGV[0], user_2=ARGV[1], pair_email=ARGV[2], names_separator=get_global_git_value('ppgit.namesseparator'))
     do_ppgit_info()
 
   elsif 0 != ARGV.length
@@ -39,7 +47,10 @@ else
     puts "\n----------------------------\n\n"
   end
 
-  unless in_test_mode
-    print_message_if_new_version_available('alainravet', 'ppgit')
+  unless in_test_mode || 'true' == ENV['PPGIT_NO_VERSION_CHECK']
+    print_message_if_new_version_available
+    puts "Note: you can prevent the version check with :"
+    puts "   PPGIT_NO_VERSION_CHECK=true ppgit <command>"
+    puts ''
   end
 end

data/lib/ppgit.rb

@@ -1 +1,9 @@
+class Ppgit
+  class Github
+    USER    = 'alainravet'
+    PROJECT = 'ppgit'
+  end
+  DEFAULT_PAIR_NAME_SEPARATOR       = '+'
+  DEFAULT_PAIR_EMAIL_USER_SEPARATOR = '_'
+end
 require 'ppgit/commands'

data/lib/ppgit/commands.rb

@@ -1,7 +1,6 @@
 require 'ppgit/git_utils'
 require 'ppgit/ppgit_utils'
-require 'ppgit/version_utils'
-
+require 'ppgit/gem_version_utils'
 
 def do_ppgit_clear
   backup_is_same_as_global?('name') ?
@@ -17,29 +16,23 @@ def do_ppgit_clear
 end
 
 
-def do_ppgit_emailroot(emailroot)
-  set_global_git_value 'ppgit.emailroot', emailroot
-end
-
-
-def do_ppgit_set_pair_as_a_user(user_1, user_2, pair_email)
+def do_ppgit_set_pair_as_a_user(user_1, user_2, pair_email, names_separator)
   backup_git_value :from => 'user.name' , :to => 'user-before-ppgit.name'
   backup_git_value :from => 'user.email', :to => 'user-before-ppgit.email'
 
-  two_users  = [user_1, user_2]
-  pair_email ||= make_email_from_email_root_and_user(two_users.sort.join('_'))
+  pair_email ||= make_email_from_email_root_and_user(user_1, user_2)
 
-  set_local_git_value 'user.name', two_users.sort.join('+')
+  set_local_git_value 'user.name', assemble_pair_name(user_1, user_2, names_separator)
   if pair_email
     set_local_git_value 'user.email', pair_email
   end
 end
 
-def make_email_from_email_root_and_user(pair_user)
+def make_email_from_email_root_and_user(user_1, user_2)
   emailroot = get_global_git_value('ppgit.emailroot')
   emailroot.blank? ?
       nil :
-      emailroot.gsub('*', pair_user)
+      emailroot.gsub('*', assemble_email_user(user_1, user_2))
 end
 
 
@@ -60,8 +53,9 @@ def do_ppgit_info
 end
 
 def ppgit_info(file)
-  name, email = get_value('user.name', file), get_value('user.email', file)
-  email_root  = get_value('ppgit.emailroot', file)
+  name, email     = get_value('user.name', file), get_value('user.email', file)
+  names_separator = get_value('ppgit.namesseparator', file)
+  email_root      = get_value('ppgit.emailroot'     , file)
   s = []
   s << "-------------------------------------------------------"
   s << "file = " + ((file == '--global') ? '~/.gitconfig' : file.gsub('--file ',''))
@@ -70,18 +64,27 @@ def ppgit_info(file)
 #    s << '  [user] is empty (user.email and user.name are not set)'
 #  else
     s << '  [user]'
-    s << "    name  = #{name }" unless name.blank?
-    s << "    email = #{email}" unless email.blank?
+    s << red("    name  = #{name }") unless name.blank?
+    s << red("    email = #{email}") unless email.blank?
     s << '  -------------------------------------------------------'
   end
-  unless email_root.blank?
+
+  unless email_root.blank? && names_separator.blank?
     s << '  [ppgit]'
-    s << "    emailroot = #{email_root}" unless email_root.blank?
+    unless email_root.blank?
+      sample_email = make_email_from_email_root_and_user('ann', 'bob')
+      s << "    emailroot = #{yellow(email_root)}   # -> `ppgit ann bob`  -> user.email = #{yellow(sample_email)}"
+    end
+    unless names_separator.blank?
+      sample_name = assemble_pair_name('ann', 'bob', names_separator)
+      s << "    namesseparator = #{yellow(names_separator)}    # -> `ppgit ann bob`  -> user.name = #{yellow(sample_name)}"
+    end
     s << '  -------------------------------------------------------'
   end
+
   name, email = get_value('user-before-ppgit.name', file), get_value('user-before-ppgit.email', file)
   unless name.blank? && email.blank?
-    s << '  [user-before-ppgit]'
+    s << '  [user-before-ppgit]  # values restored by `ppgit clear`'
     s << "    name  = #{name }" unless name.blank?
     s << "    email = #{email}" unless email.blank?
     s << '  -------------------------------------------------------'

data/lib/ppgit/gem_version_utils.rb

@@ -0,0 +1,40 @@
+require 'ppgit/github_utils'
+
+def this_version
+  version_file = File.join(File.expand_path(File.dirname(__FILE__)), '..', '..', 'VERSION')
+  File.open(version_file).read.chomp
+end
+
+def new_version_available?(user, project)
+  latest_version = latest_github_version(user, project)
+  # latest_version.nil? if the info cannot be retrieved from github.com
+  latest_version && this_version != latest_version
+end
+
+def print_message_if_new_version_available
+  user    = Ppgit::Github::USER
+  project = Ppgit::Github::PROJECT
+  changelog = latest_github_changelog(user, project, 'master', 'CHANGELOG', this_version)
+
+  if new_version_available?(user, project)
+    s = ["\033[1;33m"]  # yellow
+    s << '------------------------------------------------------'
+    s << "  There is a new version of #{project} (#{latest_github_version(user, project)})."
+    s << "  You are using version  #{this_version}"
+    s << '  To update :'
+    s << "      #{red("gem update " + project)}"
+    s << '------------------------------------------------------'
+    s << "\033[0m"      # no color
+    s << '------------------------------------------------------'
+    unless changelog.empty?
+      s << " CHANGELOG :"
+      s << '------------------------------------------------------'
+      s << changelog.map{|l| l.chomp}
+      s << this_version
+      s << '  ...'
+      s << '------------------------------------------------------'
+    end
+    msg = s.join("\n")
+    puts msg
+  end
+end

data/lib/ppgit/github_utils.rb

@@ -0,0 +1,32 @@
+require 'open-uri'
+
+def latest_github_version(user, project, branch='master', file='VERSION')
+  @@latest_github_version ||= read_file_from_github(user, project, branch, file)
+end
+
+def latest_github_changelog(user, project, branch='master', file='CHANGELOG', old_version=nil)
+  all_lines = read_file_lines_from_github(user, project, branch, file)
+  (all_lines || []).map{|line| line.chomp}.take_while{|line| line != old_version}
+end
+
+
+def read_file_from_github(user, project, branch, file)
+  file_path = "http://github.com/#{user}/#{project}/raw/#{branch}/#{file}"
+  begin
+    contents = open(file_path).read.chomp # ex: '1.2.3'
+  rescue => e #SocketError, OpenURI::HTTPError
+    puts file_path
+    puts e.inspect
+    nil    # unable to read the file
+  end
+end
+
+def read_file_lines_from_github(user, project, branch, file)
+  file_path = "http://github.com/#{user}/#{project}/raw/#{branch}/#{file}"
+  begin
+    open(file_path).readlines
+  rescue SocketError, OpenURI::HTTPError
+    nil    # unable to read the file
+  end
+end
+

data/lib/ppgit/ppgit_utils.rb

@@ -1,6 +1,22 @@
 require 'ppgit/utils'
 require 'ppgit/git_utils'
 
+# ------------------------------------------------------------------------
+
+#  'ann', 'bob'  => ''ann+bob'
+def assemble_pair_name(user_1, user_2, names_separator)
+  names_separator = Ppgit::DEFAULT_PAIR_NAME_SEPARATOR if names_separator.blank?
+  [user_1, user_2].sort.join(names_separator)
+end
+
+#  'ann', ''bob'  => ''ann+bob'
+
+def assemble_email_user(user_1, user_2)
+  [user_1, user_2].sort.join(Ppgit::DEFAULT_PAIR_EMAIL_USER_SEPARATOR)
+end
+
+# ------------------------------------------------------------------------
+
 # separator = --file, or --global_file
 # Returns :
 #   nil

data/lib/ppgit/usage.txt

@@ -6,12 +6,12 @@ ppgit : quickly update ~/.gitconfig for pair-programming sessions.
 
   Quick Usage:
 
-    $ ppgit john andy andy_john@acme.com
+    $ ppgit john andy
     $ ppgit clear
     $ ppgit info
 
-    $ ppgit --email_root *@acme.com
-    $ ppgit john andy
+    $ ppgit john andy andy_john@acme.com
+    $ ppgit --email_root *@acme.com --names_separator _and_
 
 remark : 'ppgit' is a synonym of 'git pp' => you can write
 

data/ppgit.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{ppgit}
-  s.version = "0.6.0"
+  s.version = "0.7.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Alain Ravet"]
-  s.date = %q{2010-10-07}
+  s.date = %q{2010-10-10}
   s.description = %q{git users' pairs switcher}
   s.email = %q{alain.ravet+git@gmail.com}
   s.executables = ["git-pp", "ppgit"]
@@ -32,16 +32,18 @@ Gem::Specification.new do |s|
      "doc/ppgit-img3.png",
      "lib/ppgit.rb",
      "lib/ppgit/commands.rb",
+     "lib/ppgit/gem_version_utils.rb",
      "lib/ppgit/git_utils.rb",
+     "lib/ppgit/github_utils.rb",
      "lib/ppgit/ppgit_utils.rb",
      "lib/ppgit/quick_usage.txt",
      "lib/ppgit/usage.txt",
      "lib/ppgit/utils.rb",
-     "lib/ppgit/version_utils.rb",
      "ppgit.gemspec",
      "spec/ppgit_clear_spec.rb",
      "spec/ppgit_email_root_spec.rb",
-     "spec/ppgit_info_spec.rb",
+     "spec/ppgit_info_spec.rb.disabled",
+     "spec/ppgit_names_separator_spec.rb",
      "spec/ppgit_spec.rb",
      "spec/ppgit_usage_spec.rb",
      "spec/spec.opts",
@@ -55,7 +57,7 @@ Gem::Specification.new do |s|
   s.test_files = [
     "spec/ppgit_clear_spec.rb",
      "spec/ppgit_email_root_spec.rb",
-     "spec/ppgit_info_spec.rb",
+     "spec/ppgit_names_separator_spec.rb",
      "spec/ppgit_spec.rb",
      "spec/ppgit_usage_spec.rb",
      "spec/spec_helper.rb"

data/spec/ppgit_email_root_spec.rb

@@ -16,7 +16,8 @@ describe "simple: `ppgit --email_root acme+*@gmail.com`" do
 
     it 'stores the emailroot in --global (~/.gitconfig)' do
       expected_global= ['[ppgit]'                         ,
-                        '  emailroot = acme+*@gmail.com'
+                        '  namesseparator = +'            ,
+                        '  emailroot = acme+*@gmail.com'  ,
                         ]
       @actual_global.should == expected_global.join("\n")
     end
@@ -27,7 +28,8 @@ describe "simple: `ppgit --email_root acme+*@gmail.com`" do
     before(:all) do
       @before_local    = [ ]
       @before_global   =  ['[ppgit]'                     ,
-                          '  emailroot = old+*@gmail.com' # <= the old value to replace
+                           '  namesseparator = +'        ,
+                          '  emailroot = old+*@gmail.com', # <= the old value to replace
                           ]
       @actual_local, @actual_global = execute_command_g(@cmd, @before_local, @before_global)
     end
@@ -36,7 +38,8 @@ describe "simple: `ppgit --email_root acme+*@gmail.com`" do
 
     it 'stores the new emailroot in --global (~/.gitconfig)' do
       expected_global= ['[ppgit]'                         ,
-                        '  emailroot = acme+*@gmail.com'
+                        '  namesseparator = +'            ,
+                        '  emailroot = acme+*@gmail.com'  ,
                         ]
       @actual_global.should == expected_global.join("\n")
     end
@@ -60,6 +63,7 @@ describe "medium: `ppgit john andy --email_root acme+*@gmail.com`" do
   end
   it 'stores the emailroot in --global (~/.gitconfig)' do
     expected_global= ['[ppgit]'                         ,
+                      '  namesseparator = +'            ,
                       '  emailroot = acme+*@gmail.com'
                       ].join("\n")
     @actual_global.should == expected_global
@@ -85,6 +89,7 @@ describe "complex: `ppgit john andy andy_john@test.com --email_root acme+*@gmail
 
   it 'stores the emailroot in --global (~/.gitconfig)' do
     expected_global= ['[ppgit]'                         ,
+                      '  namesseparator = +'            ,
                       '  emailroot = acme+*@gmail.com'
                       ].join("\n")
     @actual_global.should == expected_global

data/spec/{ppgit_info_spec.rb → ppgit_info_spec.rb.disabled}

@@ -13,6 +13,7 @@ describe "`ppgit info`" do
                             '  email = johnuser@gmail.be'   ]
 
       before_global   = [   '[ppgit]'                       ,
+                            'namesseparator = ZEP'          ,
                             '  emailroot = acme+*@gmail.com'  ]
       ignore, ignore, @output = execute_command_g( ppgit("info"), before_local, before_global)
     end
@@ -22,6 +23,9 @@ describe "`ppgit info`" do
     it('should display the [ppgit].emailroot info'    ) {
       @output.should match(/\[ppgit\].+emailroot\s+=\s+acme\+\*@gmail.com/mi)
     }
+    it('should display the [ppgit].namesseparator info'    ) {
+      @output.should match(/\[ppgit\].+namesseparator\s+=\s+ZEP/mi)
+    }
   end
 
   context "when NONE of the [user] and the ppgit variables are stored in the config file"  do
@@ -35,6 +39,8 @@ describe "`ppgit info`" do
     it('should not display the empty [user] section'                 ) { @output.should_not match(/\[user\]/mi) }
 
     it('should NOT display the empty [user-before-ppgit] section') { @output.should_not match(/\[user-before-ppgit\]/mi) }
-    it('should NOT display the [ppgit] info'                     ) { @output.should_not match(/\[ppgit\]/mi)             }
+    it('should insert and display the default [ppgit].namesseparator info'    ) {
+      @output.should match(/\[ppgit\].+namesseparator\s+=\s+\+/mi)
+    }
   end
 end

data/spec/ppgit_names_separator_spec.rb

@@ -0,0 +1,68 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe "simple: `ppgit --names_separator _SEP_`" do
+
+  before(:all) do
+    @cmd = "#{PPGIT_CMD} --names_separator _SEP_"
+  end
+
+  context 'when there is no namesseparator yet' do
+    before(:all) do
+      @before_local    = [ ]
+      @before_global   = [ ] # no pair_separator here.
+      @actual_local, @actual_global = execute_command_g(@cmd, @before_local, @before_global)
+    end
+
+    it("doesn't change .git/config") { @actual_local.should == @before_local.join("\n") }
+
+    it 'stores the names separator in --global (~/.gitconfig)' do
+      expected_global= ['[ppgit]'                         ,
+                        '  namesseparator = _SEP_'
+                        ]
+      @actual_global.should == expected_global.join("\n")
+    end
+  end
+
+  context 'when there is already an namesseparator' do
+    before(:all) do
+      @before_local    = [ ]
+      @before_global   =  ['[ppgit]'                     ,
+                          '  namesseparator = OLD_SEP' # <= the old value to replace
+                          ]
+      @actual_local, @actual_global = execute_command_g(@cmd, @before_local, @before_global)
+    end
+
+    it("doesn't change .git/config") { @actual_local.should == @before_local.join("\n") }
+
+    it 'stores the new names separator in --global (~/.gitconfig)' do
+      expected_global= ['[ppgit]'                         ,
+                        '  namesseparator = _SEP_'
+                        ]
+      @actual_global.should == expected_global.join("\n")
+    end
+  end
+end
+
+describe "medium: `ppgit john andy --names_separator _SEP_`" do
+  before(:all) do
+    @cmd = "#{PPGIT_CMD} john andy --names_separator _SEP_"
+    before_local    = [  ]
+    before_global   = [  ]
+    @actual_local, @actual_global = execute_command_g(@cmd, before_local, before_global)
+  end
+
+  it 'stores the user in local (.git/config)' do
+    expected_local = [  '[user]'                          ,
+                        '  name = andy_SEP_john'              ,
+                      ].join("\n")
+    @actual_local.should == expected_local
+  end
+  it 'stores the names separator in --global (~/.gitconfig)' do
+    expected_global= ['[ppgit]'                         ,
+                      '  namesseparator = _SEP_'
+                      ].join("\n")
+    @actual_global.should == expected_global
+  end
+end
+
+

data/spec/ppgit_spec.rb

@@ -18,7 +18,10 @@ context "with empty git config files"  do
                         '  name = andy+john'  ]
       @actual_local.should == expected_local.join("\n")
     end
-    it("doesn't change --global (~/.gitconfig)") { @actual_global.should == @before_global.join("\n") }
+    it("adds the default names separator to the global config (~/.gitconfig)") do
+      augmented_global = @before_global + ['[ppgit]', "  namesseparator = #{Ppgit::DEFAULT_PAIR_NAME_SEPARATOR}"]
+      @actual_global.should == (augmented_global).join("\n")
+    end
   end
 
 
@@ -34,7 +37,10 @@ context "with empty git config files"  do
 
       @actual_local.should == expected_local.join("\n")
     end
-    it("doesn't change --global (~/.gitconfig)") { @actual_global.should == @before_global.join("\n") }
+    it("adds the default names separator to the global config (~/.gitconfig)") do
+      augmented_global = @before_global + ['[ppgit]', "  namesseparator = #{Ppgit::DEFAULT_PAIR_NAME_SEPARATOR}"]
+      @actual_global.should == (augmented_global).join("\n")
+    end
   end
 end
 
@@ -56,7 +62,10 @@ context 'when there is only a ppgit.emailroot' do
                         '  email = pp+andy_john@gmail.com'  ]
       @actual_local.should == expected_local.join("\n")
     end
-    it("doesn't change --global (~/.gitconfig)") { @actual_global.should == @before_global.join("\n") }
+    it("adds the default names separtor to the global config (~/.gitconfig)") do
+      augmented_global = @before_global + ["  namesseparator = #{Ppgit::DEFAULT_PAIR_NAME_SEPARATOR}"]
+      @actual_global.should == (augmented_global).join("\n")
+    end
   end
 end
 

data/spec/spec_helper.rb

@@ -10,14 +10,6 @@ Spec::Runner.configure do |config|
 end
 
 require 'tempfile'
-require 'ftools'
-
-def temp_from_model(model, file_basename=nil)
-  newfile = Tempfile.new(file_basename)
-  source_path = File.expand_path(File.dirname(__FILE__) + "/fixtures/#{model}")
-  File.copy(source_path, newfile.path)
-  newfile
-end
 
 def temp_from_string(string, file_basename=nil)
   f = Tempfile.new(file_basename) 
@@ -26,11 +18,6 @@ def temp_from_string(string, file_basename=nil)
   f
 end
 
-def expected_contents(file_name)
-  f = File.expand_path(File.dirname(__FILE__) + "/fixtures/#{file_name}")
-  File.open(f).read
-end
-
 
 PPGIT_CMD = File.expand_path(File.dirname(__FILE__)) + '/../bin/ppgit'
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: ppgit
 version: !ruby/object:Gem::Version 
-  hash: 7
+  hash: 3
   prerelease: false
   segments: 
   - 0
-  - 6
+  - 7
   - 0
-  version: 0.6.0
+  version: 0.7.0
 platform: ruby
 authors: 
 - Alain Ravet
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-10-07 00:00:00 +02:00
+date: 2010-10-10 00:00:00 +02:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -59,16 +59,18 @@ files:
 - doc/ppgit-img3.png
 - lib/ppgit.rb
 - lib/ppgit/commands.rb
+- lib/ppgit/gem_version_utils.rb
 - lib/ppgit/git_utils.rb
+- lib/ppgit/github_utils.rb
 - lib/ppgit/ppgit_utils.rb
 - lib/ppgit/quick_usage.txt
 - lib/ppgit/usage.txt
 - lib/ppgit/utils.rb
-- lib/ppgit/version_utils.rb
 - ppgit.gemspec
 - spec/ppgit_clear_spec.rb
 - spec/ppgit_email_root_spec.rb
-- spec/ppgit_info_spec.rb
+- spec/ppgit_info_spec.rb.disabled
+- spec/ppgit_names_separator_spec.rb
 - spec/ppgit_spec.rb
 - spec/ppgit_usage_spec.rb
 - spec/spec.opts
@@ -110,7 +112,7 @@ summary: git users' pairs switcher
 test_files: 
 - spec/ppgit_clear_spec.rb
 - spec/ppgit_email_root_spec.rb
-- spec/ppgit_info_spec.rb
+- spec/ppgit_names_separator_spec.rb
 - spec/ppgit_spec.rb
 - spec/ppgit_usage_spec.rb
 - spec/spec_helper.rb

data/lib/ppgit/version_utils.rb

@@ -1,34 +0,0 @@
-require 'open-uri'
-
-def latest_github_version(user, project)
-  version_path = "http://github.com/#{user}/#{project}/raw/master/VERSION"
-  begin
-    version = open(version_path).read.chomp # ex: '1.2.3'
-  rescue SocketError
-    # unable to fetch the version
-  end
-end
-
-def this_version
-  version_file = File.join(File.expand_path(File.dirname(__FILE__)), '..', '..', 'VERSION')
-  File.open(version_file).read.chomp
-end
-
-def new_version_available?(project, user)
-  this_version != latest_github_version(user, project)
-end
-
-def print_message_if_new_version_available(user, project)
-  if new_version_available?(project, user)
-    s = ["\033[1;33m"]  # yellow
-    s << '------------------------------------------------------'
-    s << "  There is a new version of #{project} (#{latest_github_version(user, project)})."
-    s << "  You are using version  #{this_version}"
-    s << '  To update :'
-    s << "      gem update #{project}"
-    s << '------------------------------------------------------'
-    s << "\033[0m"      # no color
-    msg = s.join("\n")
-    puts msg
-  end
-end