reaper-man 0.1.8 → 0.1.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3fe0a8460eccd12458ba86ddb12122bd29fbc8a8084158a52d637bfe870fedcd
-  data.tar.gz: 9175cc079bbcab1456b48800d2822011677c527d49fe9c471e0e07a7f3ab7fa1
+  metadata.gz: 1cf55042184d29e41de23ee6dd8be3967d2fb98e292ae286a18abab8bc830715
+  data.tar.gz: af2fffcd7c97cfe2316d7d72a4e381b10bfae624dfc000c170cec5451d5c46e7
 SHA512:
-  metadata.gz: b2b31a39aedd71418287811778c8c0eb911dcb026b04caa5c4796b2ec8415a6a88248081e63f87b4dda2f157cd6a8ee1889b4e08084a21bd724dc69a97a76d83
-  data.tar.gz: 326a8a66aad3929c2b761bd7fde213f89263d84f55d4cc57e0c5d1b5a174d24a7733e28829cec0a702ad4b68c5abc4c1ff7dc2286760927dd41a3d7a66dd9c0b
+  metadata.gz: aea96d3039a0bcd80df0dfbed721e83bc5d5b613959dca7111d7fe713ba5ab3bae45a5a6ccb0f395c0a918de74ae42b5e4f92f63bd3e9c44fabbbe149373518a
+  data.tar.gz: b6e2eb3f10cf2add81632e4c5b8df1287c5dc89241c91ed6dfe1fe91e8402ad3a5a815e75e005d647faeb030100ff6bcceec528210e074e0b9589d98107f450b

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+# v0.1.10
+* [fix] extract rubygem specification data via proper API
+* [enhancement] clean rubygems package data to remove empty values
+
 # v0.1.8
 * [fix] only set specification value when value is set
 

data/lib/reaper-man.rb

@@ -1,13 +1,13 @@
-require 'multi_json'
+require "multi_json"
 
 module ReaperMan
-  autoload :Cli, 'reaper-man/cli'
-  autoload :Command, 'reaper-man/command'
-  autoload :Config, 'reaper-man/config'
-  autoload :Error, 'reaper-man/error'
-  autoload :Generator, 'reaper-man/generator'
-  autoload :PackageList, 'reaper-man/package_list'
-  autoload :Signer, 'reaper-man/signer'
-  autoload :Version, 'reaper-man/version'
-  autoload :Utils, 'reaper-man/utils'
+  autoload :Cli, "reaper-man/cli"
+  autoload :Command, "reaper-man/command"
+  autoload :Config, "reaper-man/config"
+  autoload :Error, "reaper-man/error"
+  autoload :Generator, "reaper-man/generator"
+  autoload :PackageList, "reaper-man/package_list"
+  autoload :Signer, "reaper-man/signer"
+  autoload :Version, "reaper-man/version"
+  autoload :Utils, "reaper-man/utils"
 end

data/lib/reaper-man/command.rb

@@ -1,11 +1,9 @@
-require 'reaper-man'
+require "reaper-man"
 
 module ReaperMan
   class Command < Bogo::Cli::Command
-
-    autoload :Repository, 'reaper-man/command/repository'
-    autoload :Package, 'reaper-man/command/package'
-    autoload :Sign, 'reaper-man/command/sign'
-
+    autoload :Repository, "reaper-man/command/repository"
+    autoload :Package, "reaper-man/command/package"
+    autoload :Sign, "reaper-man/command/sign"
   end
 end

data/lib/reaper-man/command/package.rb

@@ -1,12 +1,10 @@
-require 'reaper-man'
+require "reaper-man"
 
 module ReaperMan
   class Command
     class Package < Command
-
-      autoload :Add, 'reaper-man/command/package/add'
-      autoload :Remove, 'reaper-man/command/package/remove'
-
+      autoload :Add, "reaper-man/command/package/add"
+      autoload :Remove, "reaper-man/command/package/remove"
     end
   end
 end

data/lib/reaper-man/command/package/add.rb

@@ -1,11 +1,9 @@
-require 'reaper-man'
+require "reaper-man"
 
 module ReaperMan
   class Command
     class Package
-
       class Add < Package
-
         def execute!
           arguments.each do |path|
             run_action "Adding package to repository manifest: #{path}" do
@@ -16,9 +14,7 @@ module ReaperMan
             end
           end
         end
