autoproj 2.10.1 → 2.13.0

data/lib/autoproj/ops/install.rb

@@ -1,7 +1,10 @@
+# frozen_string_literal: true
+
 require 'pathname'
 require 'optparse'
 require 'fileutils'
 require 'yaml'
+require 'English'
 
 module Autoproj
     module Ops
@@ -13,6 +16,36 @@ module Ops
         class Install
             class UnexpectedBinstub < RuntimeError; end
 
+            RUBYLIB_REINIT = <<~RUBY
+                if defined?(Bundler)
+                    if Bundler.respond_to?(:with_unbundled_env)
+                        Bundler.with_unbundled_env do
+                            exec(Hash['RUBYLIB' => nil], $0, *ARGV)
+                        end
+                    else
+                        Bundler.with_clean_env do
+                            exec(Hash['RUBYLIB' => nil], $0, *ARGV)
+                        end
+                    end
+                elsif ENV['RUBYLIB']
+                    exec(Hash['RUBYLIB' => nil], $0, *ARGV)
+                end
+            RUBY
+
+            WITHOUT_BUNDLER = <<~RUBY
+                if defined?(Bundler)
+                    if Bundler.respond_to?(:with_unbundled_env)
+                        Bundler.with_unbundled_env do
+                            exec($0, *ARGV)
+                        end
+                    else
+                        Bundler.with_clean_env do
+                            exec($0, *ARGV)
+                        end
+                    end
+                end
+            RUBY
+
             # The created workspace's root directory
             attr_reader :root_dir
             # Content of the Gemfile generated to install autoproj itself
@@ -220,6 +253,10 @@ def default_gemfile_contents(autoproj_version = ">= 2.0.0")
                  "gem \"utilrb\", \">= 3.0.1\""].join("\n")
             end
 
+            def add_seed_config(path)
+                @config.merge!(YAML.safe_load(File.read(path), [Symbol]))
+            end
+
             # Parse the provided command line options and returns the non-options
             def parse_options(args = ARGV)
                 options = OptionParser.new do |opt|
@@ -232,54 +269,63 @@ def parse_options(args = ARGV)
                     opt.on '--debug', 'Run in debug mode' do
                         @autoproj_options << '--debug'
                     end
-                    opt.on '--gem-source=URL', String, "use this source for RubyGems "\
-                        "instead of rubygems.org" do |url|
+                    opt.on '--gem-source=URL', String, 'use this source for RubyGems '\
+                        'instead of rubygems.org' do |url|
                         @gem_source = url
                     end
-                    opt.on '--gems-path=PATH', "install gems under this path instead "\
-                        "of ~/.autoproj/gems" do |path|
+                    opt.on '--gems-path=PATH', 'install gems under this path instead '\
+                        'of ~/.autoproj/gems' do |path|
                         self.gems_install_path     = path
                     end
                     opt.on '--public-gems', "install gems in the default gem location" do
                         self.install_gems_in_gem_user_dir
                     end
-                    opt.on '--version=VERSION_CONSTRAINT', String, 'use the provided "\
-                        "string as a version constraint for autoproj' do |version|
+                    opt.on '--bundler-version=VERSION_CONSTRAINT', String, 'use the provided '\
+                        'string as a version constraint for bundler' do |version|
+                        @config['bundler_version'] = version
+                    end
+                    opt.on '--version=VERSION_CONSTRAINT', String, 'use the provided '\
+                        'string as a version constraint for autoproj' do |version|
                         if @gemfile
                             raise "cannot give both --version and --gemfile"
                         end
                         @gemfile = default_gemfile_contents(version)
                     end
-                    opt.on '--gemfile=PATH', String, 'use the given Gemfile to install "\
-                        "autoproj instead of the default' do |path|
+                    opt.on '--gemfile=PATH', String, 'use the given Gemfile to install '\
+                        'autoproj instead of the default' do |path|
                         if @gemfile
                             raise "cannot give both --version and --gemfile"
                         end
                         @gemfile = File.read(path)
                     end
-                    opt.on '--seed-config=PATH', String, 'path to a seed file that "\
-                        "should be used to initialize the configuration' do |path|
-                        @config.merge!(YAML.load(File.read(path)))
+                    opt.on '--no-seed-config',
+                           'when reinstalling an existing autoproj workspace, do not '\
+                           'use the config in .autoproj/ as seed' do
+                        @config.clear
+                    end
+                    opt.on '--seed-config=PATH', String, 'path to a seed file that '\
+                        'should be used to initialize the configuration' do |path|
+                        add_seed_config(path)
                     end
