origen 0.60.7 → 0.60.8

data/config/version.rb

@@ -1,7 +1,7 @@
 module Origen
   MAJOR = 0
   MINOR = 60
-  BUGFIX = 7
+  BUGFIX = 8
   DEV = nil
   VERSION = [MAJOR, MINOR, BUGFIX].join(".") + (DEV ? ".pre#{DEV}" : '')
 end

data/lib/origen/application/configuration.rb

@@ -10,9 +10,9 @@ module Origen
                     :history_file, :release_directory, :release_email_subject,
                     :production_targets,
                     :vault, :output_directory, :reference_directory,
-                    :semantically_version, :log_directory, :pattern_name_translator,
+                    :semantically_version, :log_directory,
                     :pattern_directory, :pattern_output_directory, :pattern_prefix, :pattern_postfix,
-                    :pattern_header, :default_lsf_action, :release_instructions, :proceed_with_pattern,
+                    :pattern_header, :default_lsf_action, :release_instructions,
                     :test_program_output_directory, :erb_trim_mode, :test_program_source_directory,
                     :test_program_template_directory, :referenced_pattern_list, :program_prefix,
                     :copy_command, :diff_command, :compile_only_dot_erb_files, :web_directory,
@@ -22,6 +22,8 @@ module Origen
                     :user_aliases, :release_externally, :gem_name, :disqus_shortname,
                     :default_plugin, :rc_tag_prepend_v
 
+      attr_writer :pattern_name_translator, :proceed_with_pattern
+
       # Mark any attributes that are likely to depend on properties of the target here,
       # this will raise an error if they are ever accessed before the target has been
       # instantiated (a concern for Origen core developers only).
@@ -54,7 +56,7 @@ module Origen
       ]
 
       ATTRS_THAT_ARE_SET_TO_A_BLOCK = [
-        :current_plugin_pattern_header, :application_pattern_header, :shared_pattern_header, #:pattern_footer
+        :current_plugin_pattern_header, :application_pattern_header, :shared_pattern_header # :pattern_footer
       ]
 
       def log_deprecations
@@ -134,6 +136,7 @@ module Origen
           depend_on_target: true
         }.merge(options)
         attr_writer name
+
         define_method name do |override = true, &block|
           if block # _given?
             instance_variable_set("@#{name}".to_sym, block)

data/lib/origen/application/deployer.rb

@@ -134,7 +134,7 @@ module Origen
           # Delete everything so that we don't preserve old files
           git_repo.delete_all(git_sub_dir)
           FileUtils.mkdir_p(dir) unless File.exist?(dir)
-          `chmod a+w -R #{Origen.root}/web/output`      # Ensure world writable, required?
+          `chmod a+w -R #{Origen.root}/web/output` # Ensure world writable, required?
           FileUtils.cp_r "#{Origen.root}/web/output/.", dir
           git_repo.checkin git_sub_dir, unmanaged: true, comment: @commit_message
         else
@@ -144,9 +144,9 @@ module Origen
             require_remote_directories
           end
           # Copy the new contents across
-          `chmod g+w -R #{Origen.root}/web/output`      # Ensure group writable
+          `chmod g+w -R #{Origen.root}/web/output` # Ensure group writable
           FileUtils.cp_r "#{Origen.root}/web/output/.", offline_remote_directory
-          `chmod g+w -R #{offline_remote_directory}`  # Double ensure group writable
+          `chmod g+w -R #{offline_remote_directory}` # Double ensure group writable
           # Make live
           create_symlink offline_remote_directory, latest_symlink
           index = "#{Origen.config.web_directory}/index.html"
@@ -162,7 +162,7 @@ module Origen
         remote_dir = deploy_to_git? ? "#{git_repo.local}/#{git_sub_dir}" : live_remote_directory
         if remote_dir
           file = Origen.file_handler.clean_path_to(file)