-
       end
-
     end
   end
 end

data/lib/reaper-man/command/package/remove.rb

@@ -1,11 +1,9 @@
-require 'reaper-man'
+require "reaper-man"
 
 module ReaperMan
   class Command
     class Package
-
       class Remove < Package
-
         def execute!
           arguments.each do |pkg|
             run_action "Remove package from repository manifest: #{pkg}" do
@@ -16,9 +14,7 @@ module ReaperMan
             end
           end
         end
-
       end
-
     end
   end
 end

data/lib/reaper-man/command/repository.rb

@@ -1,11 +1,9 @@
-require 'reaper-man'
+require "reaper-man"
 
 module ReaperMan
   class Command
     class Repository < Command
-
-      autoload :Generate, 'reaper-man/command/repository/generate'
-
+      autoload :Generate, "reaper-man/command/repository/generate"
     end
   end
 end

data/lib/reaper-man/command/repository/generate.rb

@@ -1,29 +1,25 @@
-require 'reaper-man'
+require "reaper-man"
 
 module ReaperMan
   class Command
     class Repository
-
       class Generate < Repository
-
         def execute!
-          run_action 'Generating repository' do
+          run_action "Generating repository" do
             ReaperMan::Generator.new(
               config.merge(
                 Smash.new(
                   :package_config => MultiJson.load(
                     File.read(config[:packages_file])
                   ).to_smash,
-                  :signer => config[:sign] ? Signer.new(config) : nil
+                  :signer => config[:sign] ? Signer.new(config) : nil,
                 )
               )
             ).generate!
             nil
           end
         end
-
       end
-
     end
   end
 end

data/lib/reaper-man/command/sign.rb

@@ -1,23 +1,22 @@
-require 'reaper-man'
+require "reaper-man"
 
 module ReaperMan
   class Command
     class Sign < Command
-
       def execute!
         files = arguments.map do |item|
-          if(File.file?(item))
+          if File.file?(item)
             item
           else
             File.directory?(item)
-            i_files = Dir.glob(File.join(item, '**', '*'))
+            i_files = Dir.glob(File.join(item, "**", "*"))
             i_files.delete_if do |path|
               !File.file?(path)
             end
             i_files
           end
         end.flatten.compact.uniq
-        run_action 'Signing file(s)' do
+        run_action "Signing file(s)" do
           signer = Signer.new(config)
           signer.package(*files)
           nil
@@ -27,7 +26,6 @@ module ReaperMan
           ui.puts "  #{ui.color(File.expand_path(path), :yellow)}"
         end
       end
-
     end
   end
 end

data/lib/reaper-man/error.rb

@@ -1,4 +1,4 @@
-require 'reaper-man'
+require "reaper-man"
 
 module ReaperMan
   # Standard error for reaper errors
@@ -12,9 +12,8 @@ module ReaperMan
     end
 
     # Define errors here
-    ['UnknownCommand', 'FileNotFound'].each_with_index do |klass_name, idx|
+    ["UnknownCommand", "FileNotFound"].each_with_index do |klass_name, idx|
       self.class_eval("class #{klass_name} < Error; CODE=#{idx + 1}; end;")
     end
-
   end
 end

data/lib/reaper-man/generator.rb

@@ -1,16 +1,15 @@
-require 'time'
-require 'zlib'
-require 'fileutils'
+require "time"
+require "zlib"
+require "fileutils"
 
-require 'reaper-man'
+require "reaper-man"
 
 module ReaperMan
   # Repository generator
   class Generator
-
-    autoload :Apt, 'reaper-man/generator/apt'
-    autoload :Yum, 'reaper-man/generator/yum'
-    autoload :Rubygems, 'reaper-man/generator/rubygems'
+    autoload :Apt, "reaper-man/generator/apt"
+    autoload :Yum, "reaper-man/generator/yum"
+    autoload :Rubygems, "reaper-man/generator/rubygems"
 
     include Utils::Checksum
 
@@ -29,17 +28,17 @@ module ReaperMan
     # @option args [String] :package_system apt/gem/etc...
     # @option args [Hash] :package_config
     # @option args [Signer] :signer
-    def initialize(args={})
+    def initialize(args = {})
       @package_system = args[:package_system]
       @package_config = args.fetch(:package_config, Smash.new)
       @signer = args[:signer]
       @options = args