-                    opt.on '--prefer-os-independent-packages', 'prefer OS-independent "\
-                        "packages (such as a RubyGem) over their OS-packaged equivalent "\
-                        "(e.g. the thor gem vs. the ruby-thor debian package)' do
+                    opt.on '--prefer-os-independent-packages', 'prefer OS-independent '\
+                        'packages (such as a RubyGem) over their OS-packaged equivalent '\
+                        '(e.g. the thor gem vs. the ruby-thor debian package)' do
                         @prefer_indep_over_os_packages = true
                     end
-                    opt.on '--[no-]color', 'do not use colored output (enabled by "\
-                        "default if the terminal supports it)' do |color|
+                    opt.on '--[no-]color', 'do not use colored output (enabled by '\
+                        'default if the terminal supports it)' do |color|
                         if color then @autoproj_options << "--color"
                         else @autoproj_options << '--no-color'
                         end
                     end
-                    opt.on '--[no-]progress', 'do not use progress output (enabled by "\
-                        "default if the terminal supports it)' do |progress|
+                    opt.on '--[no-]progress', 'do not use progress output (enabled by '\
+                        'default if the terminal supports it)' do |progress|
                         if progress then @autoproj_options << "--progress"
                         else @autoproj_options << '--no-progress'
                         end
                     end
-                    opt.on '--[no-]interactive', 'if non-interactive, use default "\
-                           "answer for questions' do |flag|
+                    opt.on '--[no-]interactive', 'if non-interactive, use default '\
+                           'answer for questions' do |flag|
                         if flag then @autoproj_options << "--interactive"
                         else @autoproj_options << "--no-interactive"
                         end
@@ -289,22 +335,50 @@ def parse_options(args = ARGV)
                 @autoproj_options + args
             end
 
-            def find_bundler(gem_program)
-                setup_paths =
-                    IO.popen([env_for_child, Gem.ruby, gem_program, 'which','-a', 'bundler/setup']) do |io|
-                        io.read
-                    end
-                return unless $?.success?
+            def bundler_version
+                @config['bundler_version']
+            end
+
+            def find_bundler(gem_program, version: nil)
                 bundler_path = File.join(gems_gem_home, 'bin', 'bundle')
-                setup_paths.each_line do |setup_path|
-                    if File.exist?(bundler_path) && setup_path.start_with?(gems_gem_home)
-                        return bundler_path
+                return unless File.exist?(bundler_path)
+
+                setup_paths =
+                    if version
+                        find_versioned_bundler_setup(gem_program, version)
+                    else
+                        find_unversioned_bundler_setup(gem_program)
                     end
+
+                setup_paths.each do |setup_path|
+                    return bundler_path if setup_path.start_with?(gems_gem_home)
                 end
-                return
+                nil
+            end
+
+            def find_versioned_bundler_setup(gem_program, version)
+                contents = IO.popen(
+                    [env_for_child, Gem.ruby, gem_program,
+                     'contents', '-v', version, 'bundler'],
+                    &:readlines
+                )
+                return [] unless $CHILD_STATUS.success?
+
+                contents.grep(%r{bundler/setup.rb$})
+            end
+
+            def find_unversioned_bundler_setup(gem_program)
+                setup_paths = IO.popen(
+                    [env_for_child, Gem.ruby, gem_program,
+                     'which', '-a', 'bundler/setup'],
+                    &:readlines
+                )
+                return [] unless $CHILD_STATUS.success?
+
+                setup_paths
             end
 
-            def install_bundler(gem_program, silent: false)
+            def install_bundler(gem_program, version: nil, silent: false)
                 local = ['--local'] if local?
 
                 redirection = Hash.new
@@ -312,6 +386,9 @@ def install_bundler(gem_program, silent: false)
                     redirection = Hash[out: :close]
                 end
 
+                version_args = []
+                version_args << '-v' << version if version
+
                 # Shut up the bundler warning about 'bin' not being in PATH
                 env = self.env
                 env['PATH'] += [File.join(gems_gem_home, 'bin')]
@@ -322,14 +399,14 @@ def install_bundler(gem_program, silent: false)
                         '--clear-sources', '--source', gem_source,
                         '--no-user-install', '--install-dir', gems_gem_home,
                         *local, "--bindir=#{File.join(gems_gem_home, 'bin')}",
-                        'bundler', **redirection)
+                        'bundler', *version_args, **redirection)
 
                 if !result
                     STDERR.puts "FATAL: failed to install bundler in #{gems_gem_home}"
                     nil
                 end
 
-                if (bundler_path = find_bundler(gem_program))
+                if (bundler_path = find_bundler(gem_program, version: version))
                     bundler_path
                 else
                     STDERR.puts "gem install bundler returned successfully, but still "\
@@ -338,7 +415,7 @@ def install_bundler(gem_program, silent: false)
                 end
             end
 
