autobuild 1.17.0 → 1.18.0

data/lib/autobuild/test.rb

@@ -44,7 +44,7 @@ def setup
             @temp_dirs = Array.new
 
             @tempdir = make_tmpdir
-            FileUtils.mkdir_p(@tempdir, :mode => 0700)
+            FileUtils.mkdir_p(@tempdir, mode: 0o700)
             Autobuild.logdir = "#{tempdir}/log"
             FileUtils.mkdir_p Autobuild.logdir
             Autobuild.silent = true
@@ -75,20 +75,21 @@ def data_dir
         attr_reader :tempdir
 
         def build_config(bind, template)
-            eval "basedir = '#{self.tempdir}'", bind
-            ryml = File.open(File.join(data_dir, "#{template}.ryml")) { |f| f.readlines }.join('')
+            bind.local_variable_set(:basedir, tempdir.to_s)
+            ryml = File.open(File.join(data_dir, "#{template}.ryml"), &:readlines)
+                .join('')
             result = ERB.new(ryml).result(bind)
 
             yml = File.join(tempdir, "#{template}.yml")
             File.open(yml, 'w+') { |f| f.write(result) }
-            
-            return yml
+
+            yml
         end
 
         def untar(file)
             file = File.expand_path(file, data_dir)
-            dir = self.tempdir
-            Dir.chdir(dir) do 
+            dir = tempdir
+            Dir.chdir(dir) do
                 system("tar xf #{file}")
             end
 

data/lib/autobuild/test_utility.rb

@@ -29,16 +29,14 @@ def coverage_enabled?
         end
 
         def coverage_available?
-            !!@coverage_source_dir
+            @coverage_source_dir
         end
 
         # Controls whether code coverage should be measured
         #
         # @param [Boolean,nil] flag enable or disable code coverage. If set to
         #   nil, will use the default from {TestUtility.coverage?}
-        def coverage_enabled=(flag)
-            @coverage_enabled = flag
-        end
+        attr_writer :coverage_enabled
 
         # Where the code coverage will be generated
         #
@@ -85,21 +83,24 @@ def install
             if !coverage_enabled?
                 return
             elsif !coverage_available?
-                package.warn "%s: #coverage_source_dir not set on #test_utility, skipping installation of the code coverage results"
+                package.warn "%s: #coverage_source_dir not set on #test_utility, "\
+                    "skipping installation of the code coverage results"
             end
 
             coverage_target_dir  = self.coverage_target_dir
             coverage_source_dir  = self.coverage_source_dir
             if "#{coverage_source_dir}/".start_with?("#{source_dir}/")
-                raise ArgumentError, "#coverage_source_dir cannot be a subdirectory of #source_dir in #{package.name}"
+                raise ArgumentError, "#coverage_source_dir cannot be a subdirectory "\
+                    "of #source_dir in #{package.name}"
             elsif target_dir == coverage_target_dir
-                raise ArgumentError, "#coverage_target_dir cannot be the same than of #target_dir in #{package.name}"
+                raise ArgumentError, "#coverage_target_dir cannot be the same than of "\
+                    "#target_dir in #{package.name}"
             end
 
             FileUtils.mkdir_p File.dirname(coverage_target_dir)
             FileUtils.cp_r coverage_source_dir, coverage_target_dir
-            package.message "%s: copied test coverage results for #{package.name} from #{coverage_source_dir} to #{coverage_target_dir}"
+            package.message "%s: copied test coverage results for #{package.name} from "\
+                "#{coverage_source_dir} to #{coverage_target_dir}"
         end
     end
 end
-

data/lib/autobuild/timestamps.rb

@@ -1,5 +1,5 @@
 module Autobuild
-    STAMPFILE = "autobuild-stamp"
+    STAMPFILE = "autobuild-stamp".freeze
 
     class << self
         # The set of global ignores for SourceTreeTask
@@ -15,17 +15,18 @@ class << self
     # The matching paths will not be considered when looking if a source tree
     # has been updated or not.
     def self.ignore(path)
-        if path.kind_of?(Regexp)
-            ignored_files << path
-        else
-            ignored_files << Regexp.new("^#{Regexp.quote(path)}")
-        end
+        ignored_files <<
+            if path.kind_of?(Regexp)
+                path
+            else
+                Regexp.new("^#{Regexp.quote(path)}")
+            end
     end
 
     def self.tree_timestamp(path, *exclude)
         # Exclude autobuild timestamps