-          sub_dir = Origen.file_handler.sub_dir_of(file, "#{Origen.root}/templates/web") .to_s
+          sub_dir = Origen.file_handler.sub_dir_of(file, "#{Origen.root}/templates/web").to_s
           page = file.basename.to_s.sub(/\..*/, '')
           # Special case for the main index page
           if page == 'index' && sub_dir == '.'

data/lib/origen/application/environment.rb

@@ -14,7 +14,7 @@ module Origen
     # An instance of this class is automatically
     # instantiated and available globally as Origen.environment.
     class Environment
-      DIR = "#{Origen.root}/environment"   # :nodoc:
+      DIR = "#{Origen.root}/environment" # :nodoc:
       SAVE_FILE = "#{DIR}/.default"      # :nodoc:
       DEFAULT_FILE = "#{DIR}/default.rb" # :nodoc:
 
@@ -89,7 +89,7 @@ module Origen
 
       # Prints out the current environment details to the command line
       def describe
-        f = self.file!
+        f = file!
         puts "Current environment: #{f.basename}"
         puts '*' * 70
         File.open(f).each do |line|
@@ -133,6 +133,7 @@ module Origen
       # otherwise returns nil
       def default_file
         return @default_file if @default_file
+
         if File.exist?(SAVE_FILE)
           File.open(SAVE_FILE) do |f|
             @default_file = Marshal.load(f)
@@ -146,6 +147,7 @@ module Origen
       # Returns the environment file (a Pathname object) if it has been defined, otherwise nil
       def file # :nodoc:
         return @file if @file
+
         if default_file && File.exist?(default_file)
           @file = default_file
         end

data/lib/origen/application/lsf.rb

@@ -34,6 +34,14 @@ module Origen
         #         -			  wam	  		WAM cron processing
         #         -			  grid	     		Low-priority batch jobs (random sim, regressions, etc). Access to all spare CPU cycles.
         attr_accessor :cores
+        # default 400. Set number of max jobs (- 100) to be able to run by this user before lsf submission pauses.
+        # eg if max jobs you want ran is 500, set to 400, which will block until local submissions
+        # is less than 400 at which point it can batch submit up to 100 more jobs
+        attr_accessor :max_jobs
+        # default false. used when calculating remote job count for comparison with the max jobs parameter
+        # If set to true, then only jobs of the specified queue will be counted, effectively making
+        # the max_jobs value a max_jobs per queue
+        attr_accessor :queue_count_only
 
         def initialize
           @group = Origen.site_config.lsf_group
@@ -42,6 +50,8 @@ module Origen
           @queue = Origen.site_config.lsf_queue
           @debug = Origen.site_config.lsf_debug
           @cores = Origen.site_config.lsf_cores
+          @max_jobs = Origen.site_config.lsf_max_jobs || 400
+          @queue_count_only = Origen.site_config.lsf_queue_count_only || false
         end
       end
 
@@ -80,7 +90,7 @@ module Origen
       def submit(command, options = {})
         options = {
           dependents: [],
-          rerunnable: true,  # Will rerun automatically if the execution host fails
+          rerunnable: true # Will rerun automatically if the execution host fails
         }.merge(options)
         limit_job_submissions do
           group = options[:group] || config.group
@@ -103,7 +113,7 @@ module Origen
           cmd = "bsub -oo /dev/null #{dependents} #{rerunnable} #{group} #{project} #{resource} #{queue} #{cores} '#{command}'"
           if config.debug
             puts cmd
-            '496212'  # Return a dummy ID to keep the caller happy
+            '496212' # Return a dummy ID to keep the caller happy
           else
             output = `#{cmd}`
             Origen.log.info output.strip
@@ -140,7 +150,16 @@ module Origen
         i = 0
         `bjobs 2>&1`.split("\n").each do |line|
           if line =~ /^(\d+).*(RUN|PEND)/
-            i += 1
+            if @queue_count_only && @queue
+              # only count jobs for current queue, helpful for when
+              # you have a service account user that runs lsf for a
+              # lot of jobs in addition to origen jobs
+              if line =~ /#{@queue}/
+                i += 1
+              end
+            else
+              i += 1
+            end
           end
         end
         i
