librarian-puppet 1.0.5 → 1.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 00ea6603b1c515215bb1f55bccc63bb1cc9a43fa
-  data.tar.gz: 369d915e3047152dac00490e1a80234bd80e1f61
+  metadata.gz: 6f2563b5575e2082f80e0b75f8b1ece49f070a8a
+  data.tar.gz: c37295903ceab4b3406d28f3f90537cd2efa20a1
 SHA512:
-  metadata.gz: 34438184c66b52a657452b3980715830e66afa94f26329785ac4d6de0e0de6f293b8e42c054264569872f56874aeb02a865d3652196049e4e716d96c10cb8d36
-  data.tar.gz: c06ea15a7cd05b84193ea63703c2c7ee56cb52da1b5fa8816ccda61dfc45a1842ced0806a1f864cf805bc8a93a8b06e8157e3f588eb8cffb6785330e285e3473
+  metadata.gz: 7c9a691b88c4813bc4dc9d90011bc194f8ccf3452ca21eb79359a95652294d049b238b955b3d6fd9fc0f13eebf44ac9e698d60621e683a29675675a20b11c9e6
+  data.tar.gz: 28e9ee98ba396f257dbf233550bc5983c335fe1d85da95e1d5d3621770ef904925b86654c833253dafd446f41a98ebd54e1b5f8457d1e135b7a775e4b2ae9a3d

data/README.md

