Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package yast2-bootloader for openSUSE:Factory checked in at 2024-03-03 20:19:08 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/yast2-bootloader (Old) and /work/SRC/openSUSE:Factory/.yast2-bootloader.new.1770 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "yast2-bootloader" Sun Mar 3 20:19:08 2024 rev:334 rq:1154077 version:5.0.6 Changes: -------- --- /work/SRC/openSUSE:Factory/yast2-bootloader/yast2-bootloader.changes 2024-02-25 14:05:25.987410503 +0100 +++ /work/SRC/openSUSE:Factory/.yast2-bootloader.new.1770/yast2-bootloader.changes 2024-03-03 20:19:31.574275258 +0100 @@ -1,0 +2,6 @@ +Mon Feb 26 13:30:54 UTC 2024 - Stefan Schubert <sch...@suse.com> + +- Feature: Do not timeout boot menue in systemd-boot. (bsc#1216366) +- 5.0.6 + +------------------------------------------------------------------- Old: ---- yast2-bootloader-5.0.5.tar.bz2 New: ---- yast2-bootloader-5.0.6.tar.bz2 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ yast2-bootloader.spec ++++++ --- /var/tmp/diff_new_pack.QkCA0V/_old 2024-03-03 20:19:32.354303477 +0100 +++ /var/tmp/diff_new_pack.QkCA0V/_new 2024-03-03 20:19:32.358303622 +0100 @@ -17,7 +17,7 @@ Name: yast2-bootloader -Version: 5.0.5 +Version: 5.0.6 Release: 0 Summary: YaST2 - Bootloader Configuration License: GPL-2.0-or-later ++++++ yast2-bootloader-5.0.5.tar.bz2 -> yast2-bootloader-5.0.6.tar.bz2 ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-5.0.5/package/yast2-bootloader.changes new/yast2-bootloader-5.0.6/package/yast2-bootloader.changes --- old/yast2-bootloader-5.0.5/package/yast2-bootloader.changes 2024-02-23 16:02:02.000000000 +0100 +++ new/yast2-bootloader-5.0.6/package/yast2-bootloader.changes 2024-02-29 11:59:06.000000000 +0100 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Mon Feb 26 13:30:54 UTC 2024 - Stefan Schubert <sch...@suse.com> + +- Feature: Do not timeout boot menue in systemd-boot. (bsc#1216366) +- 5.0.6 + +------------------------------------------------------------------- Tue Feb 20 07:42:45 UTC 2024 - Stefan Schubert <sch...@suse.com> - Proposal: Trying to take the bootloader which has been defined in diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-5.0.5/package/yast2-bootloader.spec new/yast2-bootloader-5.0.6/package/yast2-bootloader.spec --- old/yast2-bootloader-5.0.5/package/yast2-bootloader.spec 2024-02-23 16:02:02.000000000 +0100 +++ new/yast2-bootloader-5.0.6/package/yast2-bootloader.spec 2024-02-29 11:59:06.000000000 +0100 @@ -17,7 +17,7 @@ Name: yast2-bootloader -Version: 5.0.5 +Version: 5.0.6 Release: 0 Summary: YaST2 - Bootloader Configuration License: GPL-2.0-or-later diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-5.0.5/src/lib/bootloader/systemdboot.rb new/yast2-bootloader-5.0.6/src/lib/bootloader/systemdboot.rb --- old/yast2-bootloader-5.0.5/src/lib/bootloader/systemdboot.rb 2024-02-23 16:02:02.000000000 +0100 +++ new/yast2-bootloader-5.0.6/src/lib/bootloader/systemdboot.rb 2024-02-29 11:59:06.000000000 +0100 @@ -164,12 +164,22 @@ def read_menue_timeout config = CFA::SystemdBoot.load - self.menue_timeout = config.menue_timeout.to_i if config.menue_timeout + return unless config.menue_timeout + + self.menue_timeout = if config.menue_timeout == "menu-force" + -1 + else + config.menue_timeout.to_i + end end def write_menue_timeout config = CFA::SystemdBoot.load - config.menue_timeout = menue_timeout.to_s + config.menue_timeout = if menue_timeout == -1 + "menu-force" + else + menue_timeout.to_s + end config.save end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-5.0.5/src/lib/bootloader/systemdboot_widgets.rb new/yast2-bootloader-5.0.6/src/lib/bootloader/systemdboot_widgets.rb --- old/yast2-bootloader-5.0.5/src/lib/bootloader/systemdboot_widgets.rb 2024-02-23 16:02:02.000000000 +0100 +++ new/yast2-bootloader-5.0.6/src/lib/bootloader/systemdboot_widgets.rb 2024-02-29 11:59:06.000000000 +0100 @@ -17,7 +17,7 @@ end # Represents bootloader timeout value - class TimeoutWidget < CWM::IntField + class TimeoutWidget < CWM::CustomWidget include SystemdBootHelper def initialize @@ -25,27 +25,54 @@ super() - @minimum = -1 + @minimum = 0 @maximum = 600 + @default = 10 end - attr_reader :minimum, :maximum + attr_reader :minimum, :maximum, :default - def label - _("&Timeout in Seconds") + def contents + CheckBoxFrame( + Id(:cont_boot), + _("Automatically boot the default entry after a timeout"), + false, + HBox( + IntField(Id(:seconds), _("&Timeout in Seconds"), @minimum, @maximum, + systemdboot.menue_timeout.to_i), + HStretch() + ) + ) end def help - _("<p><b>Timeout in Seconds</b>\n" \ + _("<p>Continue boot process after defined seconds.</p>" \ + "<p><b>Timeout in Seconds</b>\n" \ "specifies the time the boot loader will wait until the default kernel is loaded.</p>\n") end def init - self.value = systemdboot.menue_timeout.to_i + Yast::UI.ChangeWidget(Id(:cont_boot), :Value, systemdboot.menue_timeout >= 0) + systemdboot.menue_timeout = default_value if systemdboot.menue_timeout < 0 + Yast::UI.ChangeWidget(Id(:seconds), :Value, systemdboot.menue_timeout) end def store - systemdboot.menue_timeout = value.to_s + if Yast::UI.QueryWidget(Id(:cont_boot), :Value) + systemdboot.menue_timeout = Yast::UI.QueryWidget(Id(:seconds), :Value) + else + systemdboot.menue_timeout = -1 + end + end + + private + + def default_value + # set default + ret = Yast::ProductFeatures.GetIntegerFeature("globals", + "boot_timeout").to_i + ret = @default if ret <= 0 + ret end end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-5.0.5/test/systemdboot_widgets_test.rb new/yast2-bootloader-5.0.6/test/systemdboot_widgets_test.rb --- old/yast2-bootloader-5.0.5/test/systemdboot_widgets_test.rb 2024-02-23 16:02:02.000000000 +0100 +++ new/yast2-bootloader-5.0.6/test/systemdboot_widgets_test.rb 2024-02-29 11:59:06.000000000 +0100 @@ -35,21 +35,31 @@ assign_systemd_bootloader end - it_behaves_like "labeled widget" + it_behaves_like "CWM::CustomWidget" - it "has minimal value to -1 as unlimited" do - expect(subject.minimum).to eq(-1) + it "has minimal value to 0 as unlimited" do + expect(subject.minimum).to eq(0) end it "has maximum value to 600" do expect(subject.maximum).to eq 600 end - it "is initialized to timeout value if defined" do - bootloader.menue_timeout = "10" - expect(subject).to receive(:value=).with(10) + it "has own complex content" do + expect(subject.contents).to be_a Yast::Term + end + + context "storing content" do + before do + stub_widget_value(:cont_boot, false) + stub_widget_value(:seconds, 15) + end + + it "sets timeout to -1 for using menu-force" do + subject.store - subject.init + expect(bootloader.menue_timeout).to eq(-1) + end end end