Hello community, here is the log from the commit of package autoyast2 for openSUSE:Factory checked in at 2018-06-19 11:52:21 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/autoyast2 (Old) and /work/SRC/openSUSE:Factory/.autoyast2.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "autoyast2" Tue Jun 19 11:52:21 2018 rev:245 rq:616330 version:4.0.56 Changes: -------- --- /work/SRC/openSUSE:Factory/autoyast2/autoyast2.changes 2018-05-13 15:58:31.520647990 +0200 +++ /work/SRC/openSUSE:Factory/.autoyast2.new/autoyast2.changes 2018-06-19 11:52:26.667024679 +0200 @@ -1,0 +2,21 @@ +Tue Jun 12 15:16:20 CEST 2018 - [email protected] + +- Showing AutoYaST configuration file errors onetime only. + (needed for bnc#1095113) +- 4.0.56 + +------------------------------------------------------------------- +Tue Jun 5 10:15:02 CEST 2018 - [email protected] + +- Partition configuration: Do not ask for saving values if they have + not been changed at all. (bnc#1082556) +- 4.0.55 + +------------------------------------------------------------------- +Tue May 29 13:20:54 CEST 2018 - [email protected] + +- Using new libstorage-ng in order to handle "label" tag in URL. + E.G.: autoyast=label://my_home//autoinst.xml (bnc#1094533) +- 4.0.54 + +------------------------------------------------------------------- Old: ---- autoyast2-4.0.53.tar.bz2 New: ---- autoyast2-4.0.56.tar.bz2 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ autoyast2.spec ++++++ --- /var/tmp/diff_new_pack.7ks6AN/_old 2018-06-19 11:52:27.195005076 +0200 +++ /var/tmp/diff_new_pack.7ks6AN/_new 2018-06-19 11:52:27.199004927 +0200 @@ -22,7 +22,7 @@ %endif Name: autoyast2 -Version: 4.0.53 +Version: 4.0.56 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build ++++++ autoyast2-4.0.53.tar.bz2 -> autoyast2-4.0.56.tar.bz2 ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/autoyast2-4.0.53/package/autoyast2.changes new/autoyast2-4.0.56/package/autoyast2.changes --- old/autoyast2-4.0.53/package/autoyast2.changes 2018-05-11 14:51:28.000000000 +0200 +++ new/autoyast2-4.0.56/package/autoyast2.changes 2018-06-12 16:35:19.000000000 +0200 @@ -1,4 +1,25 @@ ------------------------------------------------------------------- +Tue Jun 12 15:16:20 CEST 2018 - [email protected] + +- Showing AutoYaST configuration file errors onetime only. + (needed for bnc#1095113) +- 4.0.56 + +------------------------------------------------------------------- +Tue Jun 5 10:15:02 CEST 2018 - [email protected] + +- Partition configuration: Do not ask for saving values if they have + not been changed at all. (bnc#1082556) +- 4.0.55 + +------------------------------------------------------------------- +Tue May 29 13:20:54 CEST 2018 - [email protected] + +- Using new libstorage-ng in order to handle "label" tag in URL. + E.G.: autoyast=label://my_home//autoinst.xml (bnc#1094533) +- 4.0.54 + +------------------------------------------------------------------- Fri May 11 08:11:09 UTC 2018 - [email protected] - Handle DASD or zFCP devices even when the profile is not in a diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/autoyast2-4.0.53/package/autoyast2.spec new/autoyast2-4.0.56/package/autoyast2.spec --- old/autoyast2-4.0.53/package/autoyast2.spec 2018-05-11 14:51:28.000000000 +0200 +++ new/autoyast2-4.0.56/package/autoyast2.spec 2018-06-12 16:35:19.000000000 +0200 @@ -22,7 +22,7 @@ %endif Name: autoyast2 -Version: 4.0.53 +Version: 4.0.56 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/autoyast2-4.0.53/src/include/autoinstall/DriveDialog.rb new/autoyast2-4.0.56/src/include/autoinstall/DriveDialog.rb --- old/autoyast2-4.0.53/src/include/autoinstall/DriveDialog.rb 2018-05-11 14:51:28.000000000 +0200 +++ new/autoyast2-4.0.56/src/include/autoinstall/DriveDialog.rb 2018-06-12 16:35:19.000000000 +0200 @@ -109,7 +109,7 @@ drive = AutoinstDrive.set( drive, "use", - UI.QueryWidget(Id(:cb_reuse), :Value) + symbol2string(UI.QueryWidget(Id(:cb_reuse), :Value)) ) end deep_copy(drive) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/autoyast2-4.0.53/src/modules/AutoInstall.rb new/autoyast2-4.0.56/src/modules/AutoInstall.rb --- old/autoyast2-4.0.53/src/modules/AutoInstall.rb 2018-05-11 14:51:28.000000000 +0200 +++ new/autoyast2-4.0.56/src/modules/AutoInstall.rb 2018-06-12 16:35:19.000000000 +0200 @@ -362,6 +362,10 @@ end presenter = Y2Autoinstallation::AutoinstIssuesPresenter.new(@issues_list) + + # Showing issues onetime only. + @issues_list = Y2Autoinstallation::AutoinstIssues::List.new + log.send(level, presenter.to_plain) if log_message return true unless display_message diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/autoyast2-4.0.53/src/modules/AutoinstPartPlan.rb new/autoyast2-4.0.56/src/modules/AutoinstPartPlan.rb --- old/autoyast2-4.0.53/src/modules/AutoinstPartPlan.rb 2018-05-11 14:51:28.000000000 +0200 +++ new/autoyast2-4.0.56/src/modules/AutoinstPartPlan.rb 2018-06-12 16:35:19.000000000 +0200 @@ -395,6 +395,10 @@ log.info("entering Import with #{settings.inspect}") # index settings @AutoPartPlan = settings.map.with_index { |d, i| d.merge("_id" => i) } + # set default value + @AutoPartPlan.each do |d| + d["initialize"] = false unless d.has_key?("initialize") + end true end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/autoyast2-4.0.53/src/modules/ProfileLocation.rb new/autoyast2-4.0.56/src/modules/ProfileLocation.rb --- old/autoyast2-4.0.53/src/modules/ProfileLocation.rb 2018-05-11 14:51:28.000000000 +0200 +++ new/autoyast2-4.0.56/src/modules/ProfileLocation.rb 2018-06-12 16:35:19.000000000 +0200 @@ -7,9 +7,12 @@ # # $Id$ require "yast" +require "y2storage" module Yast class ProfileLocationClass < Module + include Yast::Logger + def main Yast.import "UI" textdomain "autoinst" @@ -125,36 +128,22 @@ ) SCR.Write(path(".etc.install_inf"), nil) elsif AutoinstConfig.scheme == "label" - Builtins.y2milestone("searching label") - Builtins.foreach(Storage.GetTargetMap) do |device, v| - Builtins.y2milestone("looking on %1", device) - if Ops.get_string(v, "label", "") == AutoinstConfig.host - AutoinstConfig.scheme = "device" - AutoinstConfig.host = Builtins.substring(device, 5) - Builtins.y2milestone("found on %1", AutoinstConfig.host) - raise Break - end - Builtins.foreach(Ops.get_list(v, "partitions", [])) do |p| - if Ops.get_string(p, "label", "") == AutoinstConfig.host - AutoinstConfig.scheme = "device" - AutoinstConfig.host = Builtins.substring( - Ops.get_string(p, "device", device), - 5 - ) - Builtins.y2milestone("found on %1", AutoinstConfig.host) - raise Break - end - Builtins.y2milestone( - "not found on %1", - Ops.get_string(p, "device", "hm?") - ) - end - raise Break if AutoinstConfig.scheme == "device" + # autoyast=label://my_home//autoinst.xml in linuxrc: + # AY is searching for a partition with the label "my_home". This partition + # will be mounted and the autoinst.xml will be used for installation. + log.info("searching label #{AutoinstConfig.host}") + fs = Y2Storage::StorageManager.instance.probed.filesystems.find do |f| + f.label == AutoinstConfig.host end - if AutoinstConfig.scheme == "label" + if fs && fs.blk_devices.first + AutoinstConfig.scheme = "device" + AutoinstConfig.host = fs.blk_devices.first.basename + log.info("found on #{AutoinstConfig.host}") + else Report.Error(_("label not found while looking for autoyast profile")) end end + filename = basename(AutoinstConfig.filepath) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/autoyast2-4.0.53/test/AutoinstPartPlan_test.rb new/autoyast2-4.0.56/test/AutoinstPartPlan_test.rb --- old/autoyast2-4.0.53/test/AutoinstPartPlan_test.rb 2018-05-11 14:51:28.000000000 +0200 +++ new/autoyast2-4.0.56/test/AutoinstPartPlan_test.rb 2018-06-12 16:35:19.000000000 +0200 @@ -64,6 +64,28 @@ end end + describe "#Import" do + let(:sda) { { "device" => "/dev/sda", "initialize" => true } } + let(:sdb) { { "device" => "/dev/sdb" } } + let(:settings) { [sda, sdb] } + + before { subject.Import(settings) } + + context "\"initialize\" is not given" do + it "set default value for \"initialize\" to false" do + drive = subject.getDrive(1) + expect(drive["initialize"]).to eq(false) + end + end + + context "\"initialize\" is given" do + it "does not overwrite \"initialize\" with default setting" do + drive = subject.getDrive(0) + expect(drive["initialize"]).to eq(true) + end + end + end + describe "#Export" do let(:exported) { subject.Export }
