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-02-17 23:40:02 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/yast2 (Old) and /work/SRC/openSUSE:Factory/.yast2.new.1958 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "yast2" Thu Feb 17 23:40:02 2022 rev:528 rq:955663 version:4.4.45 Changes: -------- --- /work/SRC/openSUSE:Factory/yast2/yast2.changes 2022-02-01 14:02:42.236176834 +0100 +++ /work/SRC/openSUSE:Factory/.yast2.new.1958/yast2.changes 2022-02-17 23:40:58.175700695 +0100 @@ -1,0 +2,14 @@ +Thu Feb 17 13:25:33 UTC 2022 - Steffen Winterfeldt <snw...@suse.com> + +- do not strip surrounding white space in CDATA XML elements (bsc#1195910) +- 4.4.45 + +------------------------------------------------------------------- +Wed Feb 16 15:17:14 UTC 2022 - Ladislav Slez??k <lsle...@suse.cz> + +- Keep the user defined $Y2STYLE and $XCURSOR_THEME environment + variables, allow changing the installer theme via these + environment variables (related to jsc#SLE-20547) +- 4.4.44 + +------------------------------------------------------------------- Old: ---- yast2-4.4.43.tar.bz2 New: ---- yast2-4.4.45.tar.bz2 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ yast2.spec ++++++ --- /var/tmp/diff_new_pack.jXSaBF/_old 2022-02-17 23:40:58.839700689 +0100 +++ /var/tmp/diff_new_pack.jXSaBF/_new 2022-02-17 23:40:58.843700689 +0100 @@ -17,7 +17,7 @@ Name: yast2 -Version: 4.4.43 +Version: 4.4.45 Release: 0 Summary: YaST2 Main Package ++++++ yast2-4.4.43.tar.bz2 -> yast2-4.4.45.tar.bz2 ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-4.4.43/library/xml/src/modules/XML.rb new/yast2-4.4.45/library/xml/src/modules/XML.rb --- old/yast2-4.4.43/library/xml/src/modules/XML.rb 2022-01-28 16:18:39.000000000 +0100 +++ new/yast2-4.4.45/library/xml/src/modules/XML.rb 2022-02-17 16:28:49.000000000 +0100 @@ -247,7 +247,13 @@ children = children.select(&:element?) # we need just direct text under node. Can be splitted with another elements # but remove whitespace only text - text = node.xpath("text()").text.strip + # + # Do NOT strip surrounding white space in CDATA blocks (bsc#1195910). + # This restores the behavior prior to SLE15-SP3. + # + # See also #add_element. + text_nodes = node.xpath("text()") + text = text_nodes.map { |n| n.cdata? ? n.content : n.content.strip }.join type = fetch_type(text, children, node) @@ -301,7 +307,13 @@ element = Nokogiri::XML::Node.new(key, doc) case value when ::String - element.content = value + # Write CDATA block if string has surrounding white space. This matches + # the stripping in #parse_node. + if value.match?(/\A\s|\s\z/) + element.add_child(doc.create_cdata(value)) + else + element.content = value + end when ::Integer element[type_attr] = "integer" element.content = value.to_s diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-4.4.43/library/xml/test/xml_test.rb new/yast2-4.4.45/library/xml/test/xml_test.rb --- old/yast2-4.4.43/library/xml/test/xml_test.rb 2022-01-28 16:18:39.000000000 +0100 +++ new/yast2-4.4.45/library/xml/test/xml_test.rb 2022-02-17 16:28:49.000000000 +0100 @@ -85,6 +85,30 @@ expect(subject.YCPToXMLString("test", input)).to eq expected end + + it "creates CDATA element if string starts with spaces" do + input = { "test" => " test", "lest" => "\nlest" } + expected = "<?xml version=\"1.0\"?>\n" \ + "<!DOCTYPE test SYSTEM \"just_testing.dtd\">\n" \ + "<test>\n" \ + " <lest><![CDATA[\nlest]]></lest>\n" \ + " <test><![CDATA[ test]]></test>\n" \ + "</test>\n" + + expect(subject.YCPToXMLString("test", input)).to eq expected + end + + it "creates CDATA element if string ends with spaces" do + input = { "test" => "test ", "lest" => "lest\n" } + expected = "<?xml version=\"1.0\"?>\n" \ + "<!DOCTYPE test SYSTEM \"just_testing.dtd\">\n" \ + "<test>\n" \ + " <lest><![CDATA[lest\n]]></lest>\n" \ + " <test><![CDATA[test ]]></test>\n" \ + "</test>\n" + + expect(subject.YCPToXMLString("test", input)).to eq expected + end end context "for hash" do @@ -283,6 +307,52 @@ expect(subject.XMLToYCPString(input)).to eq expected end + + it "strips spaces at the end of strings" do + input = "<?xml version=\"1.0\"?>\n" \ + "<test xmlns=\"http://www.suse.com/1.0/yast2ns\" xmlns:config=\"http://www.suse.com/1.0/configns\">\n" \ + " <test>foo </test>\n" \ + " <lest>bar\n" \ + " </lest>\n" \ + "</test>\n" + expected = { "test" => "foo", "lest" => "bar" } + + expect(subject.XMLToYCPString(input)).to eq expected + end + + it "preserves spaces at the end of CDATA elements" do + input = "<?xml version=\"1.0\"?>\n" \ + "<test xmlns=\"http://www.suse.com/1.0/yast2ns\" xmlns:config=\"http://www.suse.com/1.0/configns\">\n" \ + " <test><![CDATA[foo ]]></test>\n" \ + " <lest><![CDATA[bar\n]]></lest>\n" \ + "</test>\n" + expected = { "test" => "foo ", "lest" => "bar\n" } + + expect(subject.XMLToYCPString(input)).to eq expected + end + + it "strips spaces at the start of strings" do + input = "<?xml version=\"1.0\"?>\n" \ + "<test xmlns=\"http://www.suse.com/1.0/yast2ns\" xmlns:config=\"http://www.suse.com/1.0/configns\">\n" \ + " <test> foo</test>\n" \ + " <lest>\nbar" \ + " </lest>\n" \ + "</test>\n" + expected = { "test" => "foo", "lest" => "bar" } + + expect(subject.XMLToYCPString(input)).to eq expected + end + + it "preserves spaces at the start of CDATA elements" do + input = "<?xml version=\"1.0\"?>\n" \ + "<test xmlns=\"http://www.suse.com/1.0/yast2ns\" xmlns:config=\"http://www.suse.com/1.0/configns\">\n" \ + " <test><![CDATA[ foo]]></test>\n" \ + " <lest><![CDATA[\nbar]]></lest>\n" \ + "</test>\n" + expected = { "test" => " foo", "lest" => "\nbar" } + + expect(subject.XMLToYCPString(input)).to eq expected + end it "returns integer for xml element with type=\"integer\"" do input = "<?xml version=\"1.0\"?>\n" \ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-4.4.43/package/yast2.changes new/yast2-4.4.45/package/yast2.changes --- old/yast2-4.4.43/package/yast2.changes 2022-01-28 16:18:39.000000000 +0100 +++ new/yast2-4.4.45/package/yast2.changes 2022-02-17 16:28:49.000000000 +0100 @@ -1,4 +1,18 @@ ------------------------------------------------------------------- +Thu Feb 17 13:25:33 UTC 2022 - Steffen Winterfeldt <snw...@suse.com> + +- do not strip surrounding white space in CDATA XML elements (bsc#1195910) +- 4.4.45 + +------------------------------------------------------------------- +Wed Feb 16 15:17:14 UTC 2022 - Ladislav Slez??k <lsle...@suse.cz> + +- Keep the user defined $Y2STYLE and $XCURSOR_THEME environment + variables, allow changing the installer theme via these + environment variables (related to jsc#SLE-20547) +- 4.4.44 + +------------------------------------------------------------------- Fri Jan 28 14:03:04 UTC 2022 - Josef Reidinger <jreidin...@suse.com> - ProductFeatures: add boot timeout option (jsc#SLE-22667) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-4.4.43/package/yast2.spec new/yast2-4.4.45/package/yast2.spec --- old/yast2-4.4.43/package/yast2.spec 2022-01-28 16:18:39.000000000 +0100 +++ new/yast2-4.4.45/package/yast2.spec 2022-02-17 16:28:49.000000000 +0100 @@ -17,7 +17,7 @@ Name: yast2 -Version: 4.4.43 +Version: 4.4.45 Release: 0 Summary: YaST2 Main Package diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-4.4.43/scripts/yast2-funcs new/yast2-4.4.45/scripts/yast2-funcs --- old/yast2-4.4.43/scripts/yast2-funcs 2022-01-28 16:18:39.000000000 +0100 +++ new/yast2-4.4.45/scripts/yast2-funcs 2022-02-17 16:28:49.000000000 +0100 @@ -129,8 +129,12 @@ function set_inst_qt_env() { - export XCURSOR_THEME="DMZ-White" - export Y2STYLE="installation.qss" + # default values + : "${Y2STYLE=installation.qss}" + : "${XCURSOR_THEME=DMZ-White}" + + export XCURSOR_THEME + export Y2STYLE export QT_HOME_DIR="/tmp/.qt/" if [ "$Screenmode" != "" ] && [ "$Screenmode" != "default" ]; then export Y2ALTSTYLE="$Screenmode"