@@ -152,7 +171,7 @@ module Origen
       def limit_job_submissions
         @local_job_count ||= 0
         if @local_job_count == 100
-          while remote_jobs_count > 400
+          while remote_jobs_count > @max_jobs
             puts 'Waiting for submitted jobs count to fall below limit...'
             sleep 5
           end

data/lib/origen/application/lsf_manager.rb

@@ -209,7 +209,7 @@ module Origen
           if options[:type] == :all
             File.delete(remote_jobs_file) if File.exist?(remote_jobs_file)
             @remote_jobs = {}
-            return
+            nil
           else
             send("#{options[:type]}_jobs").each do |job|
               remote_jobs.delete(job[:id])
@@ -278,54 +278,53 @@ module Origen
               #   New files:        0
               #   Changed files:    0
               #   FAILED files:     1
-              begin
-                line.gsub!(/\e\[\d+m/, '')  # Remove any coloring
-                if line =~ /Total patterns:\s+(\d+)/
-                  @completed_patterns = Regexp.last_match[1].to_i
-                elsif line =~ /Total vectors:\s+(\d+)/
-                  @total_vectors = Regexp.last_match[1].to_i
-                elsif line =~ /Total duration:\s+(\d+\.\d+)/
-                  @total_duration = Regexp.last_match[1].to_f
-                elsif line =~ /Total files:\s+(\d+)/
-                  @completed_files = Regexp.last_match[1].to_i
-                elsif line =~ /Changed patterns:\s+(\d+)/
-                  @changed_patterns = Regexp.last_match[1].to_i
-                elsif line =~ /Changed files:\s+(\d+)/
-                  @changed_files = Regexp.last_match[1].to_i
-                elsif line =~ /New patterns:\s+(\d+)/
-                  @new_patterns = Regexp.last_match[1].to_i
-                elsif line =~ /New files:\s+(\d+)/
-                  @new_files = Regexp.last_match[1].to_i
-                elsif line =~ /FAILED patterns:\s+(\d+)/
-                  @failed_patterns = Regexp.last_match[1].to_i
-                elsif line =~ /FAILED files:\s+(\d+)/
-                  @failed_files = Regexp.last_match[1].to_i
-                elsif line =~ /ERROR!/
-                  stats.errors += 1
-                  Origen.log.send :relog, line, options
+
+              line.gsub!(/\e\[\d+m/, '') # Remove any coloring
+              if line =~ /Total patterns:\s+(\d+)/
+                @completed_patterns = Regexp.last_match[1].to_i
+              elsif line =~ /Total vectors:\s+(\d+)/
+                @total_vectors = Regexp.last_match[1].to_i
+              elsif line =~ /Total duration:\s+(\d+\.\d+)/
+                @total_duration = Regexp.last_match[1].to_f
+              elsif line =~ /Total files:\s+(\d+)/
+                @completed_files = Regexp.last_match[1].to_i
+              elsif line =~ /Changed patterns:\s+(\d+)/
+                @changed_patterns = Regexp.last_match[1].to_i
+              elsif line =~ /Changed files:\s+(\d+)/
+                @changed_files = Regexp.last_match[1].to_i
+              elsif line =~ /New patterns:\s+(\d+)/
+                @new_patterns = Regexp.last_match[1].to_i
+              elsif line =~ /New files:\s+(\d+)/
+                @new_files = Regexp.last_match[1].to_i
+              elsif line =~ /FAILED patterns:\s+(\d+)/
+                @failed_patterns = Regexp.last_match[1].to_i
+              elsif line =~ /FAILED files:\s+(\d+)/
+                @failed_files = Regexp.last_match[1].to_i
+              elsif line =~ /ERROR!/
+                stats.errors += 1
+                Origen.log.send :relog, line, options
+              else
+                # Compress multiple blank lines
+                if line =~ /^\s*$/ || line =~ /.*\|\|\s*$/
+                  unless last_line_blank
+                    Origen.log.send(log_method, nil)
+                    last_line_blank = true
+                  end
                 else
-                  # Compress multiple blank lines
-                  if line =~ /^\s*$/ || line =~ /.*\|\|\s*$/
-                    unless last_line_blank
-                      Origen.log.send(log_method, nil)
-                      last_line_blank = true
-                    end
-                  else
-                    # Screen std origen output
-                    unless line =~ /  origen save/ ||
-                           line =~ /Insecure world writable dir/ ||
-                           line =~ /To save all of/
-                      line.strip!
-                      Origen.log.send :relog, line, options
-                      last_line_blank = false
-                    end
+                  # Screen std origen output
+                  unless line =~ /  origen save/ ||
+                         line =~ /Insecure world writable dir/ ||
+                         line =~ /To save all of/
+                    line.strip!
+                    Origen.log.send :relog, line, options
+                    last_line_blank = false
                   end
                 end
-              rescue
-                # Sometimes illegal UTF-8 characters can get into crash dumps, if this
-                # happens just print the line out rather than die
-                Origen.log.error line
               end
+            rescue
+              # Sometimes illegal UTF-8 characters can get into crash dumps, if this
+              # happens just print the line out rather than die
+              Origen.log.error line
             end
           end
           stats.completed_patterns += @completed_patterns
@@ -628,11 +627,9 @@ module Origen
       def restore_remote_jobs
         if File.exist?(remote_jobs_file)
           File.open(remote_jobs_file) do |f|
-            begin
-              Marshal.load(f)
-            rescue
-              nil
-            end
+            Marshal.load(f)
+          rescue
+            nil
           end
         end
       end
@@ -655,8 +652,7 @@ module Origen
                                 poll_duration_in_seconds: 1,
                                 # Don't wait long by the time this runs the LSF
                                 # should have guaranteed the job has run
-                                timeout_in_seconds:       120
-                               )
+                                timeout_in_seconds:       120)
             unless options[:dependents].all? { |id| job_passed?(id) }
               File.open(log_file(options[:id]), 'w') do |f|
                 f.puts "*** ERROR! *** #{options[:cmd].join(' ')} ***"

