u3d 1.2.3 → 1.3.0

data/lib/u3d/installer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ## --- BEGIN LICENSE BLOCK ---
 # Copyright (c) 2016-present WeWantToKnow AS
 #
@@ -30,13 +32,13 @@ require 'pathname'
 require 'zip'
 
 module U3d
-  DEFAULT_LINUX_INSTALL = '/opt/'.freeze
-  DEFAULT_MAC_INSTALL = '/'.freeze
-  DEFAULT_WINDOWS_INSTALL = 'C:/Program Files/'.freeze
-  UNITY_DIR = "Unity_%<version>s".freeze
-  UNITY_DIR_LONG = "Unity_%<version>s_%<build_number>s".freeze
-  UNITY_DIR_LINUX = "unity-editor-%<version>s".freeze
-  UNITY_DIR_LINUX_LONG = "unity-editor-%<version>s_%<build_number>s".freeze
+  DEFAULT_LINUX_INSTALL = '/opt/'
+  DEFAULT_MAC_INSTALL = '/'
+  DEFAULT_WINDOWS_INSTALL = 'C:/Program Files/'
+  UNITY_DIR = "Unity_%<version>s"
+  UNITY_DIR_LONG = "Unity_%<version>s_%<build_number>s"
+  UNITY_DIR_LINUX = "unity-editor-%<version>s"
+  UNITY_DIR_LINUX_LONG = "unity-editor-%<version>s_%<build_number>s"
 
   class Installer
     def self.create
@@ -57,7 +59,7 @@ module U3d
       installer = Installer.create
       files.each do |name, file, info|
         UI.header "Installing #{info.name} (#{name})"
-        UI.message 'Installing with ' + file
+        UI.message "Installing with #{file}"
         installer.install(file, version, installation_path: installation_path, info: info)
       end
     end
@@ -71,20 +73,24 @@ module U3d
   class BaseInstaller
     def sanitize_installs
       return unless UI.interactive? || Helper.test?
+
       unclean = []
       installed.each { |unity| unclean << unity unless unity.clean_install? }
       return if unclean.empty?
+
       UI.important("u3d can optionally standardize the existing Unity installation names and locations.")
       UI.important("Check the documentation for more information:")
       UI.important("** https://github.com/DragonBox/u3d/blob/master/README.md#default-installation-paths **")
       unclean.each { |unity| sanitize_install(unity, dry_run: true) }
       return unless UI.confirm("#{unclean.count} Unity installation(s) will be moved. Proceed??")
+
       unclean.each { |unity| sanitize_install(unity) }
     end
 
     def installed_sorted_by_versions
       list = installed
       return [] if list.empty?
+
       list.sort { |a, b| UnityVersionComparator.new(a.version) <=> UnityVersionComparator.new(b.version) }
     end
 
@@ -123,7 +129,7 @@ module U3d
         end
       end
 
-      if info && info.rename_from && info.rename_to
+      if info&.rename_from && info&.rename_to
         rename_from = info.rename_from.gsub(/{UNITY_PATH}/, unity.root_path)
         rename_to = info.rename_to.gsub(/{UNITY_PATH}/, unity.root_path)
         Utils.ensure_dir(rename_to)
@@ -131,7 +137,7 @@ module U3d
         if File.file? rename_from
           FileUtils.mv(rename_from, rename_to)
         else
-          Dir.glob(rename_from + '/*').each { |path| FileUtils.mv(path, rename_to) }
+          Dir.glob("#{rename_from}/*").each { |path| FileUtils.mv(path, rename_to) }
         end
       end
 
@@ -139,7 +145,7 @@ module U3d
     end
 
     def package_destination(info, unity_root_path)
-      if info && info.destination
+      if info&.destination
         info.destination.gsub(/{UNITY_PATH}/, unity_root_path)
       else
         unity_root_path
@@ -151,14 +157,15 @@ module U3d
     # Returns an array of ruby style paths
     def extra_installation_paths
       return [] if ENV['U3D_EXTRA_PATHS'].nil?
+
       ENV['U3D_EXTRA_PATHS'].strip.split(File::PATH_SEPARATOR).map { |p| File.expand_path p }
     end
 