-            def install_autoproj(bundler)
+            def install_autoproj(bundler, bundler_version: self.bundler_version)
                 # Force bundler to update. If the user does not want this, let
                 # him specify a Gemfile with tighter version constraints
                 lockfile = File.join(dot_autoproj, 'Gemfile.lock')
@@ -353,14 +430,19 @@ def install_autoproj(bundler)
                 opts << "--path=#{gems_install_path}"
                 shims_path = File.join(dot_autoproj, 'bin')
 
-                result = system(clean_env,
-                    Gem.ruby, bundler, 'install',
-                        "--gemfile=#{autoproj_gemfile_path}",
-                        "--shebang=#{Gem.ruby}",
-                        "--binstubs=#{shims_path}",
-                        *opts, chdir: dot_autoproj)
+                version_arg = []
+                version_arg << "_#{bundler_version}_" if bundler_version
 
-                if !result
+                result = system(
+                    clean_env,
+                    Gem.ruby, bundler, *version_arg, 'install',
+                    "--gemfile=#{autoproj_gemfile_path}",
+                    "--shebang=#{Gem.ruby}",
+                    "--binstubs=#{shims_path}",
+                    *opts, chdir: dot_autoproj
+                )
+
+                unless result
                     STDERR.puts "FATAL: failed to install autoproj in #{dot_autoproj}"
                     exit 1
                 end
@@ -369,7 +451,7 @@ def install_autoproj(bundler)
                     root_dir, autoproj_gemfile_path, gems_gem_home)
             end
 
-            EXCLUDED_FROM_SHIMS = %w{rake thor}
+            EXCLUDED_FROM_SHIMS = %w[rake thor].freeze
 
             def self.rewrite_shims(shim_path, ruby_executable,
                 root_dir, autoproj_gemfile_path, gems_gem_home)