data/lib/origen/application/plugins.rb

@@ -38,13 +38,13 @@ module Origen
       def current
         return nil if @temporary == :none
         return nil if @disabled
-        name = @temporary || @current ||= begin
-          if Origen.app.session.origen_core[:default_plugin]
-            Origen.app.session.origen_core[:default_plugin]
-          elsif Origen.app.config.default_plugin && !Origen.app.session.origen_core[:default_plugin_cleared_by_user]
-            Origen.app.config.default_plugin
-          end
-        end
+
+        name = @temporary || @current ||= if Origen.app.session.origen_core[:default_plugin]
+                                            Origen.app.session.origen_core[:default_plugin]
+                                          elsif Origen.app.config.default_plugin && !Origen.app.session.origen_core[:default_plugin_cleared_by_user]
+                                            Origen.app.config.default_plugin
+                                          end
+
         find { |p| p.name.to_sym == name } if name
       end
 

data/lib/origen/application/release.rb

@@ -32,6 +32,7 @@ module Origen
         if authorized?
           Origen.app.plugins.validate_production_status(true)
           fail 'No revision control configured for this application, cannot release a new version' if Origen.app.rc.nil?
+
           unless Origen.app.rc.local_modifications.empty?
             puts <<-EOT
 Your workspace has local modifications that are preventing the requested action
@@ -50,15 +51,16 @@ Your workspace has local modifications that are preventing the requested action
           Origen.app.listeners_for(:validate_release).each(&:validate_release)
           lint_test
           get_latest_version_files
-          base_version = Origen.app.version(refresh: true)  # Read in the latest version
+          base_version = Origen.app.version(refresh: true) # Read in the latest version
           get_release_note unless note