-    def find_installations_with_path(default_root_path: '', postfix: [])
+    def find_installations_with_path(default_root_path: '', postfix: [], &block)
       ([default_root_path] | extra_installation_paths).map do |path|
         UI.verbose "Looking for installed Unity version under #{path}"
         pattern = File.join([path] + postfix)
-        Dir.glob(pattern).map { |found_path| yield found_path }
+        Dir.glob(pattern).map(&block)
       end.flatten
     end
   end
@@ -189,15 +196,16 @@ module U3d
     end
 
     def install(file_path, version, installation_path: nil, info: nil)
-      # rubocop:enable UnusedMethodArgument
       extension = File.extname(file_path)
       raise "Installation of #{extension} files is not supported on Mac" unless %w[.zip .po .pkg .dmg].include? extension
+
       path = installation_path || DEFAULT_MAC_INSTALL
-      if extension == '.po'
+      case extension
+      when '.po'
         install_po(file_path, version, info: info)
-      elsif extension == '.zip'
+      when '.zip'
         install_zip(file_path, version, info: info)
-      elsif extension == '.dmg'
+      when '.dmg'
         UI.important "Skipping installation of #{file_path} for now"
       else
         install_pkg(file_path, version: version, target_path: path)
@@ -278,7 +286,7 @@ module U3d
     end
   end
 
-  # rubocop:disable ClassLength
+  # rubocop:disable Metrics/ClassLength
   class LinuxInstaller < BaseInstaller
     def sanitize_install(unity, long: false, dry_run: false)
       source_path = File.expand_path(unity.root_path)
@@ -296,26 +304,26 @@ module U3d
       paths.map { |path| LinuxInstallation.new(root_path: path) }
     end
 
-    # rubocop:disable PerceivedComplexity
     def install(file_path, version, installation_path: nil, info: nil)
-      # rubocop:enable UnusedMethodArgument, PerceivedComplexity
       extension = File.extname(file_path)
 
       raise "Installation of #{extension} files is not supported on Linux" unless ['.zip', '.po', '.sh', '.xz', '.pkg'].include? extension
-      if extension == '.sh'
+
+      case extension
+      when '.sh'
         path = installation_path || DEFAULT_LINUX_INSTALL
         install_sh(file_path, installation_path: path)
-      elsif extension == '.xz'
+      when '.xz'
         new_path = File.join(DEFAULT_LINUX_INSTALL, format(UNITY_DIR_LINUX, version: version))
         path = installation_path || new_path
         install_xz(file_path, installation_path: path)
-      elsif extension == '.pkg'
+      when '.pkg'
         new_path = File.join(DEFAULT_LINUX_INSTALL, format(UNITY_DIR_LINUX, version: version))
         path = installation_path || new_path
         install_pkg(file_path, installation_path: path)
-      elsif extension == '.po'
+      when '.po'
         install_po(file_path, version, info: info)
-      elsif extension == '.zip'
+      when '.zip'
         install_zip(file_path, version, info: info)
       end
 
@@ -399,6 +407,7 @@ module U3d
 
     def pkg_install_path(unity_root_path, pinfo_path)
       raise "PackageInfo not found under #{pinfo_path}" unless File.exist? pinfo_path
+
       pinfo = File.read(pinfo_path)
       require 'rexml/document'
       d = REXML::Document.new(pinfo)
@@ -414,6 +423,7 @@ module U3d
       else
         install_location = d.root.attributes['install-location']
         raise "Not sure how to install this module with identifier #{identifier} install-location: #{install_location}" unless install_location.start_with? '/Applications/Unity/'
+
         install_location.gsub(%(\/Applications\/Unity), "#{unity_root_path}/Editor/Data")
       end
     end
@@ -442,7 +452,7 @@ module U3d
       paths
     end
   end
-  # rubocop:enable ClassLength
+  # rubocop:enable Metrics/ClassLength
 
   class WindowsInstaller < BaseInstaller
     def sanitize_install(unity, long: false, dry_run: false)
@@ -470,10 +480,12 @@ module U3d
     def install(file_path, version, installation_path: nil, info: nil)
       extension = File.extname(file_path)
       raise "Installation of #{extension} files is not supported on Windows" unless %w[.po .zip .exe .msi].include? extension
+
       path = installation_path || File.join(DEFAULT_WINDOWS_INSTALL, format(UNITY_DIR, version: version))
