Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package yast2-installation for 
openSUSE:Factory checked in at 2021-04-19 21:05:38
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-installation (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-installation.new.12324 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-installation"

Mon Apr 19 21:05:38 2021 rev:467 rq:885917 version:4.4.4

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-installation/yast2-installation.changes    
2021-04-10 15:26:21.542306585 +0200
+++ 
/work/SRC/openSUSE:Factory/.yast2-installation.new.12324/yast2-installation.changes
 2021-04-19 21:05:46.404009047 +0200
@@ -1,0 +2,50 @@
+Fri Apr 16 10:19:04 UTC 2021 - Dirk M??ller <dmuel...@suse.com>
+
+- spec-cleaner part five out of five: sort and deduplicate requires
+- 4.4.4
+
+-------------------------------------------------------------------
+Fri Apr 16 09:50:49 UTC 2021 - Dirk M??ller <dmuel...@suse.com>
+
+- spec-cleaner part two out of five: move conditionalized sections at the end
+- spec-cleaner part three out of five: move weak requires (recommends) below 
requires
+- spec-cleaner part four out of five: sort and deduplicate BuildRequires
+
+-------------------------------------------------------------------
+Fri Apr 16 08:39:17 UTC 2021 - Dirk M??ller <dmuel...@suse.com>
+
+- remove check for non-systemd distros
+- first out of 5 splits of "spec-cleaner -m -i *ec" run (for easier review)
+
+-------------------------------------------------------------------
+Wed Apr 14 11:08:52 UTC 2021 - Knut Anderssen <kanders...@suse.com>
+
+- Show 'Default' in the proposal summary as the PolicyKit Default
+  Privileges to be used when it is not specified or specified as
+  empty in the control file (bsc#1184277)
+- 4.4.3
+
+-------------------------------------------------------------------
+Wed Apr 14 09:46:38 UTC 2021 - Ludwig Nussel <lnus...@suse.com>
+
+- Check for usr/lib/modules to handle usrmerge (bsc#1029961)
+- 4.4.2
+
+-------------------------------------------------------------------
+Wed Apr 14 08:44:31 UTC 2021 - Dominique Leuenberger <dims...@opensuse.org>
+
+- Do not own system directories (like /usr/bin) (bsc#1184787):
+  + filesystem is responsible to bring those directories with the
+    correct permission flags. Owning them here only introduces
+    races/conflicts.
+  + As a result: expand the files section for bindir and unitdir to
+    be exact on the files.
+
+-------------------------------------------------------------------
+Fri Apr  9 16:21:31 UTC 2021 - Ladislav Slez??k <lsle...@suse.cz>
+
+- Start the "memsample" tool in a subshell to avoid "Terminated"
+  message displayed at the end (bsc#1184491)
+- 4.4.1
+
+-------------------------------------------------------------------

Old:
----
  yast2-installation-4.4.0.tar.bz2

New:
----
  yast2-installation-4.4.4.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-installation.spec ++++++
--- /var/tmp/diff_new_pack.mnPZT3/_old  2021-04-19 21:05:47.068010043 +0200
+++ /var/tmp/diff_new_pack.mnPZT3/_new  2021-04-19 21:05:47.072010049 +0200
@@ -17,122 +17,114 @@
 
 
 Name:           yast2-installation
-Version:        4.4.0
+Version:        4.4.4
 Release:        0
 Summary:        YaST2 - Installation Parts
 License:        GPL-2.0-only
 Group:          System/YaST
 URL:            https://github.com/yast/yast-installation
-
 Source0:        %{name}-%{version}.tar.bz2
 Source1:        YaST2-Second-Stage.service
 Source2:        YaST2-Firstboot.service
 
 BuildRequires:  update-desktop-files
-BuildRequires:  yast2-devtools >= 3.1.10
-# for AbortException and handle direct abort
-BuildRequires:  yast2-ruby-bindings >= 4.0.6
-# needed for xml agent reading about products
-BuildRequires:  yast2-xml
-BuildRequires:  rubygem(%{rb_default_ruby_abi}:rspec)
-BuildRequires:  rubygem(%{rb_default_ruby_abi}:yast-rake)
 # Yast2::FsSnapshotStore::IOError
 BuildRequires:  yast2 >= 4.3.53
-# Y2Packager::MediumType
-BuildRequires:  yast2-packager >= 4.2.27
 # CIOIgnore
 BuildRequires:  yast2-bootloader
-# For selinux classes
-BuildRequires:  yast2-security
+# storage-ng based version
+BuildRequires:  yast2-country >= 3.3.1
+BuildRequires:  yast2-devtools >= 3.1.10
 # For firewall widgets
 BuildRequires:  yast2-firewall
-# using /usr/bin/udevadm
-BuildRequires:  yast2-storage-ng >= 4.2.71
 # Y2Network::NtpServer
 BuildRequires:  yast2-network >= 4.2.55
+# Y2Packager::MediumType
+BuildRequires:  yast2-packager >= 4.2.27
+# for AbortException and handle direct abort
+BuildRequires:  yast2-ruby-bindings >= 4.0.6
+# For selinux classes
+BuildRequires:  yast2-security
+# using /usr/bin/udevadm
+BuildRequires:  yast2-storage-ng >= 4.2.71
 # new root password cwm widget
 BuildRequires:  yast2-users >= 3.2.8
-# storage-ng based version
-BuildRequires:  yast2-bootloader
-BuildRequires:  yast2-country >= 3.3.1
-
-PreReq:         %fillup_prereq
-Requires:       yast2-country >= 3.3.1
-Requires:       yast2-users >= 3.2.8
-# tar-gzip some system files and untar-ungzip them after the installation 
(FATE #300421, #120103)
+# needed for xml agent reading about products
+BuildRequires:  yast2-xml
+BuildRequires:  rubygem(%{rb_default_ruby_abi}:rspec)
+BuildRequires:  rubygem(%{rb_default_ruby_abi}:yast-rake)
+# Augeas lenses
+Requires:       augeas-lenses
 Requires:       coreutils
 Requires:       gzip
+# use in startup scripts
+Requires:       initviocons
+# Needed call /sbin/ip in vnc.sh/network.sh
+Requires:       iproute2
+# for the first/second stage of installation
+# currently not used
+# bugzilla #208307
+#Requires:      /usr/bin/jpegtopnm
+#Requires:      /usr/bin/pnmtopng
+# BNC 446533, /sbin/lspci called but not installed
+Requires:       pciutils
+# tar-gzip some system files and untar-ungzip them after the installation 
(FATE #300421, #120103)
 Requires:       tar
-# Augeas lenses
-Requires:       augeas-lenses
-# Only in inst-sys
-Requires:       yast2-storage-ng >= 4.0.175
+# Yast2::FsSnapshotStore::IOError
+Requires:       yast2 >= 4.3.53
+# CIOIgnore
+Requires:       yast2-bootloader
+Requires:       yast2-country >= 3.3.1
 # Language::GetLanguageItems and other API
 # Language::Set (handles downloading the translation extensions)
 Requires:       yast2-country-data >= 2.16.11
-# Pkg::ProvidePackage
-Requires:       yast2-pkg-bindings >= 3.1.33
+# Y2Network::NtpServer
+Requires:       yast2-network >= 4.2.55
 # Y2Packager::MediumType
 Requires:       yast2-packager >= 4.2.22
-# CIOIgnore
-Requires:       yast2-bootloader
-# use in startup scripts
-Requires:       initviocons
+# Pkg::ProvidePackage
+Requires:       yast2-pkg-bindings >= 3.1.33
 # Proxy settings for 2nd stage (bnc#764951)
 Requires:       yast2-proxy
+# for AbortException and handle direct abort
+Requires:       yast2-ruby-bindings >= 4.0.6
 # Systemd default target and services. This version supports
 # writing settings in the first installation stage.
 Requires:       yast2-services-manager >= 3.2.1
-# Yast2::FsSnapshotStore::IOError
-Requires:       yast2 >= 4.3.53
-# Y2Network::NtpServer
-Requires:       yast2-network >= 4.2.55
-# for AbortException and handle direct abort
-Requires:       yast2-ruby-bindings >= 4.0.6
-# for the first/second stage of installation
-# currently not used
-# bugzilla #208307
-#Requires:      /usr/bin/jpegtopnm
-#Requires:      /usr/bin/pnmtopng
-# BNC 446533, /sbin/lspci called but not installed
-Requires:       pciutils
-# Needed call /sbin/ip in vnc.sh/network.sh
-Requires:       iproute2
-%if 0%{?suse_version} >= 1210
-%{systemd_requires}
-%endif
-
+# Only in inst-sys
+Requires:       yast2-storage-ng >= 4.0.175
+Requires:       yast2-users >= 3.2.8
+PreReq:         %fillup_prereq
+Recommends:     yast2-add-on
+Recommends:     yast2-firewall
+Recommends:     yast2-online-update
+Supplements:    autoyast(deploy_image:ssh_import)
+# new autoinst_files_finish call
+Conflicts:      autoyast2 < 4.3.26
 # SingleItemSelector not enforcing an initial selection
 Conflicts:      libyui < 3.8.2
-# Pkg::SourceProvideSignedFile Pkg::SourceProvideDigestedFile
-# pkg-bindings are not directly required
-Conflicts:      yast2-pkg-bindings < 2.17.25
 # InstError
 Conflicts:      yast2 < 2.18.6
 # storage-ng based version
 Conflicts:      yast2-bootloader < 3.3.1
 # Added new function WFM::ClientExists
 Conflicts:      yast2-core < 2.17.10
-# Top bar with logo
-Conflicts:      yast2-ycp-ui-bindings < 3.1.7
-# Registration#get_updates_list does not handle exceptions
-Conflicts:      yast2-registration < 3.2.3
 # Mouse-related scripts moved to yast2-mouse
 Conflicts:      yast2-mouse < 2.18.0
-# new autoinst_files_finish call
-Conflicts:      autoyast2 < 4.3.26
+# Pkg::SourceProvideSignedFile Pkg::SourceProvideDigestedFile
+# pkg-bindings are not directly required
+Conflicts:      yast2-pkg-bindings < 2.17.25
+# Registration#get_updates_list does not handle exceptions
+Conflicts:      yast2-registration < 3.2.3
 # moved security_finish client
 Conflicts:      yast2-security < 4.3.11
-
+# Top bar with logo
+Conflicts:      yast2-ycp-ui-bindings < 3.1.7
 Obsoletes:      yast2-installation-devel-doc
-
-Recommends:     yast2-online-update
-Recommends:     yast2-firewall
-Recommends:     yast2-add-on
-
-Supplements:    autoyast(deploy_image:ssh_import)
-
 BuildArch:      noarch
+%if 0%{?suse_version} >= 1210
+%{systemd_requires}
+%endif
 
 %description
 System installation code as present on installation media.
@@ -170,8 +162,6 @@
 %post
 %{fillup_only -ns security checksig}
 
-%if 0%{suse_version} > 1140
-
 %service_add_post YaST2-Second-Stage.service YaST2-Firstboot.service
 
 # bsc#924278 Always enable these services by default, they are already listed
@@ -192,15 +182,15 @@
 %postun
 %service_del_postun YaST2-Second-Stage.service YaST2-Firstboot.service
 
-#suse_version
-%endif
-
 %files
-
 # systemd service files
-%{_unitdir}
+%{_unitdir}/YaST2-Firstboot.service
+%{_unitdir}/YaST2-Second-Stage.service
+%{_bindir}/memsample
+%{_bindir}/memsample-archive-to-csv
+%{_bindir}/memsample-csv-plot
 # yupdate script
-%{_bindir}/
+%{_bindir}/yupdate
 %{yast_clientdir}
 %{yast_moduledir}
 %{yast_desktopdir}

++++++ yast2-installation-4.4.0.tar.bz2 -> yast2-installation-4.4.4.tar.bz2 
++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-installation-4.4.0/bin/memsample-csv-plot 
new/yast2-installation-4.4.4/bin/memsample-csv-plot
--- old/yast2-installation-4.4.0/bin/memsample-csv-plot 2021-04-07 
17:13:07.000000000 +0200
+++ new/yast2-installation-4.4.4/bin/memsample-csv-plot 2021-04-16 
12:34:20.000000000 +0200
@@ -5,7 +5,7 @@
 PNG="${2-memsample.png}"
 echo >&2 "Plotting from $CSV to $PNG"
 cat <<EOS | gnuplot > "$PNG"
-set title "YaST instaler memory usage"
+set title "YaST installer memory usage"
 set datafile separator comma
 set timefmt "%Y-%m-%dT%H:%M:%S+00:00"
 set xdata time
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-4.4.0/package/yast2-installation.changes 
new/yast2-installation-4.4.4/package/yast2-installation.changes
--- old/yast2-installation-4.4.0/package/yast2-installation.changes     
2021-04-07 17:13:07.000000000 +0200
+++ new/yast2-installation-4.4.4/package/yast2-installation.changes     
2021-04-16 12:34:20.000000000 +0200
@@ -1,4 +1,54 @@
 -------------------------------------------------------------------
+Fri Apr 16 10:19:04 UTC 2021 - Dirk M??ller <dmuel...@suse.com>
+
+- spec-cleaner part five out of five: sort and deduplicate requires
+- 4.4.4
+
+-------------------------------------------------------------------
+Fri Apr 16 09:50:49 UTC 2021 - Dirk M??ller <dmuel...@suse.com>
+
+- spec-cleaner part two out of five: move conditionalized sections at the end
+- spec-cleaner part three out of five: move weak requires (recommends) below 
requires
+- spec-cleaner part four out of five: sort and deduplicate BuildRequires
+
+-------------------------------------------------------------------
+Fri Apr 16 08:39:17 UTC 2021 - Dirk M??ller <dmuel...@suse.com>
+
+- remove check for non-systemd distros
+- first out of 5 splits of "spec-cleaner -m -i *ec" run (for easier review)
+
+-------------------------------------------------------------------
+Wed Apr 14 11:08:52 UTC 2021 - Knut Anderssen <kanders...@suse.com>
+
+- Show 'Default' in the proposal summary as the PolicyKit Default
+  Privileges to be used when it is not specified or specified as
+  empty in the control file (bsc#1184277)
+- 4.4.3
+
+-------------------------------------------------------------------
+Wed Apr 14 09:46:38 UTC 2021 - Ludwig Nussel <lnus...@suse.com>
+
+- Check for usr/lib/modules to handle usrmerge (bsc#1029961)
+- 4.4.2
+
+-------------------------------------------------------------------
+Wed Apr 14 08:44:31 UTC 2021 - Dominique Leuenberger <dims...@opensuse.org>
+
+- Do not own system directories (like /usr/bin) (bsc#1184787):
+  + filesystem is responsible to bring those directories with the
+    correct permission flags. Owning them here only introduces
+    races/conflicts.
+  + As a result: expand the files section for bindir and unitdir to
+    be exact on the files.
+
+-------------------------------------------------------------------
+Fri Apr  9 16:21:31 UTC 2021 - Ladislav Slez??k <lsle...@suse.cz>
+
+- Start the "memsample" tool in a subshell to avoid "Terminated"
+  message displayed at the end (bsc#1184491)
+- 4.4.1
+
+-------------------------------------------------------------------
 Wed Apr  7 10:54:58 UTC 2021 - David Diaz <dgonza...@suse.com>
 
 - Hide the abort button when the network client is called
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-4.4.0/package/yast2-installation.spec 
new/yast2-installation-4.4.4/package/yast2-installation.spec
--- old/yast2-installation-4.4.0/package/yast2-installation.spec        
2021-04-07 17:13:07.000000000 +0200
+++ new/yast2-installation-4.4.4/package/yast2-installation.spec        
2021-04-16 12:34:20.000000000 +0200
@@ -1,7 +1,7 @@
 #
 # spec file for package yast2-installation
 #
-# Copyright (c) 2015 SUSE LINUX GmbH, Nuernberg, Germany.
+# Copyright (c) 2021 SUSE LLC
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -15,124 +15,116 @@
 # Please submit bugfixes or comments via https://bugs.opensuse.org/
 #
 
+
 Name:           yast2-installation
-Version:        4.4.0
+Version:        4.4.4
 Release:        0
-Group:          System/YaST
-License:        GPL-2.0-only
 Summary:        YaST2 - Installation Parts
-Url:            https://github.com/yast/yast-installation
-
+License:        GPL-2.0-only
+Group:          System/YaST
+URL:            https://github.com/yast/yast-installation
 Source0:        %{name}-%{version}.tar.bz2
 Source1:        YaST2-Second-Stage.service
 Source2:        YaST2-Firstboot.service
 
 BuildRequires:  update-desktop-files
-BuildRequires:  yast2-devtools >= 3.1.10
-# for AbortException and handle direct abort
-BuildRequires:  yast2-ruby-bindings >= 4.0.6
-# needed for xml agent reading about products
-BuildRequires:  yast2-xml
-BuildRequires:  rubygem(%{rb_default_ruby_abi}:rspec)
-BuildRequires:  rubygem(%{rb_default_ruby_abi}:yast-rake)
 # Yast2::FsSnapshotStore::IOError
 BuildRequires:  yast2 >= 4.3.53
-# Y2Packager::MediumType
-BuildRequires:  yast2-packager >= 4.2.27
 # CIOIgnore
 BuildRequires:  yast2-bootloader
-# For selinux classes
-BuildRequires:  yast2-security
+# storage-ng based version
+BuildRequires:  yast2-country >= 3.3.1
+BuildRequires:  yast2-devtools >= 3.1.10
 # For firewall widgets
 BuildRequires:  yast2-firewall
-# using /usr/bin/udevadm
-BuildRequires:  yast2-storage-ng >= 4.2.71
 # Y2Network::NtpServer
 BuildRequires:  yast2-network >= 4.2.55
+# Y2Packager::MediumType
+BuildRequires:  yast2-packager >= 4.2.27
+# for AbortException and handle direct abort
+BuildRequires:  yast2-ruby-bindings >= 4.0.6
+# For selinux classes
+BuildRequires:  yast2-security
+# using /usr/bin/udevadm
+BuildRequires:  yast2-storage-ng >= 4.2.71
 # new root password cwm widget
 BuildRequires:  yast2-users >= 3.2.8
-# storage-ng based version
-BuildRequires:  yast2-country >= 3.3.1
-BuildRequires:  yast2-bootloader
-
-PreReq:         %fillup_prereq
-Requires:       yast2-users >= 3.2.8
-Requires:       yast2-country >= 3.3.1
-# tar-gzip some system files and untar-ungzip them after the installation 
(FATE #300421, #120103)
-Requires:       tar
-Requires:       gzip
-Requires:       coreutils
+# needed for xml agent reading about products
+BuildRequires:  yast2-xml
+BuildRequires:  rubygem(%{rb_default_ruby_abi}:rspec)
+BuildRequires:  rubygem(%{rb_default_ruby_abi}:yast-rake)
 # Augeas lenses
 Requires:       augeas-lenses
-# Only in inst-sys
-Requires:       yast2-storage-ng >= 4.0.175
+Requires:       coreutils
+Requires:       gzip
+# use in startup scripts
+Requires:       initviocons
+# Needed call /sbin/ip in vnc.sh/network.sh
+Requires:       iproute2
+# for the first/second stage of installation
+# currently not used
+# bugzilla #208307
+#Requires:      /usr/bin/jpegtopnm
+#Requires:      /usr/bin/pnmtopng
+# BNC 446533, /sbin/lspci called but not installed
+Requires:       pciutils
+# tar-gzip some system files and untar-ungzip them after the installation 
(FATE #300421, #120103)
+Requires:       tar
+# Yast2::FsSnapshotStore::IOError
+Requires:       yast2 >= 4.3.53
+# CIOIgnore
+Requires:       yast2-bootloader
+Requires:       yast2-country >= 3.3.1
 # Language::GetLanguageItems and other API
 # Language::Set (handles downloading the translation extensions)
 Requires:       yast2-country-data >= 2.16.11
-# Pkg::ProvidePackage
-Requires:       yast2-pkg-bindings >= 3.1.33
+# Y2Network::NtpServer
+Requires:       yast2-network >= 4.2.55
 # Y2Packager::MediumType
 Requires:       yast2-packager >= 4.2.22
-# CIOIgnore
-Requires:       yast2-bootloader
-# use in startup scripts
-Requires:       initviocons
+# Pkg::ProvidePackage
+Requires:       yast2-pkg-bindings >= 3.1.33
 # Proxy settings for 2nd stage (bnc#764951)
 Requires:       yast2-proxy
+# for AbortException and handle direct abort
+Requires:       yast2-ruby-bindings >= 4.0.6
 # Systemd default target and services. This version supports
 # writing settings in the first installation stage.
 Requires:       yast2-services-manager >= 3.2.1
-# Yast2::FsSnapshotStore::IOError
-Requires:       yast2 >= 4.3.53
-# Y2Network::NtpServer
-Requires:       yast2-network >= 4.2.55
-# for AbortException and handle direct abort
-Requires:       yast2-ruby-bindings >= 4.0.6
-# for the first/second stage of installation
-# currently not used
-# bugzilla #208307
-#Requires:      /usr/bin/jpegtopnm
-#Requires:      /usr/bin/pnmtopng
-# BNC 446533, /sbin/lspci called but not installed
-Requires:       pciutils
-# Needed call /sbin/ip in vnc.sh/network.sh
-Requires:       iproute2
-%if 0%{?suse_version} >= 1210
-%{systemd_requires}
-%endif
-
-
+# Only in inst-sys
+Requires:       yast2-storage-ng >= 4.0.175
+Requires:       yast2-users >= 3.2.8
+PreReq:         %fillup_prereq
+Recommends:     yast2-add-on
+Recommends:     yast2-firewall
+Recommends:     yast2-online-update
+Supplements:    autoyast(deploy_image:ssh_import)
+# new autoinst_files_finish call
+Conflicts:      autoyast2 < 4.3.26
 # SingleItemSelector not enforcing an initial selection
 Conflicts:      libyui < 3.8.2
-# Pkg::SourceProvideSignedFile Pkg::SourceProvideDigestedFile
-# pkg-bindings are not directly required
-Conflicts:      yast2-pkg-bindings < 2.17.25
 # InstError
 Conflicts:      yast2 < 2.18.6
 # storage-ng based version
 Conflicts:      yast2-bootloader < 3.3.1
 # Added new function WFM::ClientExists
 Conflicts:      yast2-core < 2.17.10
-# Top bar with logo
-Conflicts:      yast2-ycp-ui-bindings < 3.1.7
-# Registration#get_updates_list does not handle exceptions
-Conflicts:      yast2-registration < 3.2.3
 # Mouse-related scripts moved to yast2-mouse
 Conflicts:      yast2-mouse < 2.18.0
-# new autoinst_files_finish call
-Conflicts:      autoyast2 < 4.3.26
+# Pkg::SourceProvideSignedFile Pkg::SourceProvideDigestedFile
+# pkg-bindings are not directly required
+Conflicts:      yast2-pkg-bindings < 2.17.25
+# Registration#get_updates_list does not handle exceptions
+Conflicts:      yast2-registration < 3.2.3
 # moved security_finish client
 Conflicts:      yast2-security < 4.3.11
-
+# Top bar with logo
+Conflicts:      yast2-ycp-ui-bindings < 3.1.7
 Obsoletes:      yast2-installation-devel-doc
-
-Recommends:     yast2-online-update
-Recommends:     yast2-firewall
-Recommends:     yast2-add-on
-
-Supplements:    autoyast(deploy_image:ssh_import)
-
 BuildArch:      noarch
+%if 0%{?suse_version} >= 1210
+%{systemd_requires}
+%endif
 
 %description
 System installation code as present on installation media.
@@ -171,8 +163,6 @@
 %post
 %{fillup_only -ns security checksig}
 
-%if 0%{suse_version} > 1140
-
 %service_add_post YaST2-Second-Stage.service YaST2-Firstboot.service
 
 # bsc#924278 Always enable these services by default, they are already listed
@@ -193,15 +183,15 @@
 %postun
 %service_del_postun YaST2-Second-Stage.service YaST2-Firstboot.service
 
-#suse_version
-%endif
-
 %files
-
 # systemd service files
-%{_unitdir}
+%{_unitdir}/YaST2-Firstboot.service
+%{_unitdir}/YaST2-Second-Stage.service
+%{_bindir}/memsample
+%{_bindir}/memsample-archive-to-csv
+%{_bindir}/memsample-csv-plot
 # yupdate script
-%{_bindir}/
+%{_bindir}/yupdate
 %{yast_clientdir}
 %{yast_moduledir}
 %{yast_desktopdir}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-4.4.0/src/lib/installation/clients/security_proposal.rb 
new/yast2-installation-4.4.4/src/lib/installation/clients/security_proposal.rb
--- 
old/yast2-installation-4.4.0/src/lib/installation/clients/security_proposal.rb  
    2021-04-07 17:13:07.000000000 +0200
+++ 
new/yast2-installation-4.4.4/src/lib/installation/clients/security_proposal.rb  
    2021-04-16 12:34:20.000000000 +0200
@@ -232,7 +232,7 @@
       end
 
       def polkit_default_priv_proposal
-        value = @settings.polkit_default_privileges || "default"
+        value = @settings.polkit_default_privileges.to_s
         human_value = @settings.human_polkit_privileges[value]
 
         format(_("PolicyKit Default Privileges: %s"), human_value)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-4.4.0/src/lib/installation/instsys_cleaner.rb 
new/yast2-installation-4.4.4/src/lib/installation/instsys_cleaner.rb
--- old/yast2-installation-4.4.0/src/lib/installation/instsys_cleaner.rb        
2021-04-07 17:13:07.000000000 +0200
+++ new/yast2-installation-4.4.4/src/lib/installation/instsys_cleaner.rb        
2021-04-16 12:34:20.000000000 +0200
@@ -61,7 +61,7 @@
     # kernel modules can vary significantly. This saves about 29MB on x86_64
     # and about 5MB on s390x.
     def self.unmount_kernel_modules
-      if !File.exist?(File.join(KERNEL_MODULES_MOUNT_POINT, "lib/modules"))
+      if !File.exist?(File.join(KERNEL_MODULES_MOUNT_POINT, "lib/modules")) && 
!File.exist?(File.join(KERNEL_MODULES_MOUNT_POINT, "usr/lib/modules"))
         log.warn("Kernel modules not found at #{KERNEL_MODULES_MOUNT_POINT}")
         log.warn("Skipping module cleanup")
         return
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-4.4.0/src/lib/installation/security_settings.rb 
new/yast2-installation-4.4.4/src/lib/installation/security_settings.rb
--- old/yast2-installation-4.4.0/src/lib/installation/security_settings.rb      
2021-04-07 17:13:07.000000000 +0200
+++ new/yast2-installation-4.4.4/src/lib/installation/security_settings.rb      
2021-04-16 12:34:20.000000000 +0200
@@ -146,7 +146,7 @@
 
     def human_polkit_privileges
       {
-        "default"     => _("Default"),
+        ""            => _("Default"),
         # TRANSLATORS: restrictive in sense the most restrictive policy
         "restrictive" => _("Restrictive"),
         "standard"    => _("Standard"),
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-4.4.0/src/lib/installation/widgets/polkit_default_priv.rb
 
new/yast2-installation-4.4.4/src/lib/installation/widgets/polkit_default_priv.rb
--- 
old/yast2-installation-4.4.0/src/lib/installation/widgets/polkit_default_priv.rb
    2021-04-07 17:13:07.000000000 +0200
+++ 
new/yast2-installation-4.4.4/src/lib/installation/widgets/polkit_default_priv.rb
    2021-04-16 12:34:20.000000000 +0200
@@ -18,6 +18,7 @@
 # 
------------------------------------------------------------------------------
 
 require "cwm/common_widgets"
+require "installation/security_settings"
 
 module Installation
   module Widgets
@@ -53,18 +54,17 @@
           "to allow a more seamless user experience without" \
           " interruptions in the workflow due to password " \
           "prompts.</li></ul><br>" \
-          "The \"default\" is to keep value empty and it will be" \
+          "The \"default\" is to keep value empty and it will be " \
           "assigned automatically.</p>"
         )
       end
 
       def init
-        self.value = @settings.polkit_default_privileges || "default"
+        self.value = @settings.polkit_default_privileges.to_s
       end
 
       def store
-        res = value == "default" ? nil : value
-        @settings.polkit_default_privileges = res
+        @settings.polkit_default_privileges = value.empty? ? nil : value
       end
     end
   end
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-4.4.0/startup/First-Stage/F08-logging 
new/yast2-installation-4.4.4/startup/First-Stage/F08-logging
--- old/yast2-installation-4.4.0/startup/First-Stage/F08-logging        
2021-04-07 17:13:07.000000000 +0200
+++ new/yast2-installation-4.4.4/startup/First-Stage/F08-logging        
2021-04-16 12:34:20.000000000 +0200
@@ -65,7 +65,10 @@
     if pgrep -f memsample; then
         log "\talready running"
     else
-        memsample --sleep="${MEMSAMPLE-5}" 
--archive=/var/log/YaST2/memsample.zcat &
-        log "\tPID: $!"
+        # use a subshell to avoid "Terminated" message when killing the 
process at the end
+        (
+            memsample --sleep="${MEMSAMPLE-5}" 
--archive=/var/log/YaST2/memsample.zcat &
+            log "\tPID: $!"
+        )
     fi
 fi
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-4.4.0/test/lib/clients/security_proposal_test.rb 
new/yast2-installation-4.4.4/test/lib/clients/security_proposal_test.rb
--- old/yast2-installation-4.4.0/test/lib/clients/security_proposal_test.rb     
2021-04-07 17:13:07.000000000 +0200
+++ new/yast2-installation-4.4.4/test/lib/clients/security_proposal_test.rb     
2021-04-16 12:34:20.000000000 +0200
@@ -184,5 +184,39 @@
         end
       end
     end
+
+    context "when showing the PolicyKit Default privileges selected" do
+      let(:selected) { "" }
+
+      before do
+        allow(proposal_settings).to 
receive(:polkit_default_privileges).and_return(selected)
+      end
+
+      context "and it is not set" do
+        let(:selected) { nil }
+
+        it "shows it as 'Default'" do
+          proposal = client.make_proposal({})
+          expect(proposal["preformatted_proposal"]).to include("Privileges: 
Default")
+        end
+      end
+
+      context "and it is empty" do
+        let(:selected) { nil }
+
+        it "shows it as 'Default'" do
+          proposal = client.make_proposal({})
+          expect(proposal["preformatted_proposal"]).to include("Privileges: 
Default")
+        end
+      end
+      context "and it is restrictive" do
+        let(:selected) { "restrictive" }
+
+        it "shows it as 'Restrictive'" do
+          proposal = client.make_proposal({})
+          expect(proposal["preformatted_proposal"]).to include("Privileges: 
Restrictive")
+        end
+      end
+    end
   end
 end
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-4.4.0/test/lib/widgets/polkit_default_priv_test.rb 
new/yast2-installation-4.4.4/test/lib/widgets/polkit_default_priv_test.rb
--- old/yast2-installation-4.4.0/test/lib/widgets/polkit_default_priv_test.rb   
2021-04-07 17:13:07.000000000 +0200
+++ new/yast2-installation-4.4.4/test/lib/widgets/polkit_default_priv_test.rb   
2021-04-16 12:34:20.000000000 +0200
@@ -5,7 +5,32 @@
 require "cwm/rspec"
 
 describe Installation::Widgets::PolkitDefaultPriv do
-  subject { 
described_class.new(Installation::SecuritySettings.create_instance) }
+  let(:settings) { Installation::SecuritySettings.create_instance }
+  subject { described_class.new(settings) }
 
   include_examples "CWM::ComboBox"
+
+  describe ".store" do
+    let(:selected) { "" }
+
+    before do
+      allow(subject).to receive(:value).and_return(selected)
+    end
+
+    context "when the value selected is the 'Default'" do
+      it "sets the settings polkit_default_privileges to nil" do
+        expect(settings).to receive(:polkit_default_privileges=).with(nil)
+        subject.store
+      end
+    end
+
+    context "when the value selected is other" do
+      let(:selected) { "restrictive" }
+
+      it "sets the settings with the selected value" do
+        expect(settings).to 
receive(:polkit_default_privileges=).with("restrictive")
+        subject.store
+      end
+    end
+  end
 end

Reply via email to