Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package yast2-packager for openSUSE:Factory 
checked in at 2025-07-20 15:28:23
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-packager (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-packager.new.8875 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-packager"

Sun Jul 20 15:28:23 2025 rev:446 rq:1293883 version:5.0.8

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-packager/yast2-packager.changes    
2025-01-07 20:51:26.920023261 +0100
+++ /work/SRC/openSUSE:Factory/.yast2-packager.new.8875/yast2-packager.changes  
2025-07-20 15:28:46.223320517 +0200
@@ -1,0 +2,14 @@
+Tue Jul 15 14:57:19 UTC 2025 - Martin Vidner <mvid...@suse.com>
+
+- Fix Internal Error: Encoding::CompatibilityError when
+  adding SLE-HA as add-on product (bsc#1245555)
+- 5.0.8
+
+-------------------------------------------------------------------
+Wed Jun  4 11:41:30 UTC 2025 - Stefan Hundhammer <shundham...@suse.com>
+
+- Prevent a crash if a repo doesn't have a baseurl, but possibly
+  a mirrorlist instead (bsc#1244040)
+- 5.0.7
+
+-------------------------------------------------------------------
@@ -445 +458,0 @@
-Fri Feb 19 14:07:15 UTC 2021 - Josef Reidinger <jreidin...@suse.com>
@@ -446,0 +460 @@
+Fri Feb 19 14:07:15 UTC 2021 - Josef Reidinger <jreidin...@suse.com>
@@ -2258 +2271,0 @@
-

Old:
----
  yast2-packager-5.0.6.tar.bz2

New:
----
  yast2-packager-5.0.8.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-packager.spec ++++++
--- /var/tmp/diff_new_pack.YHo2lp/_old  2025-07-20 15:28:47.043354451 +0200
+++ /var/tmp/diff_new_pack.YHo2lp/_new  2025-07-20 15:28:47.047354617 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-packager
-Version:        5.0.6
+Version:        5.0.8
 Release:        0
 Summary:        YaST2 - Package Library
 License:        GPL-2.0-or-later

++++++ yast2-packager-5.0.6.tar.bz2 -> yast2-packager-5.0.8.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-packager-5.0.6/package/yast2-packager.changes 
new/yast2-packager-5.0.8/package/yast2-packager.changes
--- old/yast2-packager-5.0.6/package/yast2-packager.changes     2025-01-06 
12:42:04.000000000 +0100
+++ new/yast2-packager-5.0.8/package/yast2-packager.changes     2025-07-16 
20:49:42.000000000 +0200
@@ -1,4 +1,18 @@
 -------------------------------------------------------------------
+Tue Jul 15 14:57:19 UTC 2025 - Martin Vidner <mvid...@suse.com>
+
+- Fix Internal Error: Encoding::CompatibilityError when
+  adding SLE-HA as add-on product (bsc#1245555)
+- 5.0.8
+
+-------------------------------------------------------------------
+Wed Jun  4 11:41:30 UTC 2025 - Stefan Hundhammer <shundham...@suse.com>
+
+- Prevent a crash if a repo doesn't have a baseurl, but possibly
+  a mirrorlist instead (bsc#1244040)
+- 5.0.7
+
+-------------------------------------------------------------------
 Mon Jan  6 10:45:35 UTC 2025 - Martin Vidner <mvid...@suse.com>
 
 - Adjust test for ruby-3.4 (bsc#1235099)
@@ -442,8 +456,8 @@
   contributed by Sasi Olin (hel@lcp.world).
 - 4.3.15
 
-Fri Feb 19 14:07:15 UTC 2021 - Josef Reidinger <jreidin...@suse.com>
 -------------------------------------------------------------------
+Fri Feb 19 14:07:15 UTC 2021 - Josef Reidinger <jreidin...@suse.com>
 
 - Show correct number of downloaded packages in log (bsc#1180278)
 - Fix crash when installation proposal require pattern and such
@@ -2255,7 +2269,6 @@
 - 3.1.93
 
 -------------------------------------------------------------------
-
 Fri Apr  1 12:37:51 UTC 2016 - knut.anders...@suse.com
 
 - Added [Network Configuration] button to Source Dialog for
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-packager-5.0.6/package/yast2-packager.spec 
new/yast2-packager-5.0.8/package/yast2-packager.spec
--- old/yast2-packager-5.0.6/package/yast2-packager.spec        2025-01-06 
12:42:04.000000000 +0100
+++ new/yast2-packager-5.0.8/package/yast2-packager.spec        2025-07-16 
20:49:42.000000000 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-packager
-Version:        5.0.6
+Version:        5.0.8
 Release:        0
 Summary:        YaST2 - Package Library
 License:        GPL-2.0-or-later
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-packager-5.0.6/src/lib/y2packager/known_repositories.rb 
new/yast2-packager-5.0.8/src/lib/y2packager/known_repositories.rb
--- old/yast2-packager-5.0.6/src/lib/y2packager/known_repositories.rb   
2025-01-06 12:42:04.000000000 +0100
+++ new/yast2-packager-5.0.8/src/lib/y2packager/known_repositories.rb   
2025-07-16 20:49:42.000000000 +0200
@@ -95,6 +95,7 @@
       repo_ids = Yast::Pkg.SourceGetCurrent(true)
 
       urls = repo_ids.map { |r| Yast::Pkg.SourceGeneralData(r)["url"] }
+      urls.compact! # bsc#1244040: mirrorlist instead of baseurl
       urls.uniq!
       urls.sort
     end
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-packager-5.0.6/src/lib/y2packager/product_finder.rb 
new/yast2-packager-5.0.8/src/lib/y2packager/product_finder.rb
--- old/yast2-packager-5.0.6/src/lib/y2packager/product_finder.rb       
2025-01-06 12:42:04.000000000 +0100
+++ new/yast2-packager-5.0.8/src/lib/y2packager/product_finder.rb       
2025-07-16 20:49:42.000000000 +0200
@@ -106,6 +106,20 @@
       order
     end
 
+    # solvable.lookup_str, but make the result UTF-8
+    #
+    # SWIG bindings always set the string encoding to ASCII_8BIT (BINARY)
+    # https://github.com/swig/swig/issues/2035
+    # but combining those with UTF_8 strings such as non-English UI texts
+    # would raise Encoding::CompatibilityError
+    #
+    # See bsc#1245555 (here, libsolv, solv-ruby) and bsc#1096758 
(libstorage-ng)
+    def lookup_str(solvable, key)
+      s = solvable.lookup_str(key) || ""
+      s.force_encoding(Encoding::UTF_8)
+      s
+    end
+
     #
     # Evaluate the products
     #
@@ -140,9 +154,9 @@
         ret << RepoProductSpec.new(
           name:         product_name,
           media_name:   media_name,
-          display_name: product_solvable.lookup_str(Solv::SOLVABLE_SUMMARY) || 
"",
+          display_name: lookup_str(product_solvable, Solv::SOLVABLE_SUMMARY),
           base:         found_base_products.include?(product_name),
-          description:  
product_solvable.lookup_str(Solv::SOLVABLE_DESCRIPTION) || "",
+          description:  lookup_str(product_solvable, 
Solv::SOLVABLE_DESCRIPTION),
           depends_on:   find_dependencies(product_solvable, selected_base),
           order:        display_order(product_solvable),
           dir:          dir,
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-packager-5.0.6/test/known_repositories_test.rb 
new/yast2-packager-5.0.8/test/known_repositories_test.rb
--- old/yast2-packager-5.0.6/test/known_repositories_test.rb    2025-01-06 
12:42:04.000000000 +0100
+++ new/yast2-packager-5.0.8/test/known_repositories_test.rb    2025-07-16 
20:49:42.000000000 +0200
@@ -55,7 +55,7 @@
   end
 
   describe "#update" do
-    it "add the current repositories to the known repositories" do
+    it "adds the current repositories to the known repositories" do
       allow(subject).to receive(:repositories).and_return(repos)
 
       allow(Yast::Pkg).to 
receive(:SourceGetCurrent).with(true).and_return([source_id])
@@ -65,6 +65,17 @@
       subject.update
       expect(repos).to eq(["http://example.com/repo";, 
"http://new.example.com";])
     end
+
+    it "survives a repo without a baseurl (bsc#1244040)" do
+      allow(subject).to receive(:repositories).and_return(repos)
+
+      allow(Yast::Pkg).to 
receive(:SourceGetCurrent).with(true).and_return([source_id])
+      allow(Yast::Pkg).to receive(:SourceGeneralData).with(source_id)
+        .and_return("url" => nil)
+
+      subject.update
+      expect(repos).to eq(["http://example.com/repo";])
+    end
   end
 
 end

Reply via email to