autoproj 2.14.0 → 2.15.0

data/lib/autoproj/cli/inspection_tool.rb

@@ -1,16 +1,16 @@
-require 'autoproj/cli/base'
+require "autoproj/cli/base"
 
 module Autoproj
     module CLI
         # Base class for CLI tools that do not change the state of the installed
         # system
         class InspectionTool < Base
-            def initialize_and_load(mainline: nil)
+            def initialize_and_load(mainline: nil, read_only: false)
                 Autoproj.silent do
-                    ws.setup
+                    ws.setup(read_only: read_only)
                     mainline = true if %w[mainline true].include?(mainline)
                     ws.load_package_sets(mainline: mainline)
-                    ws.config.save
+                    ws.config.save unless read_only
                     ws.setup_all_package_directories
                 end
             end
@@ -28,7 +28,13 @@ module Autoproj
             #   selected packages, the PackageSelection representing the
             #   selection resolution itself, and a flag telling whether some of
             #   the arguments were pointing within the configuration area
-            def finalize_setup(packages = [], non_imported_packages: :ignore, recursive: true, auto_exclude: false)
+            def finalize_setup(
+                packages = [],
+                non_imported_packages: :ignore,
+                recursive: true,
+                auto_exclude: false,
+                read_only: false
+            )
                 Autoproj.silent do
                     packages, config_selected =
                         normalize_command_line_package_selection(packages)
@@ -38,8 +44,8 @@ module Autoproj
                     ws.finalize_package_setup
                     source_packages, osdep_packages, resolved_selection =
                         resolve_selection(packages, recursive: recursive, non_imported_packages: non_imported_packages, auto_exclude: auto_exclude)
-                    ws.finalize_setup
-                    ws.export_installation_manifest
+                    ws.finalize_setup(read_only: read_only)
+                    ws.export_installation_manifest unless read_only
                     [source_packages, osdep_packages, resolved_selection, config_selected]
                 end
             end

data/lib/autoproj/cli/locate.rb

@@ -1,4 +1,4 @@
-require 'autoproj/cli/inspection_tool'
+require "autoproj/cli/inspection_tool"
 
 module Autoproj
     module CLI
@@ -11,8 +11,9 @@ module Autoproj
         class Locate < InspectionTool
             attr_reader :packages
             attr_reader :package_sets
-            
+
             class NotFound < CLIException; end
+
             class NoSuchDir < CLIException; end
 
             # Create the locate CLI interface
@@ -69,14 +70,14 @@ module Autoproj
                         candidates << pkg
                     end
                 end
-                return candidates
+                candidates
             end
 
             def find_packages_with_directory_shortnames(selection)
-                *directories, basename = *selection.split('/')
-                dirname_rx = directories.
-                    map { |d| "#{Regexp.quote(d)}\\w*" }.
-                    join("/")
+                *directories, basename = *selection.split("/")
+                dirname_rx = directories
+                             .map { |d| "#{Regexp.quote(d)}\\w*" }
+                             .join("/")
 
                 rx        = Regexp.new("#{dirname_rx}/#{Regexp.quote(basename)}")
                 rx_strict = Regexp.new("#{dirname_rx}/#{Regexp.quote(basename)}$")
@@ -85,12 +86,8 @@ module Autoproj
                 candidates_strict = []
                 packages.each do |pkg|
                     name = pkg.name
-                    if name =~ rx
-                        candidates << pkg
-                    end
-                    if name =~ rx_strict
-                        candidates_strict << pkg
-                    end
+                    candidates << pkg if name =~ rx
+                    candidates_strict << pkg if name =~ rx_strict
                 end
 
                 if candidates.size > 1 && candidates_strict.size == 1
@@ -114,22 +111,18 @@ module Autoproj
                            :build_dir
                        elsif options.delete(:prefix)
                            :prefix_dir
-                       elsif log_type = options[:log]
-                           if log_type == 'log'
-                               options.delete(:log)
-                           end
+                       elsif (log_type = options[:log])
+                           options.delete(:log) if log_type == "log"
                            :log
                        else
                            :source_dir
                        end
                 options[:mode] ||= mode
-                if selections.empty?
-                    selections << ws.root_dir
-                end
-                return selections, options
+                selections << ws.root_dir if selections.empty?
+                [selections, options]
             end
 
-            RESOLUTION_MODES = [:source_dir, :build_dir, :prefix_dir, :log]
+            RESOLUTION_MODES = %i[source_dir build_dir prefix_dir log]
 
             def run(selections, cache: !!packages, mode: :source_dir, log: nil)
                 if !RESOLUTION_MODES.include?(mode)
