using Package class for checking installed chrony package (#112)

* using Package class for checking installed chrony package
yast · Apr 19, 2018 · 48e72b6 · 48e72b6
1 parent 39ea859
commit 48e72b6
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/package/yast2-ntp-client.changes b/package/yast2-ntp-client.changes
@@ -1,3 +1,10 @@
+-------------------------------------------------------------------
+Thu Apr 19 08:56:08 CEST 2018 - schubi@suse.de
+
+- Fix in checking installed chrony package while writing ntp
+  settings. (bnc#1089962)
+- 4.0.12
+
 -------------------------------------------------------------------
 Fri Apr 13 10:34:33 CEST 2018 - schubi@suse.de
 

diff --git a/package/yast2-ntp-client.spec b/package/yast2-ntp-client.spec
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-ntp-client
-Version:        4.0.11
+Version:        4.0.12
 Release:        0
 Summary:        YaST2 - NTP Client Configuration
 License:        GPL-2.0+

diff --git a/src/lib/y2ntp_client/client/finish.rb b/src/lib/y2ntp_client/client/finish.rb
@@ -4,7 +4,7 @@
 require "installation/finish_client"
 
 Yast.import "NtpClient"
-Yast.import "Pkg"
+Yast.import "Package"
 
 module Y2NtpClient
   module Client
@@ -25,7 +25,9 @@ def modes
       end
 
       def write
-        unless Pkg.PkgInstalled(NtpClientClass::REQUIRED_PACKAGE)
+        unless Package.Installed(NtpClientClass::REQUIRED_PACKAGE)
+          # Not using Pkg.PkgInstalled because it is not working correctly here.
+          # (bnc#1089962)
           Report.Error(Builtins.sformat(
             # TRANSLATORS: Popup message. %1 is the missing package name.
             _("Cannot save NTP configuration because the package %1 is not installed."),