-      extend self.class.const_get(package_system.to_s.split('_').map(&:capitalize).join.to_sym)
+      extend self.class.const_get(package_system.to_s.split("_").map(&:capitalize).join.to_sym)
     end
 
     # Generate new repository
     def generate!
-      raise NoMethodError.new 'Not implemented'
+      raise NoMethodError.new "Not implemented"
     end
 
     # Create new file
@@ -51,8 +50,8 @@ module ReaperMan
     def create_file(*name)
       path = File.join(options[:output_directory], *name)
       FileUtils.mkdir_p(File.dirname(path))
-      file = File.open(path, 'wb+')
-      if(block_given?)
+      file = File.open(path, "wb+")
+      if block_given?
         yield file
       end
       file.close unless file.closed?
@@ -68,8 +67,8 @@ module ReaperMan
     def for_file(*name)
       path = File.join(options[:output_directory], *name)
       FileUtils.mkdir_p(File.dirname(path))
-      if(block_given?)
-        file = File.open(path, 'a+')
+      if block_given?
+        file = File.open(path, "a+")
         yield file
         file.close
       end
@@ -86,13 +85,11 @@ module ReaperMan
       base_file = File.open(for_file(path))
       create_file(compressed_path) do |file|
         compressor = Zlib::GzipWriter.new(file)
-        while(data = base_file.read(2048))
+        while data = base_file.read(2048)
           compressor.write(data)
         end
         compressor.close
       end
     end
-
   end
-
 end

data/lib/reaper-man/generator/apt.rb

@@ -1,4 +1,4 @@
-require 'reaper-man'
+require "reaper-man"
 
 module ReaperMan
   class Generator
@@ -21,8 +21,8 @@ module ReaperMan
               arches.each do |arch_name, packages|
                 package_file(origin_name, dist_name, component_name, arch_name, packages)
                 release_headers = Smash.new
-                release_headers['Label'] = dist_args['label']
-                release_headers['Archive'] = dist_name
+                release_headers["Label"] = dist_args["label"]
+                release_headers["Archive"] = dist_name
                 sign_file_if_setup do
                   release_file(origin_name, dist_name, component_name, arch_name, release_headers)
                 end
@@ -30,19 +30,19 @@ module ReaperMan
             end
             release_headers = Smash[
               %w(Codename Suite Label Description Version).map do |field_name|
-                if(val = dist_args[field_name.downcase])
+                if val = dist_args[field_name.downcase]
                   [field_name, val]
                 end
               end.compact
             ]
-            release_headers['Components'] = dist_args[:components].keys.join(' ')
-            signed = sign_file_if_setup('--clearsign') do
+            release_headers["Components"] = dist_args[:components].keys.join(" ")
+            signed = sign_file_if_setup("--clearsign") do
               release_file(origin_name, dist_name, release_headers)
             end
-            if(File.exists?("#{signed}.gpg"))
+            if File.exists?("#{signed}.gpg")
               FileUtils.mv(
                 "#{signed}.gpg",
-                File.join(File.dirname(signed), 'InRelease')
+                File.join(File.dirname(signed), "InRelease")
               )
             end
             sign_file_if_setup do
@@ -57,9 +57,9 @@ module ReaperMan
       #
       # @yield block returning file path
       # @return [String] file path
-      def sign_file_if_setup(opts=nil)
+      def sign_file_if_setup(opts = nil)
         path = yield
-        if(signer && options[:sign])
+        if signer && options[:sign]
           signer.file(path, nil, opts)
         end
         path
@@ -71,12 +71,12 @@ module ReaperMan
       # @return [String] path to compressed Packages file
       def package_file(*args)
         pkgs = args.pop
-        args.insert(1, 'dists')
-        create_file(*args.push('Packages')) do |file|
+        args.insert(1, "dists")
+        create_file(*args.push("Packages")) do |file|
           pkgs.each do |pkg_name, pkgs|
             pkgs.each do |pkg_version, pkg_meta|
               pkg_meta.each do |field_name, field_value|
-                if(field_value)
+                if field_value
                   file.puts "#{field_name}: #{field_value}"
                 end
               end
@@ -92,35 +92,34 @@ module ReaperMan
       # @param args [String] argument list for file path
       # @return [TrueClass]
       def release_file(*args)
