librarian-puppet 1.2.0 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9e38c81fc33fcc4e0223be2ce7f705a93f3f0896
-  data.tar.gz: e48f55419ffc8e6fd2df107c08111c35f0c679fa
+  metadata.gz: aea15d42f3f537214c5256248b821f12558b266f
+  data.tar.gz: d0f110467f43c2f090bb8697988731f0020e32b2
 SHA512:
-  metadata.gz: e9f5ade15b27c5a35eb7ada03313f1da390cc5854d1abfe184041640192bcfcd2e6601b4bdf2cfe21ae65eda5bd912a6bf0286c1c1bd1414e5c364b9b673359d
-  data.tar.gz: 2054b93a68ce5f4a9e64c7bf02a057ce274250737528f97ccaa39c4058dad3828c7c7e59bebea6dc8d3d0dd82a75dbcd0640cf6c3d19201a12031e3bb12003f7
+  metadata.gz: 9b0df8f65615913ab73d48d03a8d29fc1753cf26009a96aaf821e49368d8fbbab1b51dd89c513f98644764dcda6bb84c826bb0b4e2bf3132f42964ce3de1aeb8
+  data.tar.gz: 03ffefa4e42cabf289f754b24763d98a44f671f8198f89f27a39758703615e1c090259f9a1426c15d03bb1f2d3e30523b7c32562739c142abb336ba887689164

data/README.md

@@ -37,17 +37,20 @@ See the [Changelog](Changelog.md) for more details.
 
 ## The Puppetfile
 
-Every Puppet repository that uses Librarian-puppet will have a file named
-`Puppetfile` in the root directory of that repository.  The full specification
-for which modules your puppet infrastructure repository  depends goes in here.
+Every Puppet repository that uses Librarian-puppet may have a file named
+`Puppetfile`, `metadata.json` or `Modulefile` in the root directory of that repository.
+The full specification
+for which modules your puppet infrastructure repository depends goes in here.
 
-### Simple Puppetfile
+### Simple usage
 
-This Puppetfile will download all the dependencies listed in your Modulefile from the Puppet Forge
+If no Puppetfile is present, `librarian-puppet` will download all the dependencies
+listed in your `metadata.json` or `Modulefile` from the Puppet Forge,
+as if the Puppetfile contained
 
     forge "https://forgeapi.puppetlabs.com"
 
-    modulefile
+    metadata
 
 
 ### Example Puppetfile

data/lib/librarian/puppet/action.rb

@@ -0,0 +1 @@
+require "librarian/puppet/action/install"

data/lib/librarian/puppet/action/install.rb

@@ -0,0 +1,24 @@
+module Librarian
+  module Puppet
+    module Action
+      class Install < Librarian::Action::Install
+
+        private
+
+        def create_install_path
+          install_path.rmtree if install_path.exist? && destructive?
+          install_path.mkpath
+        end
+
+        def destructive?
+          environment.config_db.local['destructive'] == 'true'
+        end
+
+        def check_specfile
+          # don't fail if Puppetfile doesn't exist as we'll use the Modulefile or metadata.json
+        end
+
+      end
+    end
+  end
+end

data/lib/librarian/puppet/cli.rb

@@ -2,6 +2,7 @@ require 'librarian/helpers'
 
 require 'librarian/cli'
 require 'librarian/puppet'
+require 'librarian/puppet/action'
 
 module Librarian
   module Puppet
@@ -47,11 +48,6 @@ module Librarian
       option "use-v1-api", :type => :boolean, :default => true
       def install
 
-        unless File.exist?('Puppetfile')
-          say "Could not find Puppetfile in #{Dir.pwd}", :red
-          exit 1
-        end
-
         ensure!
         clean! if options["clean"]
         unless options["destructive"].nil?
@@ -89,6 +85,14 @@ module Librarian
       def version
         say "librarian-puppet v#{Librarian::Puppet::VERSION}"
       end
+
+      private
+
+      # override the actions to use our own
+
+      def install!(options = { })
+        Action::Install.new(environment, options).run
+      end
     end
   end
 end

data/lib/librarian/puppet/dsl.rb

@@ -17,6 +17,14 @@ module Librarian
       def run(specfile = nil, sources = [])
         specfile, sources = nil, specfile if specfile.kind_of?(Array) && sources.empty?
 
