Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package yast2 for openSUSE:Factory checked 
in at 2022-10-15 16:35:02
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2 (Old)
 and      /work/SRC/openSUSE:Factory/.yast2.new.2275 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2"

Sat Oct 15 16:35:02 2022 rev:542 rq:1010964 version:4.5.17

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2/yast2.changes      2022-10-10 
18:43:54.602784388 +0200
+++ /work/SRC/openSUSE:Factory/.yast2.new.2275/yast2.changes    2022-10-15 
16:36:23.630000059 +0200
@@ -1,0 +2,6 @@
+Fri Oct 14 14:54:58 UTC 2022 - Steffen Winterfeldt <snw...@suse.com>
+
+- remove #postprocess_url from RelURL class (jsc#SLE-22578, jsc#SLE-24584)
+- 4.5.17
+
+-------------------------------------------------------------------

Old:
----
  yast2-4.5.16.tar.bz2

New:
----
  yast2-4.5.17.tar.bz2

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

Other differences:
------------------
++++++ yast2.spec ++++++
--- /var/tmp/diff_new_pack.y7zIUK/_old  2022-10-15 16:36:24.326001731 +0200
+++ /var/tmp/diff_new_pack.y7zIUK/_new  2022-10-15 16:36:24.334001750 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2
-Version:        4.5.16
+Version:        4.5.17
 Release:        0
 
 Summary:        YaST2 Main Package

++++++ yast2-4.5.16.tar.bz2 -> yast2-4.5.17.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-4.5.16/library/general/src/lib/yast2/rel_url.rb 
new/yast2-4.5.17/library/general/src/lib/yast2/rel_url.rb
--- old/yast2-4.5.16/library/general/src/lib/yast2/rel_url.rb   2022-10-07 
11:03:03.000000000 +0200
+++ new/yast2-4.5.17/library/general/src/lib/yast2/rel_url.rb   2022-10-14 
17:31:35.000000000 +0200
@@ -104,7 +104,6 @@
         ret.path = absolute_path
       end
 
-      postprocess_url(ret)
       ret
     end
 
@@ -119,17 +118,5 @@
       url.path = File.join("/", url.host, url.path)
       url.host = nil
     end
-
-    # a reverse method to "preprocess_url", fix the URL if it is a "file://" 
URL
-    def postprocess_url(url)
-      return if url.scheme != "file" && !url.host.nil? && !url.host.empty?
-
-      path = url.path.sub(/\A\/+/, "").split("/")
-      url.host = path.shift
-
-      rest = File.join(path)
-      rest.prepend("/") unless rest.empty?
-      url.path = rest
-    end
   end
 end
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-4.5.16/library/general/test/yast2/rel_url_test.rb 
new/yast2-4.5.17/library/general/test/yast2/rel_url_test.rb
--- old/yast2-4.5.16/library/general/test/yast2/rel_url_test.rb 2022-10-07 
11:03:03.000000000 +0200
+++ new/yast2-4.5.17/library/general/test/yast2/rel_url_test.rb 2022-10-14 
17:31:35.000000000 +0200
@@ -64,7 +64,22 @@
 
     it "returns the relative URL if the base URL is empty" do
       relurl = Yast2::RelURL.new("", "relurl://test")
-      expect(relurl.absolute_url.to_s).to eq("relurl://test")
+      expect(relurl.absolute_url.to_s).to eq("relurl:/test")
+    end
+
+    it "relurl can have 1 slash (relurl:/)" do
+      relurl = Yast2::RelURL.new("", "relurl:/test")
+      expect(relurl.absolute_url.to_s).to eq("relurl:/test")
+    end
+
+    it "relurl can have 2 slashes (relurl://)" do
+      relurl = Yast2::RelURL.new("", "relurl://test")
+      expect(relurl.absolute_url.to_s).to eq("relurl:/test")
+    end
+
+    it "relurl can have 3 slashes (relurl:///)" do
+      relurl = Yast2::RelURL.new("", "relurl:///test")
+      expect(relurl.absolute_url.to_s).to eq("relurl:/test")
     end
 
     it "returns empty URL if both relative and base URLs are empty" do
@@ -179,9 +194,14 @@
       expect(relurl.absolute_url.to_s).to eq("ftp://example.com/test";)
     end
 