@@ -9,7 +9,7 @@ librarian-puppet to manage the puppet modules your infrastructure depends on,
 whether the modules come from the [Puppet Forge](https://forge.puppetlabs.com/),
 Git repositories or a just a path.
 
-* Librarian-puppet can reuse the dependencies listed in your Modulefile
+* Librarian-puppet can reuse the dependencies listed in your `Modulefile` or `metadata.json`
 * Forge modules can be installed from [Puppetlabs Forge](https://forge.puppetlabs.com/) or an internal Forge such as [Pulp](http://www.pulpproject.org/)
 * Git modules can be installed from a branch, tag or specific commit, optionally using a path inside the repository
 * Modules can be installed from GitHub using tarballs, without needing Git installed
@@ -70,7 +70,7 @@ This Puppetfile will download all the dependencies listed in your Modulefile fro
 ### Recursive module dependency resolution
 
 When fetching a module all dependencies specified in its
-`Modulefile` and `Puppetfile` will be resolved and installed.
+`Modulefile`, `metadata.json` and `Puppetfile` will be resolved and installed.
 
 ### Puppetfile Breakdown
 

data/lib/librarian/puppet/dsl.rb

@@ -45,6 +45,14 @@ module Librarian
             regexp =~ line && mod($2, $4)
           end
         end
+
+        # implement the 'metadata' syntax for Puppetfile
+        def metadata
+          dependencyList = JSON.parse(File.read(Pathname.new(specfile).parent.join('metadata.json')))['dependencies']
+          dependencyList.each do |d|
+            mod(d['name'], d['version_requirement'])
+          end
+        end
       end
     end
   end

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

@@ -121,7 +121,7 @@ module Librarian
         end
 
         def install_path(name)
-          environment.install_path.join(name.split('-').last)
+          environment.install_path.join(organization_name(name))
         end
 
         def fetch_version(name, version_uri)

data/lib/librarian/puppet/source/forge/repo.rb

@@ -59,7 +59,7 @@ module Librarian
               install_path.rmtree
             end
 
-            unpacked_path = version_unpacked_cache_path(version).join(name.split('-').last)
+            unpacked_path = version_unpacked_cache_path(version).join(organization_name(name))
 
             unless unpacked_path.exist?
               raise Error, "#{unpacked_path} does not exist, something went wrong. Try removing it manually"

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

@@ -98,7 +98,7 @@ module Librarian
         end
 
         def install_path(name)
-          environment.install_path.join(name.split('-').last)
+          environment.install_path.join(organization_name(name))
         end
 
         def fetch_version(name, version_uri)

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

@@ -1,16 +1,5 @@
 require 'librarian/puppet/util'
 
-begin
-  require 'puppet'
-  require 'puppet/module_tool'
-rescue LoadError
-  $stderr.puts <<-EOF
-Unable to load puppet, the puppet gem is required for :git and :path source.
-Install it with: gem install puppet
-  EOF
-  exit 1
-end
-
 module Librarian
   module Puppet
     module Source
@@ -30,7 +19,7 @@ module Librarian
             warn { "Invalid module name '#{name}', you should qualify it with 'ORGANIZATION-#{name}' for resolution to work correctly" }
           end
 
-          install_path = environment.install_path.join(name.split('-').last)
+          install_path = environment.install_path.join(organization_name(name))
           if install_path.exist?
             debug { "Deleting #{relative_path_to(install_path)}" }
             install_path.rmtree
@@ -48,13 +37,21 @@ module Librarian
         def fetch_dependencies(name, version, extra)
           dependencies = Set.new
 
-          if modulefile?
-            evaluate_modulefile(modulefile).dependencies.each do |dependency|
-              dependency_name = dependency.instance_variable_get(:@full_module_name)
-              version = dependency.instance_variable_get(:@version_requirement)
-              gem_requirement = Requirement.new(version).gem_requirement
-              dependencies << Dependency.new(dependency_name, gem_requirement, forge_source)
+          dependencyList = if metadata?
+            JSON.parse(File.read(metadata))['dependencies']
+          elsif modulefile?
+            evaluate_modulefile(modulefile).dependencies.map do |dependency|
+              {
+                'name' => dependency.instance_variable_get(:@full_module_name),
+                'version_requirement' => dependency.instance_variable_get(:@version_requirement)
+              }
             end
+          else []
+          end
+
+          dependencyList.each do |d|
+            gem_requirement = Requirement.new(d['version_requirement']).gem_requirement
+            dependencies << Dependency.new(d['name'], gem_requirement, forge_source)
           end
 
           if specfile?
@@ -77,7 +74,23 @@ module Librarian
           evaluate_modulefile(modulefile).version
         end
 
+        def require_puppet
+          begin
+            require 'puppet'
+            require 'puppet/module_tool'
+          rescue LoadError
+            $stderr.puts <<-EOF
+          Unable to load puppet, the puppet gem is required for :git and :path source.
+          Install it with: gem install puppet
+            EOF
+            exit 1
+          end
+          true
+        end
+
         def evaluate_modulefile(modulefile)
+          @@require_puppet ||= require_puppet
+
           metadata = ::Puppet::ModuleTool::Metadata.new
           begin
             ::Puppet::ModuleTool::ModulefileReader.evaluate(metadata, modulefile)
@@ -100,6 +113,14 @@ module Librarian
           File.exists?(modulefile)
         end
 
+        def metadata
+          File.join(filesystem_path, 'metadata.json')
+        end
+
+        def metadata?
+          File.exists?(metadata)
+        end
+
         def specfile
           File.join(filesystem_path, environment.specfile_name)
         end

data/lib/librarian/puppet/templates/Puppetfile

@@ -3,8 +3,11 @@
 
 forge "https://forgeapi.puppetlabs.com"
 
+# use dependencies defined in metadata.json
+metadata
+
 # use dependencies defined in Modulefile
-modulefile
+# modulefile
 
 # A module from the Puppet Forge
 # mod 'puppetlabs-stdlib'

data/lib/librarian/puppet/util.rb

@@ -38,6 +38,11 @@ module Librarian
       def normalize_name(name)
         name.sub('/','-')
       end
+
+      # get the organization name from organization-module
+      def organization_name(name)
+        name.split('-',2).last
+      end
     end
   end
 end

data/lib/librarian/puppet/version.rb

@@ -1,5 +1,5 @@
 module Librarian
   module Puppet
-    VERSION = "1.0.5"
+    VERSION = "1.0.6"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: librarian-puppet
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 1.0.6
 platform: ruby
 authors:
 - Tim Sharpe
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-22 00:00:00.000000000 Z
+date: 2014-08-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: librarian