Hello community, here is the log from the commit of package yast2-storage for openSUSE:Factory checked in at 2016-02-09 13:31:14 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/yast2-storage (Old) and /work/SRC/openSUSE:Factory/.yast2-storage.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "yast2-storage" Changes: -------- --- /work/SRC/openSUSE:Factory/yast2-storage/yast2-storage.changes 2015-12-27 01:58:22.000000000 +0100 +++ /work/SRC/openSUSE:Factory/.yast2-storage.new/yast2-storage.changes 2016-02-09 13:31:15.000000000 +0100 @@ -0,0 +1,15 @@ +------------------------------------------------------------------- +Thu Feb 5 10:21:35 UTC 2016 - [email protected] + +- Fixed expert partioner to only offer the option to enable Btrfs + snapshots for the root (/) filesystem (bsc#944252) +- Fixed expert partitioner to not forget fstab options when dealing + with Btrfs partitions (bsc#954691) +- 3.1.75 + +-------------------------------------------------------------------- +Thu Feb 4 11:45:44 UTC 2016 - [email protected] + +- Simplify encrypted home proposal as installer no longer allows + network based user authentication (FATE#314695) + Old: ---- yast2-storage-3.1.74.tar.bz2 New: ---- yast2-storage-3.1.75.tar.bz2 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ yast2-storage.spec ++++++ --- /var/tmp/diff_new_pack.x4l5e8/_old 2016-02-09 13:31:16.000000000 +0100 +++ /var/tmp/diff_new_pack.x4l5e8/_new 2016-02-09 13:31:16.000000000 +0100 @@ -1,7 +1,7 @@ # # spec file for package yast2-storage # -# Copyright (c) 2015 SUSE LINUX GmbH, Nuernberg, Germany. +# Copyright (c) 2016 SUSE LINUX GmbH, Nuernberg, Germany. # # All modifications and additions to the file contributed by third parties # remain the property of their copyright owners, unless otherwise agreed @@ -17,7 +17,7 @@ Name: yast2-storage -Version: 3.1.74 +Version: 3.1.75 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build ++++++ yast2-storage-3.1.74.tar.bz2 -> yast2-storage-3.1.75.tar.bz2 ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-storage-3.1.74/package/yast2-storage.changes new/yast2-storage-3.1.75/package/yast2-storage.changes --- old/yast2-storage-3.1.74/package/yast2-storage.changes 2015-12-22 14:58:11.000000000 +0100 +++ new/yast2-storage-3.1.75/package/yast2-storage.changes 2016-02-05 11:13:09.000000000 +0100 @@ -1,3 +1,18 @@ +------------------------------------------------------------------- +Thu Feb 5 10:21:35 UTC 2016 - [email protected] + +- Fixed expert partioner to only offer the option to enable Btrfs + snapshots for the root (/) filesystem (bsc#944252) +- Fixed expert partitioner to not forget fstab options when dealing + with Btrfs partitions (bsc#954691) +- 3.1.75 + +-------------------------------------------------------------------- +Thu Feb 4 11:45:44 UTC 2016 - [email protected] + +- Simplify encrypted home proposal as installer no longer allows + network based user authentication (FATE#314695) + -------------------------------------------------------------------- Wed Dec 17 15:15:29 UTC 2015 - [email protected] diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-storage-3.1.74/package/yast2-storage.spec new/yast2-storage-3.1.75/package/yast2-storage.spec --- old/yast2-storage-3.1.74/package/yast2-storage.spec 2015-12-22 14:58:11.000000000 +0100 +++ new/yast2-storage-3.1.75/package/yast2-storage.spec 2016-02-05 11:13:09.000000000 +0100 @@ -17,7 +17,7 @@ Name: yast2-storage -Version: 3.1.74 +Version: 3.1.75 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-storage-3.1.74/src/clients/partitions_proposal.rb new/yast2-storage-3.1.75/src/clients/partitions_proposal.rb --- old/yast2-storage-3.1.74/src/clients/partitions_proposal.rb 2015-12-22 14:58:11.000000000 +0100 +++ new/yast2-storage-3.1.75/src/clients/partitions_proposal.rb 2016-02-05 11:13:09.000000000 +0100 @@ -232,12 +232,6 @@ if !Builtins.isempty(part) && !Ops.get_boolean(part, "format", false) Builtins.y2milestone("/home partition will not be formatted") - Yast.import "UsersSimple" - if UsersSimple.AfterAuth != "users" - Builtins.y2milestone("non-local user authentication") - return true - end - device = Ops.get_string(part, "device", "") resize_info = {} content_info = {} diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-storage-3.1.74/src/include/partitioning/custom_part_dialogs.rb new/yast2-storage-3.1.75/src/include/partitioning/custom_part_dialogs.rb --- old/yast2-storage-3.1.74/src/include/partitioning/custom_part_dialogs.rb 2015-12-22 14:58:11.000000000 +0100 +++ new/yast2-storage-3.1.75/src/include/partitioning/custom_part_dialogs.rb 2016-02-05 11:13:09.000000000 +0100 @@ -1533,11 +1533,11 @@ # Dialog: Subvolume handling - # @parm old map with original partition - # @parm new map with changes filled in - def SubvolHandling(old, new) - old = deep_copy(old) - new = deep_copy(new) + # @parm old_partition map with original partition + # @parm new_partition map with changes filled in + def SubvolHandling(old_partition, new_partition) + old_partition = deep_copy(old_partition) + new_partition = deep_copy(new_partition) # help text, richtext format helptext = _( @@ -1550,10 +1550,10 @@ ) end - old_subvol = Ops.get_list(new, "subvol", []) - old_userdata = Ops.get_map(new, "userdata", {}) + initial_subvol = Ops.get_list(new_partition, "subvol", []) + initial_userdata = Ops.get_map(new_partition, "userdata", {}) - items = SubvolNames(new) + items = SubvolNames(new_partition) contents = VBox( # label text @@ -1582,7 +1582,7 @@ ) ) - if Mode.installation() + if Mode.installation() && snapshots_supported?(new_partition) contents = Builtins.add(contents, VSpacing(0.5)) contents = Builtins.add(contents, Left( @@ -1614,7 +1614,7 @@ ) ) - UI.ChangeWidget(Id(:snapshots), :Value, old_userdata["/"] == "snapshots") + UI.ChangeWidget(Id(:snapshots), :Value, initial_userdata["/"] == "snapshots") UI.ChangeWidget(:help, :HelpText, helptext) @@ -1631,12 +1631,12 @@ Builtins.y2milestone("SubvolHandling remove path:%1", pth) Builtins.y2milestone( "SubvolHandling remove subvol:%1", - Ops.get_list(new, "subvol", []) + Ops.get_list(new_partition, "subvol", []) ) Ops.set( - new, + new_partition, "subvol", - Builtins.maplist(Ops.get_list(new, "subvol", [])) do |p| + Builtins.maplist(Ops.get_list(new_partition, "subvol", [])) do |p| if Ops.get_string(p, "name", "") == pth Ops.set(p, "delete", true) p = Builtins.remove(p, "create") @@ -1646,9 +1646,9 @@ ) Builtins.y2milestone( "SubvolHandling remove subvol:%1", - Ops.get_list(new, "subvol", []) + Ops.get_list(new_partition, "subvol", []) ) - items = SubvolNames(new) + items = SubvolNames(new_partition) Builtins.y2milestone("SubvolHandling remove items:%1", items) changed = true UI.ChangeWidget(Id(:subvol), :Items, items) @@ -1661,7 +1661,7 @@ pth, svtmp ) - Builtins.y2milestone("SubvolHandling names:%1", SubvolNames(new)) + Builtins.y2milestone("SubvolHandling names:%1", SubvolNames(new_partition)) if pth == nil || Builtins.size(pth) == 0 Popup.Message(_("Empty subvolume name not allowed.")) else @@ -1676,41 +1676,41 @@ Popup.Message(tmp) pth = svtmp + pth end - if Builtins.contains(SubvolNames(new), pth) + if Builtins.contains(SubvolNames(new_partition), pth) Popup.Message( Builtins.sformat(_("Subvolume name %1 already exists."), pth) ) else Ops.set( - new, + new_partition, "subvol", Builtins.add( - Ops.get_list(new, "subvol", []), + Ops.get_list(new_partition, "subvol", []), { "create" => true, "name" => pth } ) ) changed = true end end - items = SubvolNames(new) + items = SubvolNames(new_partition) UI.ChangeWidget(Id(:subvol), :Items, items) UI.ChangeWidget(Id(:new_path), :Value, "") when :ok val = UI.QueryWidget(Id(:snapshots), :Value) if val - old_userdata["/"] = "snapshots" + initial_userdata["/"] = "snapshots" else - old_userdata.delete("/") + initial_userdata.delete("/") end - Ops.set(new, "userdata", old_userdata) + Ops.set(new_partition, "userdata", initial_userdata) when :cancel if changed if Popup.YesNo( _("Modifications done so far in this dialog will be lost.") ) - Ops.set(new, "subvol", old_subvol) + new_partition["subvol"] = initial_subvol else ret = :again end @@ -1724,11 +1724,18 @@ Builtins.y2milestone( "SubvolHandling subvol:%1 userdata:%2", - Ops.get_list(new, "subvol", []), - Ops.get_map(new, "userdata", {}) + new_partition["subvol"], + new_partition["userdata"] ) - deep_copy(new) + deep_copy(new_partition) end + # Checks whether Btrfs snapshots are supported for the partition + # + # @param partition [Hash] map representing the partition + # @return [Boolean] + def snapshots_supported?(partition) + partition["mount"] == "/" + end end end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-storage-3.1.74/src/modules/Storage.rb new/yast2-storage-3.1.75/src/modules/Storage.rb --- old/yast2-storage-3.1.74/src/modules/Storage.rb 2015-12-22 14:58:11.000000000 +0100 +++ new/yast2-storage-3.1.75/src/modules/Storage.rb 2016-02-05 11:13:09.000000000 +0100 @@ -1924,18 +1924,13 @@ def HandleBtrfsSimpleVolumes(tg) tg = deep_copy(tg) if Builtins.haskey(tg, "/dev/btrfs") - simple = Builtins.filter( - Ops.get_list(tg, ["/dev/btrfs", "partitions"], []) - ) do |p| - Ops.less_or_equal(Builtins.size(Ops.get_list(p, "devices", [])), 1) + btrfs_partitions = Ops.get_list(tg, ["/dev/btrfs", "partitions"], []) + simple = Builtins.filter(btrfs_partitions) do |p| + p["devices"].nil? || p["devices"].size <= 1 + end + tg["/dev/btrfs"]["partitions"] = Builtins.filter(btrfs_partitions) do |p| + p["devices"] && p["devices"].size > 1 end - Ops.set( - tg, - ["/dev/btrfs", "partitions"], - Builtins.filter(Ops.get_list(tg, ["/dev/btrfs", "partitions"], [])) do |p| - Ops.greater_than(Builtins.size(Ops.get_list(p, "devices", [])), 1) - end - ) Builtins.y2milestone("HandleBtrfsSimpleVolumes simple\n%1", format_target_map(simple)) keys = [ "subvol", @@ -1946,10 +1941,11 @@ "mount", "mountby", "used_fs", + "fstopt", "userdata" ] Builtins.foreach(simple) do |p| - mp = GetPartition(tg, Ops.get_string(p, "device", "")) + mp = GetPartition(tg, p["device"]) Builtins.y2milestone("HandleBtrfsSimpleVolumes before %1", mp) Builtins.foreach(keys) do |k| if Ops.get(p, k) != nil