-      if extension == '.po'
+      case extension
+      when '.po'
         install_po(file_path, version, info: info)
-      elsif extension == '.zip'
+      when '.zip'
         install_zip(file_path, version, info: info)
       else
         install_exe(file_path, installation_path: path, info: info)
@@ -489,14 +501,14 @@ module U3d
         if info.command
           command = info.command
           if /msiexec/ =~ command
-            command.sub!(/{FILENAME}/, '"' + U3dCore::Helper.windows_path(file_path) + '"')
+            command.sub!(/{FILENAME}/, "\"#{U3dCore::Helper.windows_path(file_path)}\"")
           else
             command.sub!(/{FILENAME}/, file_path.argescape)
           end
           command.sub!(/{INSTDIR}/, final_path)
           command.sub!(/{DOCDIR}/, final_path)
           command.sub!(/{MODULEDIR}/, final_path)
-          command.sub!(%r{\/D=}, '/S /D=') unless %r{\/S} =~ command
+          command.sub!(%r{/D=}, '/S /D=') unless %r{/S} =~ command
         end
         command ||= file_path.argescape
         U3dCore::CommandExecutor.execute(command: command, admin: true)
@@ -572,9 +584,8 @@ module U3d
         raise 'Cannot install dependencies on your Linux distribution'
       end
 
-      if UI.interactive?
-        return unless UI.confirm "Install dependencies? (#{DEPENDENCIES.length} dependency(ies) to install)"
-      end
+      return if UI.interactive? && !(UI.confirm "Install dependencies? (#{DEPENDENCIES.length} dependency(ies) to install)")
+
       U3dCore::CommandExecutor.execute(command: "#{prefix} #{DEPENDENCIES.join(' ')}", admin: true)
     end
   end

data/lib/u3d/log_analyzer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ## --- BEGIN LICENSE BLOCK ---
 # Copyright (c) 2016-present WeWantToKnow AS
 #
@@ -25,9 +27,9 @@ require 'u3d/failure_reporter'
 
 module U3d
   # Analyzes log by filtering output along a set of rules
-  # rubocop:disable ClassLength, PerceivedComplexity, BlockNesting
+  # rubocop:disable Metrics/ClassLength, Metrics/PerceivedComplexity, Metrics/BlockNesting
   class LogAnalyzer
-    RULES_PATH = File.expand_path('../../../config/log_rules.json', __FILE__)
+    RULES_PATH = File.expand_path('../../config/log_rules.json', __dir__)
     MEMORY_SIZE = 10
 
     def initialize
@@ -55,6 +57,7 @@ module U3d
         # Phase parsing
         next unless phase['active']
         next if phase['phase_start_pattern'].nil?
+
         phase['phase_start_pattern'] = Regexp.new phase['phase_start_pattern']
         phase['phase_end_pattern'] = Regexp.new phase['phase_end_pattern'] if phase['phase_end_pattern']
         phase.delete('comment')
@@ -79,6 +82,7 @@ module U3d
       @phases.each do |name, phase|
         next if name == @active_phase
         next unless line =~ phase['phase_start_pattern']
+
         finish_phase if @active_phase
         @active_phase = name
         UI.verbose("--- Beginning #{name} phase ---")
@@ -116,12 +120,10 @@ module U3d
           # It's not the end of the rules, should the line be stored?
           elsif rule['store_lines']
             match = false
-            if rule['ignore_lines']
-              rule['ignore_lines'].each do |pat|
-                if line =~ pat
-                  match = true
-                  break
-                end
+            rule['ignore_lines']&.each do |pat|
+              if line =~ pat
+                match = true
+                break
               end
             end
             @rule_lines_buffer << line.chomp unless match
@@ -133,6 +135,7 @@ module U3d
           ruleset.each do |rn, r|
             pattern = r['start_pattern']
             next unless line =~ pattern
+
             @active_rule = rn if r['end_pattern']
             match = line.match(pattern)
             @context = match.names.map(&:to_sym).zip(match.captures).to_h
@@ -145,10 +148,12 @@ module U3d
                   match = false
                   r['fetch_first_line_not_matching'].each do |pat|
                     next unless l =~ pat
+
                     match = true
                     break
                   end
                   next if match
