Merge pull request #12 from yock/ruby-git

refactor: Replacing system() call for Git commands with ruby-git

Author: yock

.gitmodules

@@ -1,6 +1,7 @@
 source 'https://rubygems.org'
 
 gem 'thor'
+gem 'git'
 
 group :development, :test do
   gem 'rspec', '~> 2.14'

Gemfile

@@ -7,6 +7,7 @@ GEM
     columnize (0.3.6)
     debugger-linecache (1.2.0)
     diff-lcs (1.2.5)
+    git (1.2.6)
     rspec (2.14.1)
       rspec-core (~> 2.14.0)
       rspec-expectations (~> 2.14.0)
@@ -22,5 +23,6 @@ PLATFORMS
 
 DEPENDENCIES
   byebug
+  git
   rspec (~> 2.14)
   thor

Gemfile.lock

@@ -2,7 +2,9 @@ $:.unshift File.expand_path("../lib", __FILE__)
 
 Gem::Specification.new do |s|
   s.name = 'forte'
-  s.version = '0.0.2'
+  s.description = 'Create an authorized_keys file for a shared shell account from a repository of public key files '
+  s.homepage = 'https://github.com/yock/forte'
+  s.version = '0.0.3'
   s.summary = 'Create an authorized_keys file for a shared shell account from a repository of public key files'
   s.platform = Gem::Platform::RUBY
   s.authors = ['Michael Yockey', 'Ryan Cromwell']

forte.gemspec

@@ -1,2 +1,2 @@
 #!/usr/bin/env sh
-ruby -Ilib bin/forte
+ruby -Ilib bin/forte print https://github.com/yock/forte.git

forte.sh

@@ -1,5 +1,7 @@
 require 'thor'
-require 'forte/git'
+require 'git'
+require 'fileutils'
 require 'forte/authorized_keys'
 require 'forte/base'
 require 'forte/keyfile'
+require 'forte/repo_location'

lib/forte.rb

@@ -1,8 +1,9 @@
 class Base < Thor
   desc "print", "Prints the public keys to STDOUT"
   def print(uri)
-    dir = Git.clone(uri)
-    puts AuthorizedKeys.build(dir)
-    Git.cleanup
+    loc = RepoLocation.new(uri)
+    repo = Git.clone(loc.uri, File.join(Dir.pwd, './.tmp'))
+    puts AuthorizedKeys.build(repo.dir.path)
+    FileUtils.rm_rf(repo.dir.path)
   end
 end

lib/forte/base.rb

@@ -0,0 +1,14 @@
+class RepoLocation
+  attr_accessor :path
+
+  def initialize(path)
+    @path = path
+  end
+
+  def uri
+    return path if path.start_with?('http')
+    return path if Dir.exists?(File.join(path, '.git'))
+    "https://github.com/#{path}.git"
+  end
+
+end