+        if specfile.kind_of?(Pathname) and !File.exists?(specfile)
+          debug { "Specfile not found, using defaults: #{specfile}" }
+          specfile = Proc.new do
+            forge "https://forgeapi.puppetlabs.com"
+            metadata
+          end
+        end
+
         Target.new(self).tap do |target|
           target.precache_sources(sources)
           debug_named_source_cache("Pre-Cached Sources", target)
@@ -30,17 +38,20 @@ module Librarian
       end
 
       class Receiver < Librarian::Dsl::Receiver
-        attr_reader :specfile
+        attr_reader :specfile, :working_path
 
         # save the specfile and call librarian
         def run(specfile = nil)
+          @working_path = specfile.kind_of?(Pathname) ? specfile.parent : Pathname.new(Dir.pwd)
           @specfile = specfile
           super
         end
 
         # implement the 'modulefile' syntax for Puppetfile
         def modulefile
-          File.read(Pathname.new(specfile).parent.join('Modulefile')).lines.each do |line|
+          f = modulefile_path
+          raise Error, "Modulefile file does not exist: #{f}" unless File.exists?(f)
+          File.read(f).lines.each do |line|
             regexp = /\s*dependency\s+('|")([^'"]+)\1\s*(?:,\s*('|")([^'"]+)\3)?/
             regexp =~ line && mod($2, $4)
           end
@@ -48,11 +59,28 @@ module Librarian
 
         # implement the 'metadata' syntax for Puppetfile
         def metadata
-          dependencyList = JSON.parse(File.read(Pathname.new(specfile).parent.join('metadata.json')))['dependencies']
+          f = working_path.join('metadata.json')
+          unless File.exists?(f)
+            msg = "Metadata file does not exist: #{f}"
+            # try modulefile, in case we don't have a Puppetfile and we are using the default template
+            if File.exists?(modulefile_path)
+              modulefile
+              return
+            else
+              raise Error, msg
+            end
+          end
+          dependencyList = JSON.parse(File.read(f))['dependencies']
           dependencyList.each do |d|
             mod(d['name'], d['version_requirement'])
           end
         end
+
+        private
+
+        def modulefile_path
+          working_path.join('Modulefile')
+        end
       end
     end
   end

data/lib/librarian/puppet/extension.rb

@@ -13,6 +13,9 @@ module Librarian
     def initialize(name, requirement, source)
       assert_name_valid! name
 
+      # Issue #235 fail if forge source is not defined
+      raise Error, "forge entry is not defined in Puppetfile" if source.instance_of?(Array) && source.empty?
+
       # let's settle on provider-module syntax instead of provider/module
       self.name = normalize_name(name)
       self.requirement = Requirement.new(requirement)
@@ -80,22 +83,6 @@ module Librarian
     end
   end
 
-  module Action
-    class Install < Base
-
-    private
-
-      def create_install_path
-        install_path.rmtree if install_path.exist? && destructive?
-        install_path.mkpath
-      end
-
-      def destructive?
-        environment.config_db.local['destructive'] == 'true'
-      end
-    end
-  end
-
   class ManifestSet
     include Librarian::Puppet::Util
 

data/lib/librarian/puppet/source/local.rb

@@ -94,6 +94,7 @@ module Librarian
           metadata = ::Puppet::ModuleTool::Metadata.new
           begin
             ::Puppet::ModuleTool::ModulefileReader.evaluate(metadata, modulefile)
+            raise SyntaxError, "Missing version" unless metadata.version
           rescue ArgumentError, SyntaxError => error
             warn { "Unable to parse #{modulefile}, ignoring: #{error}" }
             if metadata.respond_to? :version=

data/lib/librarian/puppet/version.rb

@@ -1,5 +1,5 @@
 module Librarian
   module Puppet
-    VERSION = "1.2.0"
+    VERSION = "1.3.0"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: librarian-puppet
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.3.0
 platform: ruby
 authors:
 - Tim Sharpe
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-01 00:00:00.000000000 Z
+date: 2014-08-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: librarian
@@ -154,6 +154,8 @@ files:
 - README.md
 - bin/librarian-puppet
 - lib/librarian/puppet.rb
+- lib/librarian/puppet/action.rb
+- lib/librarian/puppet/action/install.rb
 - lib/librarian/puppet/cli.rb
 - lib/librarian/puppet/dsl.rb
 - lib/librarian/puppet/environment.rb