+
                   fetched_line = l
                   break
                 end
@@ -196,7 +201,7 @@ module U3d
       if @active_rule
         # Active rule should be finished
         # If it is still active during phase change, it means that something went wrong
-        context = @lines_memory.map { |l| "> #{l}" }.join('')
+        context = @lines_memory.map { |l| "> #{l}" }.join
         UI.error("[#{@active_phase}] Could not finish active rule '#{@active_rule}'. Aborting it. Context:\n#{context}")
 
         U3d::FailureReporter.report(
@@ -237,26 +242,25 @@ module U3d
       message
     end
 
-    def parse_rule(r)
-      return false unless r['active']
-      return false if r['start_pattern'].nil?
-      r['start_pattern'] = Regexp.new r['start_pattern']
-      r['end_pattern'] = Regexp.new r['end_pattern'] if r['end_pattern']
-      if r['fetch_line_at_index']
-        r.delete('fetch_line_at_index') if r['fetch_line_at_index'] >= MEMORY_SIZE
-        r.delete('fetch_line_at_index') if r['fetch_line_at_index'] <= 0
-      elsif r['fetch_first_line_not_matching']
-        r['fetch_first_line_not_matching'].map! { |pat| Regexp.new pat }
-      end
-      if r['fetch_line_at_index'] || r['fetch_first_line_not_matching']
-        r['fetched_line_pattern'] = Regexp.new r['fetched_line_pattern'] if r['fetched_line_pattern']
+    def parse_rule(rule)
+      return false unless rule['active']
+      return false if rule['start_pattern'].nil?
+
+      rule['start_pattern'] = Regexp.new rule['start_pattern']
+      rule['end_pattern'] = Regexp.new rule['end_pattern'] if rule['end_pattern']
+      if rule['fetch_line_at_index']
+        rule.delete('fetch_line_at_index') if rule['fetch_line_at_index'] >= MEMORY_SIZE
+        rule.delete('fetch_line_at_index') if rule['fetch_line_at_index'] <= 0
+      elsif rule['fetch_first_line_not_matching']
+        rule['fetch_first_line_not_matching'].map! { |pat| Regexp.new pat }
       end
-      r['type'] = 'important' if r['type'] == 'warning'
-      r['type'] = 'message' if r['type'] && r['type'] != 'error' && r['type'] != 'important' && r['type'] != 'success'
-      r['type'] ||= 'message'
-      r['ignore_lines'].map! { |pat| Regexp.new pat } if r['ignore_lines']
+      rule['fetched_line_pattern'] = Regexp.new rule['fetched_line_pattern'] if (rule['fetch_line_at_index'] || rule['fetch_first_line_not_matching']) && (rule['fetched_line_pattern'])
+      rule['type'] = 'important' if rule['type'] == 'warning'
+      rule['type'] = 'message' if rule['type'] && rule['type'] != 'error' && rule['type'] != 'important' && rule['type'] != 'success'
+      rule['type'] ||= 'message'
+      rule['ignore_lines']&.map! { |pat| Regexp.new pat }
       true
     end
   end
-  # rubocop:enable ClassLength, PerceivedComplexity, BlockNesting
+  # rubocop:enable Metrics/ClassLength, Metrics/PerceivedComplexity, Metrics/BlockNesting
 end

data/lib/u3d/unity_license.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ## --- BEGIN LICENSE BLOCK ---
 # Copyright (c) 2016-present WeWantToKnow AS
 #

data/lib/u3d/unity_module.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ## --- BEGIN LICENSE BLOCK ---
 # Copyright (c) 2016-present WeWantToKnow AS
 #

data/lib/u3d/unity_project.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ## --- BEGIN LICENSE BLOCK ---
 # Copyright (c) 2016-present WeWantToKnow AS
 #
@@ -46,9 +48,10 @@ module U3d
       require 'yaml'
       project_version = File.join(project_settings_path, 'ProjectVersion.txt')
       return nil unless File.exist? project_version
+
       yaml = YAML.safe_load(File.read(project_version))
       version = yaml['m_EditorVersion']
-      version.gsub!(/(\d+\.\d+\.\d+)(?:x)?(\w\d+)(?:Linux)?/, '\1\2') if Helper.linux?
+      version = version.gsub(/(\d+\.\d+\.\d+)(?:x)?(\w\d+)(?:Linux)?/, '\1\2') if Helper.linux?
       version
     end
   end

data/lib/u3d/unity_runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ## --- BEGIN LICENSE BLOCK ---
 # Copyright (c) 2016-present WeWantToKnow AS
 #
@@ -45,6 +47,7 @@ module U3d
       if log_file
         tail_thread = start_tail_thread(log_file, output_callback)
         return unless tail_thread.status
+
         tail_thread.run
       end
 
@@ -78,9 +81,9 @@ module U3d
     end
 
     class << self
-      # rubocop:disable MethodName
+      # rubocop:disable Naming/MethodName
       def find_logFile_in_args(args)
-        # rubocop:enable MethodName
+        # rubocop:enable Naming/MethodName
         find_arg_in_args('-logFile', args)
       end
 
@@ -90,6 +93,7 @@ module U3d
 
       def find_arg_in_args(arg_to_find, args)
         raise 'Only arguments of type array supported right now' unless args.is_a?(Array)
+
         args.each_with_index do |arg, index|
           return args[index + 1] if arg == arg_to_find && index < args.count - 1
         end
@@ -101,12 +105,10 @@ module U3d
 
     def start_tail_thread(log_file, output_callback)
       tail_thread = Thread.new do
-        begin
-          pipe(log_file) { |line| output_callback.call(line) }
-        rescue StandardError => e
-          UI.error "Failure while trying to pipe #{log_file}: #{e.message}"
-          e.backtrace.each { |l| UI.error "  #{l}" }
-        end
+        pipe(log_file) { |line| output_callback.call(line) }
+      rescue StandardError => e
+        UI.error "Failure while trying to pipe #{log_file}: #{e.message}"
+        e.backtrace.each { |l| UI.error "  #{l}" }
       end
 
       # Wait for tail_thread setup to be complete
@@ -114,14 +116,14 @@ module U3d
       tail_thread
     end
 
-    def pipe(file)
+    def pipe(file, &block)
       File.open(file, 'r') do |f|
         f.extend File::Tail
         f.interval = 0.1
         f.max_interval = 0.4
         f.backward 0
         Thread.stop
-        f.tail { |l| yield l }
+        f.tail(&block)
       end
     end
   end

data/lib/u3d/unity_version_definition.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ## --- BEGIN LICENSE BLOCK ---
 # Copyright (c) 2016-present WeWantToKnow AS
 #
@@ -51,6 +53,7 @@ module U3d
 
     def [](package)
       return nil unless available_package? package
+
       @packages.find { |pack| pack.id == package.downcase }
     end
 

data/lib/u3d/unity_version_number.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ## --- BEGIN LICENSE BLOCK ---
 # Copyright (c) 2016-present WeWantToKnow AS
 #
@@ -24,14 +26,14 @@ require 'u3d/utils'
 
 module U3d
   class UnityVersionNumber
-    attr_reader :unity_version
-    attr_reader :parts
+    attr_reader :unity_version, :parts
 
     def initialize(version)
       @unity_version = version
       parsed = Utils.parse_unity_version(@unity_version)
       parsed.each_with_index do |val, index|
         next if val.nil? || (index == 3)
+
         parsed[index] = val.to_i
       end
       @parts = parsed
@@ -52,12 +54,16 @@ module U3d
     def <=>(other)
       comp = @version.parts[0] <=> other.version.parts[0]
       return comp if comp.nonzero?
+
       comp = @version.parts[1] <=> other.version.parts[1]
       return comp if comp.nonzero?
+
       comp = @version.parts[2] <=> other.version.parts[2]
       return comp if comp.nonzero?
+
       comp = RELEASE_LETTER_STRENGTH[@version.parts[3].to_sym] <=> RELEASE_LETTER_STRENGTH[other.version.parts[3].to_sym]
       return comp if comp.nonzero?
+
       return @version.parts[4] <=> other.version.parts[4]
     end
 

data/lib/u3d/unity_versions.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ## --- BEGIN LICENSE BLOCK ---
 # Copyright (c) 2016-present WeWantToKnow AS
 #
@@ -47,6 +49,7 @@ module U3d
     def fetch_cookie
       UI.verbose "FetchCookie? #{@cookie}"
       return @cookie if @cookie
+
       cookie_str = ''
       url = 'https://forum.unity.com/forums/linux-editor.93/' # a page that triggers cookies
       uri = URI(url)
@@ -56,15 +59,15 @@ module U3d
         response = http.request request
 
         case response
-        when Net::HTTPSuccess then
+        when Net::HTTPSuccess
           UI.verbose "unexpected result"
-        when Net::HTTPRedirection then
+        when Net::HTTPRedirection
           # A session must be opened with the server before accessing forum
           res = nil
           cookie_str = ''
           # Store the name and value of the cookies returned by the server
           response['set-cookie'].gsub(/\s+/, '').split(',').each do |c|
-            cookie_str << c.split(';', 2)[0] + '; '
+            cookie_str << ("#{c.split(';', 2)[0]}; ")
           end
           cookie_str.chomp!('; ')
 
@@ -78,6 +81,7 @@ module U3d
           end
 
           raise 'Unexpected result' unless res.is_a? Net::HTTPRedirection
+
           # It should be a redirection to the forum to perform authentication
           uri = URI(res['location'])
           UI.verbose "Redirecting to #{uri}"
@@ -90,50 +94,51 @@ module U3d
           raise 'Unable to establish a session with Unity forum' unless res.is_a? Net::HTTPRedirection
 
           UI.verbose "Found cookie_str #{cookie_str}"
-          cookie_str << '; ' + res['set-cookie'].gsub(/\s+/, '').split(';', 2)[0]
+          cookie_str << ("; #{res['set-cookie'].gsub(/\s+/, '').split(';', 2)[0]}")
         end
       end
       UI.verbose "Found @cookie #{cookie_str}"
       @cookie = cookie_str
     end
   end
+
   # Takes care of fectching versions and version list
   module UnityVersions
     #####################################################
     # @!group URLS: Locations to fetch information from
     #####################################################
     # URL for the forum thread listing all the Linux releases
-    UNITY_LINUX_DOWNLOADS = 'https://forum.unity.com/threads/unity-on-linux-release-notes-and-known-issues.350256/'.freeze
+    UNITY_LINUX_DOWNLOADS = 'https://forum.unity.com/threads/unity-on-linux-release-notes-and-known-issues.350256/'
     # URL for the main releases for Windows and Macintosh
-    UNITY_DOWNLOADS = 'https://unity3d.com/get-unity/download/archive'.freeze
+    UNITY_DOWNLOADS = 'https://unity3d.com/get-unity/download/archive'
     # URL for the LTS releases for Windows and Macintosh
-    UNITY_LTSES = 'https://unity3d.com/unity/qa/lts-releases'.freeze
+    UNITY_LTSES = 'https://unity3d.com/unity/qa/lts-releases'
     # URL for the patch releases for Windows and Macintosh
-    UNITY_PATCHES = 'https://unity3d.com/unity/qa/patch-releases'.freeze
+    UNITY_PATCHES = 'https://unity3d.com/unity/qa/patch-releases'
     # URL for the beta releases list, they need to be accessed after
-    UNITY_BETAS = 'https://unity3d.com/unity/beta/archive'.freeze
+    UNITY_BETAS = 'https://unity3d.com/unity/beta/archive'
     # URL for a specific beta, takes into parameter a version string (%s)
-    UNITY_BETA_URL = 'https://unity3d.com/unity/beta/unity%<version>s'.freeze
+    UNITY_BETA_URL = 'https://unity3d.com/unity/beta/unity%<version>s'
     # URL for latest releases listing (since Unity 2017.1.5f1), takes into parameter os (windows => win32, mac => darwin)
-    UNITY_LATEST_JSON_URL = 'https://public-cdn.cloud.unity3d.com/hub/prod/releases-%<os>s.json'.freeze
+    UNITY_LATEST_JSON_URL = 'https://public-cdn.cloud.unity3d.com/hub/prod/releases-%<os>s.json'
 
     #####################################################
     # @!group REGEX: expressions to interpret data
     #####################################################
     # Captures a version and its base url
-    LINUX_DOWNLOAD = %r{['"](https?:\/\/[\w/\.-]+/[0-9a-f\+]{12,13}\/)(.\/)?UnitySetup-(\d+\.\d+\.\d+\w\d+)['"]}
+    LINUX_DOWNLOAD = %r{['"](https?://[\w/.-]+/[0-9a-f+]{12,13}/)(./)?UnitySetup-(\d+\.\d+\.\d+\w\d+)['"]}.freeze
 
-    MAC_WIN_SHADERS = %r{"(https?://[\w/\.-]+/[0-9a-f\+]{12,13}/)builtin_shaders-(\d+\.\d+\.\d+\w\d+)\.?\w+"}
-    LINUX_INSTALLER = %r{(https?://[\w/\.-]+/[0-9a-f\+]{12,13}/)LinuxEditorInstaller/Unity.tar.xz}
+    MAC_WIN_SHADERS = %r{"(https?://[\w/.-]+/[0-9a-f+]{12,13}/)builtin_shaders-(\d+\.\d+\.\d+\w\d+)\.?\w+"}.freeze
+    LINUX_INSTALLER = %r{(https?://[\w/.-]+/[0-9a-f+]{12,13}/)LinuxEditorInstaller/Unity.tar.xz}.freeze
 
-    LINUX_DOWNLOAD_DATED = %r{"(https?://[\w/\._-]+/unity\-editor\-installer\-(\d+\.\d+\.\d+\w\d+).*\.sh)"}
-    LINUX_DOWNLOAD_RECENT_PAGE = %r{"(https?://beta\.unity3d\.com/download/[a-zA-Z0-9/\.\+]+/public_download\.html)"}
-    LINUX_DOWNLOAD_RECENT_FILE = %r{'(https?://beta\.unity3d\.com/download/[a-zA-Z0-9/\.\+]+/unity\-editor\-installer\-(\d+\.\d+\.\d+(?:x)?\w\d+).*\.sh)'}
+    LINUX_DOWNLOAD_DATED = %r{"(https?://[\w/._-]+/unity-editor-installer-(\d+\.\d+\.\d+\w\d+).*\.sh)"}.freeze
+    LINUX_DOWNLOAD_RECENT_PAGE = %r{"(https?://beta\.unity3d\.com/download/[a-zA-Z0-9/.+]+/public_download\.html)"}.freeze
+    LINUX_DOWNLOAD_RECENT_FILE = %r{'(https?://beta\.unity3d\.com/download/[a-zA-Z0-9/.+]+/unity-editor-installer-(\d+\.\d+\.\d+(?:x)?\w\d+).*\.sh)'}.freeze
     # Captures a beta version in html page
-    UNITY_BETAVERSION_REGEX = %r{\/unity\/beta\/unity(\d+\.\d+\.\d+\w\d+)"}
-    UNITY_EXTRA_DOWNLOAD_REGEX = %r{"(https?:\/\/[\w\/.-]+\.unity3d\.com\/(\w+))\/[a-zA-Z\/.-]+\/download.html"}
+    UNITY_BETAVERSION_REGEX = %r{/unity/beta/unity(\d+\.\d+\.\d+\w\d+)"}.freeze
+    UNITY_EXTRA_DOWNLOAD_REGEX = %r{"(https?://[\w/.-]+\.unity3d\.com/(\w+))/[a-zA-Z/.-]+/download.html"}.freeze
     # For the latest releases fetched from json
-    UNITY_LATEST_JSON = %r{(https?://[\w/\.-]+/[0-9a-f\+]{12,13}/)}
+    UNITY_LATEST_JSON = %r{(https?://[\w/.-]+/[0-9a-f+]{12,13}/)}.freeze
 
     class << self
       def list_available(os: nil)
@@ -191,7 +196,7 @@ module U3d
     end
 
     class LinuxVersions
-      JSON_OS = 'linux'.freeze
+      JSON_OS = 'linux'
 
       @unity_forums = U3d::UnityForums.new
       class << self
@@ -300,7 +305,7 @@ module U3d
     end
 
     class MacVersions
-      JSON_OS = 'darwin'.freeze
+      JSON_OS = 'darwin'
 
       class << self
         def list_available
@@ -312,7 +317,7 @@ module U3d
     end
 
     class WindowsVersions
-      JSON_OS = 'win32'.freeze
+      JSON_OS = 'win32'
 
       class << self
         def list_available