@@ -425,11 +507,7 @@ def self.shim_bundler(script_lines, ruby_executable, autoproj_gemfile_path, gems
 #
 
 # Autoproj generated preamble
-if defined?(Bundler)
-    Bundler.with_clean_env do
-        exec($0, *ARGV)
-    end
-end
+#{WITHOUT_BUNDLER}
 ENV['BUNDLE_GEMFILE'] ||= '#{autoproj_gemfile_path}'
 ENV['GEM_HOME'] = '#{gems_gem_home}'
 ENV.delete('GEM_PATH')
@@ -441,12 +519,7 @@ def self.shim_bundler(script_lines, ruby_executable, autoproj_gemfile_path, gems
             def self.shim_bundler_old(ruby_executable, autoproj_gemfile_path, gems_gem_home)
 "#! #{ruby_executable}
 
-if defined?(Bundler)
-    Bundler.with_clean_env do
-        exec($0, *ARGV)
-    end
-end
-
+#{WITHOUT_BUNDLER}
 ENV['BUNDLE_GEMFILE'] ||= '#{autoproj_gemfile_path}'
 ENV['GEM_HOME'] = '#{gems_gem_home}'
 ENV.delete('GEM_PATH')
@@ -471,14 +544,7 @@ def self.shim_script(script_lines, ruby_executable, root_dir,
 #
 
 # Autoproj generated preamble, v1
-if defined?(Bundler)
-    Bundler.with_clean_env do
-        exec(Hash['RUBYLIB' => nil], $0, *ARGV)
-    end
-elsif ENV['RUBYLIB']
-    exec(Hash['RUBYLIB' => nil], $0, *ARGV)
-end
-
+#{RUBYLIB_REINIT}
 ENV['BUNDLE_GEMFILE'] = '#{autoproj_gemfile_path}'
 ENV['AUTOPROJ_CURRENT_ROOT'] = '#{root_dir}'
 Gem.paths = Hash['GEM_HOME' => '#{gems_gem_home}', 'GEM_PATH' => '']
@@ -490,14 +556,7 @@ def self.shim_script_old(ruby_executable, root_dir, autoproj_gemfile_path,
                 gems_gem_home, load_line)
 "#! #{ruby_executable}
 
-if defined?(Bundler)
-    Bundler.with_clean_env do
-        exec(Hash['RUBYLIB' => nil], $0, *ARGV)
-    end
-elsif ENV['RUBYLIB']
-    exec(Hash['RUBYLIB' => nil], $0, *ARGV)
-end
-
+#{RUBYLIB_REINIT}
 ENV['BUNDLE_GEMFILE'] = '#{autoproj_gemfile_path}'
 ENV['AUTOPROJ_CURRENT_ROOT'] = '#{root_dir}'
 require 'rubygems'
@@ -591,8 +650,11 @@ def gem_bindir
                 #
                 # So, we're calling 'gem' as a subcommand to discovery the
                 # actual bindir
-                bindir = IO.popen(env_for_child,
-                    [Gem.ruby, '-e', 'puts "#{Gem.user_dir}/bin"']).read
+                bindir = IO.popen(
+                    env_for_child,
+                    [Gem.ruby, '-e', 'puts "#{Gem.user_dir}/bin"'], # rubocop:disable Lint/InterpolationCheck
+                    &:read
+                )
                 if bindir
                     @gem_bindir = bindir.chomp
                 else
@@ -600,11 +662,11 @@ def gem_bindir
                 end
             end
 
-            def install
+            def install(bundler_version: self.bundler_version)
                 if ENV['BUNDLER_GEMFILE']
                     raise "cannot run autoproj_install or autoproj_bootstrap while "\
-                        "under a 'bundler exec' subcommand or having loaded an env.sh. "\
-                        "Open a new console and try again"
+                          "under a 'bundler exec' subcommand or having loaded an "\
+                          "env.sh. Open a new console and try again"
                 end
 
                 gem_program  = self.class.guess_gem_program
@@ -612,13 +674,12 @@ def install
                 env['GEM_HOME'] = [gems_gem_home]
                 env['GEM_PATH'] = [gems_gem_home]
 
-                if bundler = find_bundler(gem_program)
+                if (bundler = find_bundler(gem_program, version: bundler_version))
                     puts "Detected bundler at #{bundler}"
                 else
                     puts "Installing bundler in #{gems_gem_home}"
-                    if !(bundler = install_bundler(gem_program))
-                        exit 1
-                    end
+                    bundler = install_bundler(gem_program, version: bundler_version)
+                    exit(1) unless bundler
                 end
                 self.class.rewrite_shims(
                     File.join(dot_autoproj, 'bin'),
@@ -630,7 +691,7 @@ def install
                 save_gemfile
 
                 puts "Installing autoproj in #{gems_gem_home}"
-                install_autoproj(bundler)
+                install_autoproj(bundler, bundler_version: bundler_version)
             end
 
             def load_config

data/lib/autoproj/ops/phase_reporting.rb

@@ -0,0 +1,49 @@
+module Autoproj
+    module Ops
+        # Common logic to generate build/import/utility reports
+        class PhaseReporting
+            def initialize(name, path, metadata_get)
+                @name = name
+                @path = path
+                @metadata_get = metadata_get
+            end
+
+            def create_report(autobuild_packages)
+                info = autobuild_packages.each_with_object({}) do |p, map|
+                    map[p.name] = @metadata_get.call(p)
+                end
+
+                dump = JSON.dump(
+                    "#{@name}_report" => {
+                        'timestamp' => Time.now,
+                        'packages' => info
+                    }
+                )
+
+                FileUtils.mkdir_p File.dirname(@path)
+                File.open(@path, 'w') do |io|
+                    io.write dump
+                end
+            end
+
+            def initialize_incremental_report
+                FileUtils.mkdir_p File.dirname(@path)
+                @incremental_report = ""
+            end
+
+            def report_incremental(autobuild_package)
+                new_metadata = @metadata_get.call(autobuild_package)
+                prefix = @incremental_report.empty? ? "\n" : ",\n"
+                @incremental_report.concat(
+                    "#{prefix}\"#{autobuild_package.name}\": #{JSON.dump(new_metadata)}"
+                )
+                File.open(@path, 'w') do |io|
+                    io.write "{ \"#{@name}_report\": "\
+                             "{\"timestamp\": #{JSON.dump(Time.now)}, \"packages\": {"
+                    io.write(@incremental_report)
+                    io.write "}}}"
+                end
+            end
+        end
+    end
+end

data/lib/autoproj/ops/snapshot.rb

@@ -119,6 +119,7 @@ def error_or_warn(package, error)
 
         def snapshot_packages(packages, target_dir = nil, only_local: true, fingerprint: false)
             result = Array.new
+            fingerprint_memo = Hash.new
             packages.each do |package_name|
                 package  = manifest.find_package_definition(package_name)
                 if !package
@@ -141,7 +142,7 @@ def snapshot_packages(packages, target_dir = nil, only_local: true, fingerprint:
                     end
 
                 if fingerprint
-                    vcs_info['fingerprint'] = package.autobuild.fingerprint
+                    vcs_info['fingerprint'] = package.autobuild.fingerprint(memo: fingerprint_memo)
                 end
 
                 if vcs_info

data/lib/autoproj/ops/tools.rb

@@ -24,8 +24,8 @@ def initialize(text_name, srcdir, importer = nil)
                 @@packages.delete(text_name)
             end
 
-            def import(options = Hash.new)
-                importer.import(self, options)
+            def import(**options)
+                importer.import(self, **options)
             end
 
             def add_stat(*args)