-        exclude << (/#{Regexp.quote(STAMPFILE)}$/)
-        exclude << (/\.autobuild-patches$/)
+        exclude << /#{Regexp.quote(STAMPFILE)}$/
+        exclude << /\.autobuild-patches$/
 
         Autobuild.message "getting tree timestamp for #{path}" if Autobuild.debug
         latest = Time.at(0)
@@ -33,13 +34,15 @@ def self.tree_timestamp(path, *exclude)
 
         Find.find(path) do |p|
             Find.prune if File.basename(p) =~ /^\./
-            exclude.each do |pattern| 
+            exclude.each do |pattern|
                 if pattern === p
-                    Autobuild.message "  excluding #{p} because of #{pattern}" if Autobuild.debug
+                    if Autobuild.debug
+                        Autobuild.message "  excluding #{p} because of #{pattern}"
+                    end
                     Find.prune
                 end
             end
-            next if !File.file?(p)
+            next unless File.file?(p)
 
             p_time = File.mtime(p)
             if latest < p_time
@@ -49,7 +52,7 @@ def self.tree_timestamp(path, *exclude)
         end
 
         Autobuild.message "  newest file: #{latest_file} at #{latest}" if Autobuild.debug
-        return latest_file, latest
+        [latest_file, latest]
     end
 
     class SourceTreeTask < Rake::Task
@@ -62,26 +65,28 @@ def initialize(*args, &block)
             @exclude = Autobuild.ignored_files.dup
             super
         end
-            
+
         def timestamp
-            if @newest_time
-                return @newest_time
-            end
+            return @newest_time if @newest_time
 
             @newest_file, @newest_time =
-                Autobuild.tree_timestamp(name, %r#(?:^|/)(?:CVS|_darcs|\.svn)$#, *@exclude)
+                Autobuild.tree_timestamp(name,
+                    %r{(?:^|/)(?:CVS|_darcs|\.svn)$}, *@exclude)
             @newest_time
         end
     end
     def self.source_tree(path, &block)
         task = SourceTreeTask.define_task(path)
-        block.call(task) unless !block
+        block&.call(task)
         task
     end
-            
+
     def self.get_stamp(stampfile)
-        return Time.at(0) if !File.exist?(stampfile)
-        return File.mtime(stampfile)
+        if File.exist?(stampfile)
+            File.mtime(stampfile)
+        else
+            Time.at(0)
+        end
     end
 
     def self.hires_modification_time?
@@ -103,9 +108,10 @@ def self.touch_stamp(stampfile)
         elsif !File.exist?(dir)
             FileUtils.mkdir_p dir
         end
+
         FileUtils.touch(stampfile)
 
-        if !hires_modification_time?
+        unless hires_modification_time?
             # File modification times on most Unix filesystems have a granularity of
             # one second, so we (unfortunately) have to sleep 1s to make sure that
             # time comparisons will work as expected.
@@ -113,4 +119,3 @@ def self.touch_stamp(stampfile)
         end
     end
 end
-

data/lib/autobuild/tools.rb

@@ -2,7 +2,7 @@ module Autobuild
     class << self
         # Configure the programs used by different packages
         attr_reader :programs
-        # A cache of entries in programs to their resolved full path 
+        # A cache of entries in programs to their resolved full path
         #
         # @return [{String=>[String,String,String]}] the triplet (full path,
         #   tool name, value of ENV['PATH']). The last two values are used to
@@ -13,7 +13,7 @@ class << self
 
         # Get a given program, using its name as default value. For
         # instance
-        #   tool('automake') 
+        #   tool('automake')
         # will return 'automake' unless the autobuild script defined
         # another automake program in Autobuild.programs by doing
         #   Autobuild.programs['automake'] = 'automake1.9'
@@ -30,29 +30,30 @@ def find_in_path(file, envvar = 'PATH')
         def tool_in_path(name, env: self.env)
             path, path_name, path_env = programs_in_path[name]
             current = tool(name)
-            env_PATH = env.resolved_env['PATH']
-            if (path_env != env_PATH) || (path_name != current)
+            env_path = env.resolved_env['PATH']
+            if (path_env != env_path) || (path_name != current)
                 # Delete the current entry given that it is invalid
                 programs_in_path.delete(name)
-                if current[0, 1] == "/"
-                    # This is already a full path
-                    path = current
-                else
-                    path = env.find_executable_in_path(current)
+                path =
+                    if current[0, 1] == "/"
+                        # This is already a full path
+                        current
+                    else
+                        env.find_executable_in_path(current)
+                    end
+
+                unless path
+                    raise ArgumentError, "tool #{name}, set to #{current}, "\
+                        "can not be found in PATH=#{env_path}"
                 end
 
-                if !path
-                    raise ArgumentError, "tool #{name}, set to #{current}, can not be found in PATH=#{env_PATH}"
-                end
-
-                programs_in_path[name] = [path, current, env_PATH]
+                programs_in_path[name] = [path, current, env_path]
             end
 
-            return path
+            path
         end
     end
 
     @programs = Hash.new
     @programs_in_path = Hash.new
 end
-

data/lib/autobuild/utility.rb

@@ -28,7 +28,7 @@ def initialize(name, package, install_on_error: false)
             @source_ref_dir = nil
             @source_dir = nil
             @target_dir = nil
-            @install_on_error = !!install_on_error
+            @install_on_error = install_on_error
         end
 
         # Directory in which the utility will generate some files The
@@ -43,9 +43,7 @@ def initialize(name, package, install_on_error: false)
         # Absolute path to where this utulity should output its results. Returns nil if
         # {source_dir} has not been set.
         def source_dir
-            if @source_dir
-                File.expand_path(@source_dir, source_ref_dir || package.srcdir)
-            end
+            File.expand_path(@source_dir, source_ref_dir || package.srcdir) if @source_dir
         end
 
         # Directory in which the utility would install some files.
@@ -62,7 +60,9 @@ def source_dir
         # @return [String,nil]
         def target_dir
             if @target_dir
-                File.expand_path(@target_dir, File.expand_path(Autobuild.send("#{name}_prefix") || name,  package.prefix))
+                utility_prefix = Autobuild.send("#{name}_prefix") || name
+                File.expand_path(@target_dir,
+                    File.expand_path(utility_prefix, package.prefix))
             else
                 File.join(package.logdir, "#{name}-results", package.name)
             end
@@ -79,6 +79,7 @@ def target_dir
         # @return [Rake::Task]
         def task(&block)
             return if @task
+
             @task = package.task task_name do
                 # This flag allows to disable this utility's task
                 # once {task} has been called
@@ -92,7 +93,7 @@ def task(&block)
                 end
             end
 
-            package.task name => task_name 
+            package.task name => task_name
             @task
         end
 
@@ -101,16 +102,11 @@ def call_task_block
 
             # Allow the user to install manually in the task
             # block
-            if !@installed && target_dir
-                install
-            end
-
+            install if !@installed && target_dir
         rescue Interrupt
             raise
         rescue ::Exception => e
-            if install_on_error? && !@installed && target_dir
-                install
-            end
+            install if install_on_error? && !@installed && target_dir
 
             if Autobuild.send("pass_#{name}_errors")
                 raise
@@ -154,8 +150,10 @@ def enabled?
         attr_writer :enabled
 
         def install
-            if !File.directory?(source_dir)
-                raise "#{source_dir} was expected to be a directory, but it is not. Check the package's #{name} generation. The generated #{name} products should be in #{source_dir}"
+            unless File.directory?(source_dir)
+                raise "#{source_dir} was expected to be a directory, but it is not. "\
+                    "Check the package's #{name} generation. "\
+                    "The generated #{name} products should be in #{source_dir}"
             end
 
             target_dir  = self.target_dir
@@ -163,7 +161,8 @@ def install
             FileUtils.rm_rf   target_dir
             FileUtils.mkdir_p File.dirname(target_dir)
             FileUtils.cp_r    source_dir, target_dir
-            Autoproj.message "  copied #{name} results for #{package.name} from #{source_dir} to #{target_dir}"
+            Autoproj.message "  copied #{name} results for #{package.name} "\
+                "from #{source_dir} to #{target_dir}"
 
             @installed = true
         end
@@ -188,8 +187,7 @@ def task_name
         #
         # @return [Boolean]
         def has_task?
-            !!Rake.application.lookup(task_name)
+            Rake.application.lookup(task_name)
         end
     end
 end
-

data/lib/autobuild/version.rb

@@ -1,3 +1,3 @@
 module Autobuild
-    VERSION = "1.17.0" unless defined? Autobuild::VERSION
+    VERSION = "1.18.0".freeze unless defined? Autobuild::VERSION
 end

metadata

@@ -1,83 +1,83 @@
 --- !ruby/object:Gem::Specification
 name: autobuild
 version: !ruby/object:Gem::Version
-  version: 1.17.0
+  version: 1.18.0
 platform: ruby
 authors:
 - Sylvain Joyeux
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-02-18 00:00:00.000000000 Z
+date: 2019-03-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: pastel
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.0'
+        version: 0.7.0
     - - ">="
       - !ruby/object:Gem::Version
-        version: 12.3.0
+        version: 0.7.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.0'
+        version: 0.7.0
     - - ">="
       - !ruby/object:Gem::Version
-        version: 12.3.0
+        version: 0.7.0
 - !ruby/object:Gem::Dependency
-  name: utilrb
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '12.0'
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: 12.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '12.0'
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: 12.3.0
 - !ruby/object:Gem::Dependency
-  name: tty-prompt
+  name: tty-cursor
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.15.0
+        version: 0.5.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.15.0
+        version: 0.5.0
 - !ruby/object:Gem::Dependency
-  name: tty-cursor
+  name: tty-prompt
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: 0.15.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: 0.15.0
 - !ruby/object:Gem::Dependency
   name: tty-screen
   requirement: !ruby/object:Gem::Requirement
@@ -93,25 +93,39 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.6.4
 - !ruby/object:Gem::Dependency
-  name: pastel
+  name: utilrb
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: '3.0'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: '3.0'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: fakefs
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: flexmock
   requirement: !ruby/object:Gem::Requirement
@@ -152,20 +166,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
-- !ruby/object:Gem::Dependency
-  name: fakefs
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -191,6 +191,7 @@ extra_rdoc_files: []
 files:
 - ".gitattributes"
 - ".gitignore"
+- ".rubocop.yml"
 - ".travis.yml"
 - Changes.txt
 - Gemfile
@@ -252,7 +253,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 1.9.3
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="