yast-rake 0.2.40 → 0.2.44

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2642c0e7364635725ef46461b0d6c816fd1ebca543b87a813292cb8b6092a392
-  data.tar.gz: de07ea039866173b29bf17d3e505ab9d3635ff4a7b50e65c4c9ee4f8da4fdec4
+  metadata.gz: b131d89f00dea687fc5d44232d1cf6ce503659ca2ca810d63b17169d69eb5e00
+  data.tar.gz: 1e2d455ac8ca10a363b7bf89585f8bf0927dd06c94204d222a1384f421b4270d
 SHA512:
-  metadata.gz: cd3a1697577fb14c191261d5b953e123c8cf2052ddb58388d5836f6a0477b233903aab0f58663d723a9eb692d9e3319475e4f1db9878264288b39a9e1758eee5
-  data.tar.gz: b6153a3bfe3601cef2f3cfad9dfe899e80cbe11eb65563029a84c14a01fe9f8831fc0f8e1f270dd90975badd3abde483db0891f71ca619c5086ee653c9ef5ee1
+  metadata.gz: 5a83c6e94d862c23c0f04e84cdf68d48c0aa4b4f51caba4000a6f5106831097c38343b10a553fc83fd3355e37a9b719befab007656b3a1626b2364b76f8db35a
+  data.tar.gz: 13828bd3771752722aa0fe5eb0c7f8f305b6f447bf8f0310e71c5df33e4810d5f071fa2cdef39cbcdfc17a2eff2e9e4e12b2f37675aff1c6fbda0d5d4b1b9a24

data/VERSION

@@ -1 +1 @@
-0.2.40
+0.2.44

data/data/targets.yml

@@ -85,11 +85,21 @@
   obs_project: "Devel:YaST:SLE-15-SP3"
   obs_sr_project: "SUSE:SLE-15-SP3:Update"
   obs_target: "SUSE_SLE-15-SP3_Update"
+:sle15sp4:
+  obs_api: "https://api.suse.de/"
+  obs_project: "Devel:YaST:SLE-15-SP4"
+  obs_sr_project: "SUSE:SLE-15-SP4:Update"
+  obs_target: "SUSE_SLE-15-SP4_Update"
+:sle15sp5:
+  obs_api: "https://api.suse.de/"
+  obs_project: "Devel:YaST:SLE-15-SP5"
+  obs_sr_project: "SUSE:SLE-15-SP5:Update"
+  obs_target: "SUSE_SLE-15-SP5_Update"
 :sle_latest:
   obs_api: "https://api.suse.de/"
   obs_project: "Devel:YaST:Head"
-  obs_sr_project: "SUSE:SLE-15-SP3:GA"
-  obs_target: "SUSE_SLE-15-SP3_GA"
+  obs_sr_project: "SUSE:SLE-15-SP4:GA"
+  obs_target: "SUSE_SLE-15-SP4_GA"
 :factory:
   obs_project: "YaST:Head"
   obs_sr_project: "openSUSE:Factory"

data/lib/tasks/container_runner.rb

@@ -28,8 +28,7 @@ class ContainerRunner
   # @param client [String,nil] the client name, nil or empty string = find
   #   the client automatically
   def run(client)
-    image = find_image
-    container = GithubActions::Container.new(image)
+    container = find_container
     container.pull
     container.start
     container.copy_current_dir
@@ -44,16 +43,16 @@ private
 
   # find the Docker image to use in the container
   # @return [String] the image name
-  def find_image
+  def find_container
     # explicitly requested image
     image = ENV["DOCKER_IMAGE"]
-    return image if image && !image.empty?
+    return GithubActions::Container.new(image) if image && !image.empty?
 
     # scan the Docker images used in the GitHub Actions
-    images = workflow_images
-    return images.first if images.size == 1
+    containers = workflow_containers
+    return containers.first if containers.size == 1
 
-    if images.empty?
+    if containers.empty?
       error("No Docker image was found in the GitHub Actions")
       puts "Use DOCKER_IMAGE=<name> option for specifying the image name"
       abort
