Hello community, here is the log from the commit of package yast2-bootloader for openSUSE:Factory checked in at 2017-06-17 10:18:17 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/yast2-bootloader (Old) and /work/SRC/openSUSE:Factory/.yast2-bootloader.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "yast2-bootloader" Sat Jun 17 10:18:17 2017 rev:247 rq:503765 version:3.2.22 Changes: -------- --- /work/SRC/openSUSE:Factory/yast2-bootloader/yast2-bootloader.changes 2017-06-07 13:08:31.930619286 +0200 +++ /work/SRC/openSUSE:Factory/.yast2-bootloader.new/yast2-bootloader.changes 2017-06-17 10:18:18.399384381 +0200 @@ -1,0 +2,14 @@ +Wed Jun 14 14:54:38 UTC 2017 - [email protected] + +- Use udev device for prep partition if it is available + (bsc#1041692) +- 3.2.22 + +------------------------------------------------------------------- +Mon Jun 12 11:00:48 UTC 2017 - [email protected] + +- Do not crash if all devices have defined bios_id, but none is + the first one (bsc#1043759) +- 3.2.21 + +------------------------------------------------------------------- Old: ---- yast2-bootloader-3.2.20.tar.bz2 New: ---- yast2-bootloader-3.2.22.tar.bz2 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ yast2-bootloader.spec ++++++ --- /var/tmp/diff_new_pack.l5X9U2/_old 2017-06-17 10:18:19.607214003 +0200 +++ /var/tmp/diff_new_pack.l5X9U2/_new 2017-06-17 10:18:19.611213438 +0200 @@ -17,7 +17,7 @@ Name: yast2-bootloader -Version: 3.2.20 +Version: 3.2.22 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build ++++++ yast2-bootloader-3.2.20.tar.bz2 -> yast2-bootloader-3.2.22.tar.bz2 ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.2.20/package/yast2-bootloader.changes new/yast2-bootloader-3.2.22/package/yast2-bootloader.changes --- old/yast2-bootloader-3.2.20/package/yast2-bootloader.changes 2017-05-30 14:00:40.931263978 +0200 +++ new/yast2-bootloader-3.2.22/package/yast2-bootloader.changes 2017-06-15 08:56:02.918822057 +0200 @@ -1,4 +1,18 @@ ------------------------------------------------------------------- +Wed Jun 14 14:54:38 UTC 2017 - [email protected] + +- Use udev device for prep partition if it is available + (bsc#1041692) +- 3.2.22 + +------------------------------------------------------------------- +Mon Jun 12 11:00:48 UTC 2017 - [email protected] + +- Do not crash if all devices have defined bios_id, but none is + the first one (bsc#1043759) +- 3.2.21 + +------------------------------------------------------------------- Tue May 30 07:37:01 UTC 2017 - [email protected] - Repropose bootloader configuration when storage proposal is modified (bsc#1035746) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.2.20/package/yast2-bootloader.spec new/yast2-bootloader-3.2.22/package/yast2-bootloader.spec --- old/yast2-bootloader-3.2.20/package/yast2-bootloader.spec 2017-05-30 14:00:40.931263978 +0200 +++ new/yast2-bootloader-3.2.22/package/yast2-bootloader.spec 2017-06-15 08:56:02.918822057 +0200 @@ -17,7 +17,7 @@ Name: yast2-bootloader -Version: 3.2.20 +Version: 3.2.22 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.2.20/src/lib/bootloader/device_map.rb new/yast2-bootloader-3.2.22/src/lib/bootloader/device_map.rb --- old/yast2-bootloader-3.2.20/src/lib/bootloader/device_map.rb 2017-05-30 14:00:41.051263978 +0200 +++ new/yast2-bootloader-3.2.22/src/lib/bootloader/device_map.rb 2017-06-15 08:56:03.054822057 +0200 @@ -203,7 +203,9 @@ remove_mapping("hd0") remove_mapping(grub_dev) add_mapping("hd0", priority_device) - add_mapping(grub_dev, replaced_dev) + # switch order only if there was previously device at hd0. It can be empty e.g. + # if bios_id is defined, but not for 0x80 + add_mapping(grub_dev, replaced_dev) if replaced_dev end # Check if MD raid is build on disks not on paritions diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.2.20/src/lib/bootloader/stage1_proposal.rb new/yast2-bootloader-3.2.22/src/lib/bootloader/stage1_proposal.rb --- old/yast2-bootloader-3.2.20/src/lib/bootloader/stage1_proposal.rb 2017-05-30 14:00:41.151263978 +0200 +++ new/yast2-bootloader-3.2.22/src/lib/bootloader/stage1_proposal.rb 2017-06-15 08:56:03.182822057 +0200 @@ -5,6 +5,7 @@ Yast.import "Storage" require "bootloader/stage1_device" +require "bootloader/udev_mapping" module Bootloader # Represents object that can set passed stage1 to proposed values. @@ -198,7 +199,9 @@ def propose partition = proposed_prep_partition if partition - assign_bootloader_device([:custom, partition]) + # ensure that stage1 device is in udev (bsc#1041692) + udev_partition = UdevMapping.to_mountby_device(partition) + assign_bootloader_device([:custom, udev_partition]) stage1.activate = !on_gpt?(partition) # do not activate on gpt disks see (bnc#983194) stage1.generic_mbr = false diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.2.20/test/stage1_test.rb new/yast2-bootloader-3.2.22/test/stage1_test.rb --- old/yast2-bootloader-3.2.20/test/stage1_test.rb 2017-05-30 14:00:41.479263978 +0200 +++ new/yast2-bootloader-3.2.22/test/stage1_test.rb 2017-06-15 08:56:03.414822057 +0200 @@ -100,6 +100,18 @@ expect(subject.devices).to eq(["/dev/sda1"]) end + it "sets udev link for device" do + expect(Yast::Storage).to receive(:GetPartition).with(anything, "/dev/sdc1") + .and_return("create" => true) + + expect(Bootloader::UdevMapping).to receive(:to_mountby_device).with("/dev/sdc1") + .and_return("/dev/disk/by-id/partition1") + + subject.propose + + expect(subject.devices).to eq(["/dev/disk/by-id/partition1"]) + end + it "activate partition if it is on DOS partition table" do expect(Yast::Storage).to receive(:GetDisk).with(anything, "/dev/sdb1") .and_return("label" => "dos")