-    it "works with cd:// URL" do
-      relurl = Yast2::RelURL.new("cd://", "relurl://test")
-      expect(relurl.absolute_url.to_s).to eq("cd://test")
+    it "works with cd:/ URL" do
+      relurl = Yast2::RelURL.new("cd:/", "relurl://test")
+      expect(relurl.absolute_url.to_s).to eq("cd:/test")
+    end
+
+    it "corrects cd:// URL to cd:/" do
+      relurl = Yast2::RelURL.new("cd:/", "relurl://test")
+      expect(relurl.absolute_url.to_s).to eq("cd:/test")
     end
 
     it "works with cifs:// URL" do
@@ -194,19 +214,24 @@
       expect(relurl.absolute_url.to_s).to eq("nfs://server/export/path/test")
     end
 
-    it "works with file:// base URL" do
+    it "works with file:/ base URL" do
+      relurl = Yast2::RelURL.new("file:/foo/bar", "relurl://test")
+      expect(relurl.absolute_url.to_s).to eq("file:///foo/bar/test")
+    end
+
+    it "corrects file:// base URL to file:///" do
       relurl = Yast2::RelURL.new("file://foo/bar", "relurl://test")
-      expect(relurl.absolute_url.to_s).to eq("file://foo/bar/test")
+      expect(relurl.absolute_url.to_s).to eq("file:///foo/bar/test")
     end
 
     it "works with file:/// base URL" do
       relurl = Yast2::RelURL.new("file:///", "relurl://test")
-      expect(relurl.absolute_url.to_s).to eq("file://test")
+      expect(relurl.absolute_url.to_s).to eq("file:///test")
     end
 
-    it "goes up with file:// base URL properly" do
-      relurl = Yast2::RelURL.new("file://foo/bar", "relurl://../../test")
-      expect(relurl.absolute_url.to_s).to eq("file://test")
+    it "follows '..' with file:/ base URL properly" do
+      relurl = Yast2::RelURL.new("file:/foo/bar", "relurl://../../test")
+      expect(relurl.absolute_url.to_s).to eq("file:///test")
     end
 
     it "adds the requested path to the absolute URL" do
@@ -229,7 +254,7 @@
       # might not be exactly what you would expect as the result but this is a 
corner
       # case, do not overengineer the code, the most important fact is that it 
does
       # not crash and the result is a valid file path
-      expect(relurl.absolute_url("foo/bar").to_s).to eq("//foo/bar")
+      expect(relurl.absolute_url("foo/bar").to_s).to eq("/foo/bar")
     end
   end
 
@@ -238,7 +263,7 @@
       allow(Yast::InstURL).to receive(:installInf2Url).and_return(inst_url)
     end
 
-    let(:rel_url) { "relurl://test" }
+    let(:rel_url) { "relurl:/test" }
     subject { Yast2::RelURL.from_installation_repository(rel_url) }
 
     context "during installation" do
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-4.5.16/package/yast2.changes 
new/yast2-4.5.17/package/yast2.changes
--- old/yast2-4.5.16/package/yast2.changes      2022-10-07 11:03:03.000000000 
+0200
+++ new/yast2-4.5.17/package/yast2.changes      2022-10-14 17:31:35.000000000 
+0200
@@ -1,4 +1,10 @@
 -------------------------------------------------------------------
+Fri Oct 14 14:54:58 UTC 2022 - Steffen Winterfeldt <snw...@suse.com>
+
+- remove #postprocess_url from RelURL class (jsc#SLE-22578, jsc#SLE-24584)
+- 4.5.17
+
+-------------------------------------------------------------------
 Thu Oct  6 13:48:28 UTC 2022 - Josef Reidinger <jreidin...@suse.com>
 
 - add Yast::ReducedRecorder for Cheetah to filter out certain streams to
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-4.5.16/package/yast2.spec 
new/yast2-4.5.17/package/yast2.spec
--- old/yast2-4.5.16/package/yast2.spec 2022-10-07 11:03:03.000000000 +0200
+++ new/yast2-4.5.17/package/yast2.spec 2022-10-14 17:31:35.000000000 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2
-Version:        4.5.16
+Version:        4.5.17
 
 Release:        0
 Summary:        YaST2 Main Package

Reply via email to