@@ -61,20 +60,36 @@ private
 
     # multiple images found
     error("Found multiple Docker images in the GitHub Actions:")
-    error(images.inspect)
+    error(containers.map { |c| [c.image, c.options] })
     puts "Use DOCKER_IMAGE=<name> option for specifying the image name"
+    puts "and DOCKER_OPTIONS=<options> option for specifying the extra Docker"
+    puts "command line parameters."
     abort
   end
 
   # extract the Docker images from the GitHub Actions,
   # the duplicates are removed
-  # @return [Array<String>] image names
-  def workflow_images
-    GithubActions::Workflow.read.each_with_object([]) do |workflow, images|
+  # @return [Array<GithubActions::Container>] image names
+  def workflow_containers
+    ret = GithubActions::Workflow.read.each_with_object([]) do |workflow, containers|
       workflow.jobs.each do |job|
-        container = job.container
-        images << container if container && !images.include?(container)
+        container_data = job.container
+
+        if container_data.is_a?(String)
+          containers << GithubActions::Container.new(container_data)
+        elsif container_data.is_a?(Hash)
+          # to_s converts missing options (nil) to empty options ("")
+          # to treat these as equal in comparison
+          containers << GithubActions::Container.new(
+            container_data["image"],
+            container_data["options"].to_s
+          )
+        else
+          abort "Unsupported container definition: #{container_data.inspect}"
+        end
       end
     end
+
+    ret.uniq { |c| [c.image, c.options] }
   end
 end

data/lib/tasks/github_actions/github_actions/container.rb

@@ -26,7 +26,7 @@ module GithubActions
   class Container
     include Colorizer
 
-    attr_reader :image, :container
+    attr_reader :image, :options, :container
 
     # the default timeout in seconds, maximum time for the running container,
     # after the time runs out the container is automatically stopped and removed
@@ -43,8 +43,10 @@ module GithubActions
 
     # constructor
     # @param image [String] name of the Docker image to use
-    def initialize(image)
+    # @param options [String, nil] extra docker options
+    def initialize(image, options = nil)
       @image = image
+      @options = options
     end
 
     # pull the Docker image, ensure that the latest version is used
@@ -73,7 +75,7 @@ module GithubActions
       end
 
       cmd = "docker create #{env_options(ENV_VARIABLES)} --rm --entrypoint " \
-        "#{run} #{image.shellescape} #{args}"
+        "#{run} #{options} #{ENV["DOCKER_OPTIONS"]} #{image.shellescape} #{args}"
 
       # contains the container ID
       @container = `#{cmd}`.chomp

data/lib/tasks/github_actions/github_actions/job_runner.rb

@@ -58,12 +58,29 @@ module GithubActions
 
     # pull the Docker image and start the container
     def start_container
-      # prefer the custom image if requested
-      @container = Container.new(image || job.container)
+      @container = find_container
       container.pull
       container.start
     end
 
+    # Get the container configuration
+    # @return [Container] container which should run the job
+    def find_container
+      # prefer the custom image if requested
+      image_name = if image
+        image
+      elsif job.container.is_a?(String)
+        job.container
+      elsif job.container.is_a?(Hash)
+        options = job.container["options"]
+        job.container["image"]
+      else
+        abort "Unsupported container definition: #{job.container.inspect}"
+      end
+
+      Container.new(image_name, options.to_s)
+    end
+
     # run a job step
     # @param step [GithubActions::Step] the step to run
     # @return [Boolean] `true` if the step succeeded, `false` otherwise

data/lib/tasks/rubocop.rake

@@ -15,12 +15,24 @@
 #
 #++
 