-        header = args.detect{|a| a.is_a?(Hash)}
+        header = args.detect { |a| a.is_a?(Hash) }
         header ? args.delete(header) : header = Smash.new
         header.merge(Smash[%w(Origin Codename Component Architecture).zip(args)])
-        header['Date'] = Time.now.utc.strftime('%a, %d %b %Y %H:%M:%S %Z')
-        args.insert(1, 'dists')
-        create_file(*args.dup.push('Release')) do |file|
-          contents = Dir.glob(File.join(File.dirname(file.path), '**', '*'))
+        header["Date"] = Time.now.utc.strftime("%a, %d %b %Y %H:%M:%S %Z")
+        args.insert(1, "dists")
+        create_file(*args.dup.push("Release")) do |file|
+          contents = Dir.glob(File.join(File.dirname(file.path), "**", "*"))
           header_content = header.map do |key, value|
             next unless value
             "#{key.to_s.capitalize}: #{value}"
           end.compact.join("\n")
           file.puts header_content
-          [['MD5Sum', :md5], ['SHA1', :sha1], ['SHA256', :sha256]].each do |field, digest|
+          [["MD5Sum", :md5], ["SHA1", :sha1], ["SHA256", :sha256]].each do |field, digest|
             file.puts "#{field}:"
             contents.each do |content|
               next if File.expand_path(content) == File.expand_path(file.path) || File.directory?(content)
-              File.open(content, 'r') do |content_file|
-                line = [' ']
+              File.open(content, "r") do |content_file|
+                line = [" "]
                 line << checksum(content_file, digest)
                 line << content_file.size
-                line << content_file.path.sub(File.dirname(file.path), '').sub(/^\//, '')
-                file.puts line.join(' ')
+                line << content_file.path.sub(File.dirname(file.path), "").sub(/^\//, "")
+                file.puts line.join(" ")
               end
             end
           end
           true
         end
       end
-
     end
   end
 end

data/lib/reaper-man/generator/rubygems.rb

@@ -1,4 +1,4 @@
-require 'reaper-man'
+require "reaper-man"
 
 module ReaperMan
   class Generator
@@ -18,9 +18,9 @@ module ReaperMan
       end
 
       def generate_indexing(gems)
-        build_spec_file('specs', gems.fetch(:release, {}))
-        build_spec_file('latest_specs', gems.fetch(:release, {}))
-        build_spec_file('prerelease', gems.fetch(:prerelease, {}))
+        build_spec_file("specs", gems.fetch(:release, {}))
+        build_spec_file("latest_specs", gems.fetch(:release, {}))
+        build_spec_file("prerelease", gems.fetch(:prerelease, {}))
       end
 
       def create_index(gems)
@@ -34,11 +34,11 @@ module ReaperMan
       end
 
       def marshal_path
-        ['Marshal', marshal_version].join('.')
+        ["Marshal", marshal_version].join(".")
       end
 
       def marshal_version
-        [Marshal::MAJOR_VERSION, Marshal::MINOR_VERSION].join('.')
+        [Marshal::MAJOR_VERSION, Marshal::MINOR_VERSION].join(".")
       end
 
       def build_spec_file(name, gems)
@@ -54,7 +54,7 @@ module ReaperMan
           list.each do |version, info|
             spec = Gem::Specification.new(name)
             info.each do |var, value|
-              if(value && spec.respond_to?("#{var}="))
+              if value && spec.respond_to?("#{var}=")
                 begin
                   # Ensure we convert Smash instances
                   value = value.to_hash if value.is_a?(Hash)
@@ -70,7 +70,7 @@ module ReaperMan
               spec.add_dependency(*dep)
             end
             deflator = Zlib::Deflate.new
-            create_file('quick', marshal_path, "#{name}-#{version}.gemspec.rz") do |file|
+            create_file("quick", marshal_path, "#{name}-#{version}.gemspec.rz") do |file|
               file.write(deflator.deflate(Marshal.dump(spec), Zlib::SYNC_FLUSH))
               file.write(deflator.finish)
             end
@@ -84,12 +84,11 @@ module ReaperMan
       # @return [String] file path
       def sign_file_if_setup
         path = yield
-        if(signer && options[:sign])
+        if signer && options[:sign]
           signer.file(path)
         end
         path
       end
-
     end
   end
 end