@@ -139,9 +132,7 @@ module Autoproj
                 end
 
                 selections.each do |string|
-                    if File.directory?(string)
-                        string = "#{File.expand_path(string)}/"
-                    end
+                    string = "#{File.expand_path(string)}/" if File.directory?(string)
                     if mode == :source_dir
                         puts source_dir_of(string)
                     elsif mode == :build_dir
@@ -149,7 +140,7 @@ module Autoproj
                     elsif mode == :prefix_dir
                         puts prefix_dir_of(string)
                     elsif mode == :log
-                        if all_logs = (log == 'all')
+                        if (all_logs = (log == "all"))
                             log = nil
                         end
                         result = logs_of(string, log: log)
@@ -179,17 +170,15 @@ module Autoproj
                     # If there is more than one candidate, check if there are some that are not
                     # present on disk
                     present = matching_packages.find_all { |pkg| File.directory?(pkg.srcdir) }
-                    if present.size == 1
-                        matching_packages = present
-                    end
+                    matching_packages = present if present.size == 1
                 end
 
                 if matching_packages.empty?
                     raise CLIInvalidArguments, "cannot find '#{selection}' in the current autoproj installation"
                 elsif matching_packages.size > 1
-                    raise CLIAmbiguousArguments, "multiple packages match '#{selection}' in the current autoproj installation: #{matching_packages.map(&:name).sort.join(", ")}"
+                    raise CLIAmbiguousArguments, "multiple packages match '#{selection}' in the current autoproj installation: #{matching_packages.map(&:name).sort.join(', ')}"
                 else
-                    return matching_packages.first
+                    matching_packages.first
                 end
             end
 
@@ -203,7 +192,7 @@ module Autoproj
             def source_dir_of(selection)
                 if workspace_dir?(selection)
                     ws.root_dir
-                elsif pkg_set = find_package_set(selection)
+                elsif (pkg_set = find_package_set(selection))
                     pkg_set.user_local_dir
                 else
                     resolve_package(selection).srcdir
@@ -246,20 +235,21 @@ module Autoproj
             #
             # The workspace is resolved as the main configuration
             #
-            # If 'log' is nil and multiple logs are available, 
+            # If 'log' is nil and multiple logs are available,
             def logs_of(selection, log: nil)
                 if workspace_dir?(selection) || (pkg_set = find_package_set(selection))
-                    if log && log != 'import'
-                        return []
-                    end
+                    return [] if log && log != "import"
+
                     name = if pkg_set then pkg_set.name
-                           else "autoproj main configuration"
+                           else
+                               "autoproj main configuration"
                            end
 
                     import_log = File.join(ws.log_dir, "#{name}-import.log")
                     if File.file?(import_log)
-                        return [import_log]
-                    else return []
+                        [import_log]
+                    else
+                        []
                     end
                 else
                     pkg = resolve_package(selection)
@@ -269,7 +259,7 @@ module Autoproj
 
             # Interactively select a log file among a list
             def select_log_file(log_files)
-                require 'tty/prompt'
+                require "tty/prompt"
 
                 log_files = log_files.map do |path|
                     [path, File.stat(path).mtime]
@@ -294,4 +284,3 @@ module Autoproj
         end
     end
 end
-

data/lib/autoproj/cli/log.rb

@@ -1,4 +1,4 @@
-require 'autoproj/cli/base'
+require "autoproj/cli/base"
 
 module Autoproj
     module CLI
@@ -15,15 +15,15 @@ module Autoproj
                 end
 
                 common_args = [Autobuild.tool(:git), "--git-dir=#{ws.config_dir}/.git"]
-                if since = options[:since]
-                    exec(*common_args, 'diff', parse_log_entry(since), 'autoproj@{0}')
+                if (since = options[:since])
+                    exec(*common_args, "diff", parse_log_entry(since), "autoproj@{0}")
                 elsif args.empty?
-                    exec(*common_args, 'reflog',
-                        Ops::Snapshot.import_state_log_ref, '--format=%Cgreen%gd %Cblue%cr %Creset%gs')
+                    exec(*common_args, "reflog",
+                         Ops::Snapshot.import_state_log_ref, "--format=%Cgreen%gd %Cblue%cr %Creset%gs")
                 elsif options[:diff]
-                    exec(*common_args, 'diff', *args.map { |entry| parse_log_entry(entry) })
+                    exec(*common_args, "diff", *args.map { |entry| parse_log_entry(entry) })
                 else
-                    exec(*common_args, 'show', *args.map { |entry| parse_log_entry(entry) })
+                    exec(*common_args, "show", *args.map { |entry| parse_log_entry(entry) })
                 end
             end