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