-          new_version = get_or_confirm_version(type)  # Don't mask this like the above!
+          new_version = get_or_confirm_version(type) # Don't mask this like the above!
           write_version(new_version)
           # Refresh the version in the current thread
           Origen.app.version(refresh: true)
           if Origen.app.version != new_version
             fail "Sorry something has gone wrong trying to update the version counter to #{new_version}!"
           end
+
           Origen.app.version_tracker.add_version(Origen.app.version) if Origen.app.version.production?
           write_history
           Origen.app.listeners_for(:before_release_tag).each do |listener|
@@ -103,7 +105,7 @@ Your workspace has local modifications that are preventing the requested action
       end
 
       def release_gem
-        if File.exist?(File.join Origen.root, "#{Origen.app.gem_name}.gemspec")
+        if File.exist?(File.join(Origen.root, "#{Origen.app.gem_name}.gemspec"))
           Origen.app.listeners_for(:before_release_gem).each(&:before_release_gem)
           unless system 'rake gem:release'
             puts '***************************************'.red

data/lib/origen/application/runner.rb

@@ -1,7 +1,7 @@
 require 'fileutils'
 module Origen
   class Application
-    autoload :Statistics,    'origen/application/statistics'
+    autoload :Statistics, 'origen/application/statistics'
 
     # The Runner is responsible for co-ordinating all compile and generate
     # requests from the command line
@@ -140,15 +140,13 @@ module Origen
             # Only record user invocations at this time, also bypass windows since it seems
             # that threads can't be trusted not to block
             unless Origen.running_remotely? # || Origen.running_on_windows?
-              # rubocop:disable Style/RescueModifier
               record_invocation = Thread.new do
                 Origen.client.record_invocation(options[:action]) if options[:action]
-                rescue
+              rescue
                 # Dont allow server being down to flood the screen with the stacktrace
               end
-              # rubocop:enable Style/RescueModifier
             end
-           rescue
+          rescue
             # Don't allow this to kill an origen command
           end
         end
@@ -164,7 +162,7 @@ module Origen
                 record_invocation.value
               end
             end
-           rescue
+          rescue
             # Don't allow this to kill an origen command
           end
         end
@@ -174,6 +172,7 @@ module Origen
       # things like :compile => true, the extract_action method handles the old code
       def extract_action(options)
         return options[:action] if options[:action]
+
         if options[:compile]
           :compile
         elsif options[:program]

data/lib/origen/application/statistics.rb

@@ -10,6 +10,7 @@ module Origen
 
       class Pattern
         attr_accessor :vectors, :cycles, :duration
+
         def initialize
           @vectors = 0
           @cycles = 0

data/lib/origen/application/target.rb

@@ -12,7 +12,7 @@ module Origen
     # An instance of this class is automatically instantiated and available globally
     # as Origen.app.target
     class Target
-      DIR = "#{Origen.root}/target"        # :nodoc:
+      DIR = "#{Origen.root}/target" # :nodoc:
       SAVE_FILE = "#{DIR}/.default"      # :nodoc:
       DEFAULT_FILE = "#{DIR}/default.rb" # :nodoc:
 
@@ -34,7 +34,7 @@ module Origen
       def loop(options = {})
         options = {
           set_target:  true,
-          force_debug: false,  # Set true to force debug mode for all targets
+          force_debug: false # Set true to force debug mode for all targets
         }.merge(options)
         targets = [options.delete(:target), options.delete(:targets)].flatten.compact.uniq
         targets = [file!.basename.to_s] if targets.empty?
@@ -57,7 +57,7 @@ module Origen
       #   end
       def each_production(options = {})
         options = {
-          force_debug: false,  # Set true to force debug mode for all targets
+          force_debug: false # Set true to force debug mode for all targets
         }.merge(options)
         prod_targets.each do |moo, targets|
           [targets].flatten.each do |target|
@@ -79,7 +79,7 @@ module Origen
       #   end
       def each_unique_production(options = {})
         options = {
-          force_debug: false,  # Set true to force debug mode for all targets
+          force_debug: false # Set true to force debug mode for all targets
         }.merge(options)
         targets = {}
         prod_targets.each do |moo, moos_targets|
