Merge pull request #1130 from yast/self_update_fix_SP6

[SLE15-SP6] Fixed comparing versions in the self-update version check

Author: lslezak

package/yast2-installation.changes

@@ -1,3 +1,15 @@
+-------------------------------------------------------------------
+Tue Dec 17 15:05:22 UTC 2024 - Ladislav Slezák <[email protected]>
+
+- Self-update fix (bsc#1234661)
+  - Properly compare the package versions from the inst-sys and
+    the self-update repository
+  - Remove the architecture suffix when reading the package version
+    from the inst-sys (the /.packages.root file)
+  - This fixes false alarm about incorrect self-update repository
+    when the installation is booted from PXE using the updated
+    tftpboot-installation-* packages (not the GA version)
+
 -------------------------------------------------------------------
 Tue Aug 27 09:06:22 UTC 2024 - Stefan Hundhammer <[email protected]>
 

package/yast2-installation.spec

@@ -16,7 +16,7 @@
 #
 
 Name:           yast2-installation
-Version:        4.6.13
+Version:        4.6.14
 Release:        0
 Summary:        YaST2 - Installation Parts
 License:        GPL-2.0-only

src/lib/installation/instsys_packages.rb

@@ -27,6 +27,9 @@ def self.read(file = "/.packages.root")
         name, version = /^(\S+) \[(\S+)\]/.match(line)[1, 2]
         next unless name && version
 
+        # remove the architecture suffix
+        version.sub!(/\.(noarch|aarch64|i[3-6]86|ppc64|ppc64le|s390x?|x86_64)$/, "")
+
         # nil repository ID
         packages << Y2Packager::Package.new(name, nil, version)
       end

test/instsys_packages_test.rb

@@ -29,9 +29,15 @@
 
     it "reads the package versions" do
       pkgs = Installation::InstsysPackages.read(test_file)
+
       yast2 = pkgs.find { |p| p.name == "yast2" }
+      yast2_add_on = pkgs.find { |p| p.name == "yast2-add-on" }
+
+      # no "x86_64" suffix
+      expect(yast2.version).to eq("4.2.67-1.7")
 
-      expect(yast2.version).to eq("4.2.67-1.7.x86_64")
+      # no "noarch" suffix
+      expect(yast2_add_on.version).to eq("4.2.15-1.30")
     end
   end
 end