+# the default old version used
+OLD_RUBOCOP_VERSION = "0.41.2"
+
+def rubocop_version
+  return @rubocop_version if @rubocop_version
+
+  @rubocop_version = if File.read(".rubocop.yml").match(/rubocop-(\d+\.\d+\.\d+)/)
+    Regexp.last_match[1]
+  else
+    OLD_RUBOCOP_VERSION
+  end
+end
+
 def rubocop_bin
   return @rubocop_bin if @rubocop_bin
   return @rubocop_bin = ENV["RUBOCOP_BIN"] if ENV["RUBOCOP_BIN"]
 
-  version = File.read(".rubocop.yml").include?("rubocop-0.71.0") ? "0.71.0" : "0.41.2"
-  binary = `/usr/sbin/update-alternatives --list rubocop | grep '#{version}'`.strip
+  binary = `/usr/sbin/update-alternatives --list rubocop | grep '#{rubocop_version}'`.strip
   if !system("which #{binary}")
     raise "cannot find proper version of rubocop binary in " \
       "'/usr/sbin/update-alternatives --list rubocop'." \
@@ -32,18 +44,26 @@ end
 # run Rubocop in parallel
 # @param params [String] optional Rubocop parameters
 def run_rubocop(params = "")
-  # how it works:
-  # 1) get the list of inspected files by Rubocop
-  # 2) shuffle it randomly (better would be evenly distribute them according to
-  #    the analysis complexity but that is hard to evaluate and even simply
-  #    distributing by file size turned out to be ineffective and slower than
-  #    a simple random shuffling)
-  # 3) pass that as input for xargs
-  #    a) use -P with number of processors to run the commands in parallel
-  #    b) use -n to set the maximum number of files per process, this number
-  #       is computed to equally distribute the files across the workers
-  sh "#{rubocop_bin} -L | sort -R | xargs -P`nproc` -n$(expr `#{rubocop_bin} -L | wc -l` / " \
-    "`nproc` + 1) #{rubocop_bin} #{params}"
+  # newer Rubocop versions support the "-P" ("--parallel") option,
+  # but that is not compatible with the "-a" ("--auto-correct") option
+  if rubocop_version != OLD_RUBOCOP_VERSION && !params.to_s.match(/-a|--auto-correct/)
+    sh "#{rubocop_bin} -P #{params}"
+  else
+    # for older Rubocop or auto-correct mode manually start multiple instances in parallel
+    #
+    # how it works:
+    # 1) get the list of inspected files by Rubocop
+    # 2) shuffle it randomly (better would be evenly distribute them according to
+    #    the analysis complexity but that is hard to evaluate and even simply
+    #    distributing by file size turned out to be ineffective and slower than
+    #    a simple random shuffling)
+    # 3) pass that as input for xargs
+    #    a) use -P with number of processors to run the commands in parallel
+    #    b) use -n to set the maximum number of files per process, this number
+    #       is computed to equally distribute the files across the workers
+    sh "#{rubocop_bin} -L | sort -R | xargs -P`nproc` -n$(expr `#{rubocop_bin} -L | wc -l` / " \
+      "`nproc` + 1) #{rubocop_bin} #{params}"
+  end
 end
 
 namespace :check do

data/lib/tasks/server.rake

@@ -18,12 +18,16 @@
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 #++
 
-require_relative "../yast/tarball_server"
-
 # Rake task for running a source code web server,
 # designed for the `yupdate` script.
 desc "Start an HTTP server providing dynamically generated source code tarball"
 task :server, [:port] do |_task, args|
+  begin
+    require_relative "../yast/tarball_server"
+  rescue LoadError
+    abort "Webrick server is not installed, please install the webrick Ruby gem"
+  end
+
   server = Yast::TarballServer.new(args[:port])
 
   puts "Starting tarball webserver:"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: yast-rake
 version: !ruby/object:Gem::Version
-  version: 0.2.40
+  version: 0.2.44
 platform: ruby
 authors:
 - Josef Reidinger
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-03-29 00:00:00.000000000 Z
+date: 2022-01-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: packaging_rake_tasks
@@ -101,7 +101,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.6.2
+rubygems_version: 2.7.6.3
 signing_key: 
 specification_version: 4
 summary: Rake tasks providing basic work-flow for Yast development