@@ -118,7 +118,7 @@ module Origen
       # defined there.
       def load!(options = {})
         options = {
-          force_debug: false,  # Set true to force debug mode for all targets
+          force_debug: false # Set true to force debug mode for all targets
         }.merge(options)
         Origen.app.load_target!(options)
       end
@@ -235,7 +235,7 @@ module Origen
 
       # Prints out the current target details to the command line
       def describe
-        f = self.file!
+        f = file!
         puts "Current target: #{f.basename}"
         puts '*' * 70
         File.open(f).each do |line|
@@ -279,6 +279,7 @@ module Origen
       # otherwise returns nil
       def default_file
         return @default_file if @default_file
+
         if File.exist?(SAVE_FILE)
           File.open(SAVE_FILE) do |f|
             @default_file = Marshal.load(f)
@@ -292,6 +293,7 @@ module Origen
       # Returns the target file (a Pathname object) if it has been defined, otherwise nil
       def file # :nodoc:
         return @file if @file
+
         if default_file && File.exist?(default_file)
           @file = default_file
         end
@@ -374,6 +376,7 @@ module Origen
       def prod_targets # :nodoc:
         return {} unless Origen.config.production_targets
         return @prod_targets if @prod_targets
+
         @prod_targets = {}
         Origen.config.production_targets.each do |key, value|
           @prod_targets[key.upcase] = value

data/lib/origen/application/version_tracker.rb

@@ -21,13 +21,12 @@ module Origen
       # Returns the persisted storage container (a Hash)
       def storage
         return @storage if @storage
+
         if File.exist?(STORAGE_FILE)
           File.open(STORAGE_FILE) do |f|
-            begin
-              @storage = Marshal.load(f)
-            rescue
-              @storage = {}
-            end
+            @storage = Marshal.load(f)
+          rescue
+            @storage = {}
           end
         else
           @storage = {}

data/lib/origen/application/workspace_manager.rb

@@ -50,6 +50,7 @@ module Origen
       # all imported application workspaces
       def imports_directory
         return @imports_directory if @imports_directory
+
         old = "#{container_directory}/#{revision_control_root.basename}_imports_DO_NOT_HAND_MODIFY"
         @imports_directory = "#{container_directory}/.#{revision_control_root.basename}_imports_DO_NOT_HAND_MODIFY"
         FileUtils.rm_rf(old) if File.exist?(old)
@@ -60,6 +61,7 @@ module Origen
       # all remotes workspaces
       def remotes_directory
         return @remotes_directory if @remotes_directory
+
         old = "#{container_directory}/#{revision_control_root.basename}_remotes_DO_NOT_HAND_MODIFY"
         @remotes_directory = "#{container_directory}/.#{revision_control_root.basename}_remotes_DO_NOT_HAND_MODIFY"
         FileUtils.rm_rf(old) if File.exist?(old)
@@ -86,7 +88,7 @@ module Origen
         remote_ref = "#{origen_root(workspace)}/.ref"
         unless File.exist?(remote_ref)
           FileUtils.mkdir_p(remote_ref)
-          `touch #{remote_ref}/dont_delete`  # Make sure the pop does not blow this away
+          `touch #{remote_ref}/dont_delete` # Make sure the pop does not blow this away
         end
         if Origen.running_on_windows?
           system("call mklink /h #{reference_dir} #{remote_ref}")
@@ -113,6 +115,7 @@ module Origen
         if File.exist?(path.to_s) && !options[:allow_rebuild]
           fail "Sorry but #{path} already exists!"
         end
+
         FileUtils.rm_rf(path.to_s) if File.exist?(path.to_s)
         rc = RevisionControl.new options.merge(remote: options[:rc_url], local: path.to_s)
         rc.build
@@ -121,7 +124,7 @@ module Origen
       # Switches the given workspace path to the given version tag
       def switch_version(workspace, tag, options = {})
         options = {
-          origen_root_only: false, # When true pop the Origen.root dir only instead
+          origen_root_only: false # When true pop the Origen.root dir only instead
           # of the whole application workspace - these may or may
           # not be the same thing depending on the application.
         }.merge(options)

data/lib/origen/application.rb

@@ -23,8 +23,8 @@ module Origen
     require 'origen/users'
     include Origen::Users
 
-    attr_accessor :name
-    attr_accessor :namespace
+    attr_writer :name                       # attr_accessor leads to a duplicate method
+    attr_writer :namespace                  # attr_accessor leads to a duplicate method
 
     class << self
       def inherited(base)
@@ -84,6 +84,7 @@ module Origen
         @apps_by_namespace ||= {}
         @apps_by_namespace[namespace] ||= begin
           return Origen.app if Origen.app.namespace == namespace
+
           Origen.app.plugins.each do |plugin|
             return plugin if plugin.namespace == namespace
           end
@@ -105,7 +106,7 @@ module Origen
       include Rake::DSL
 
       def load_tasks
-        $VERBOSE = nil  # Don't care about world writable dir warnings and the like
+        $VERBOSE = nil # Don't care about world writable dir warnings and the like
         require 'colored'
 
         # Load all Origen tasks first
@@ -463,12 +464,14 @@ END
       if max && listeners.size > max
         fail "You can only define a #{callback} callback #{max > 1 ? (max.to_s + 'times') : 'once'}, however you have declared it #{listeners.size} times for instances of: #{listeners.map(&:class)}"
       end
+
       listeners
     end
 
     def version(options = {})
       @version = nil if options[:refresh]
       return @version if @version
+
       if Origen.running_globally?
         @version = Origen.version
       else
@@ -567,6 +570,7 @@ END
           if capture && line =~ /^#+ by (.*) on (.*(AM|PM))/
             user = Origen.fsl.find_by_name(Regexp.last_match(1))
             return user if user
+
             return Regexp.last_match(1)
           end
         else
@@ -871,7 +875,7 @@ END
       # declares here, the objects registered with origen should be refreshed accordingly
       clear_dynamic_resources
       load_event(:transient) do
-        Origen.mode = Origen.app.session.origen_core[:mode] || :production  # Important since a production target may rely on the default
+        Origen.mode = Origen.app.session.origen_core[:mode] || :production # Important since a production target may rely on the default
         begin
           $_target_options = @target_load_options
           Origen.target.set_signature(@target_load_options)
@@ -1028,14 +1032,10 @@ END
     #   When debug is enabled, exception_class will be raised, defaulting to <code>RuntimeError</code>.
     def fail!(message: nil, exception_class: RuntimeError, exit_status: 1)
       if Origen.debug?
-        # rubocop:disable Style/RedundantSelf
         self.fail(message: message, exception_class: exception_class)
-        # rubocop:enable Style/RedundantSelf
       else
         begin
-          # rubocop:disable Style/RedundantSelf
           self.fail(message: message, exception_class: exception_class)
-          # rubocop:enable Style/RedundantSelf
         rescue exception_class => e
           Origen.log.error(e.message)
           exit exit_status
@@ -1051,7 +1051,7 @@ END
     #     require "my_backend" # in lib/my_backend
     #     config.i18n.backend = MyBackend
     #   end
-    def add_lib_to_load_path! #:nodoc:
+    def add_lib_to_load_path! # :nodoc:
       [root.join('lib'), root.join('vendor', 'lib'), root.join('app', 'lib')].each do |path|
         $LOAD_PATH.unshift(path.to_s) if File.exist?(path) && !$LOAD_PATH.include?(path.to_s)
       end

data/lib/origen/boot/api.rb

@@ -8,6 +8,9 @@ module Origen
 
   # Platform independent means of retrieving the hostname
   def self.hostname
+    # gethostbyname is deprecated in favor of Addrinfo#getaddrinfo
+    # rubocop:disable Lint/DeprecatedClassMethods
     Socket.gethostbyname(Socket.gethostname).first.downcase
+    # rubocop:enable Lint/DeprecatedClassMethods
   end
 end