Accepting request 513701 from home:wolfi323:branches:KDE:Applications

- Add upstream patches to not show the system configuration problems dialog if disabled (kde#381368, boo#1051368)
  * Fix-do-not-show-again.patch
  * Add-readCheckSystemConfig.patch

OBS-URL: https://build.opensuse.org/request/show/513701
OBS-URL: https://build.opensuse.org/package/show/KDE:Applications/k3b?expand=0&rev=30

Add-readCheckSystemConfig.patch

@@ -0,0 +1,49 @@
+From 9408b834d4eb20264a526beb3bbbb1c5882db564 Mon Sep 17 00:00:00 2001
+From: Leslie Zhai <lesliezhai@llvm.org.cn>
+Date: Tue, 1 Aug 2017 14:24:42 +0800
+Subject: Add readCheckSystemConfig before checkSystem to fix  "do not show
+ again" from system configuration problems dialog  is not remembered issue.
+
+A great patch by Wolfgang Bauer!
+
+BUG: 381368
+---
+ src/k3bsystemproblemdialog.cpp | 13 ++++++++-----
+ 1 file changed, 8 insertions(+), 5 deletions(-)
+
+diff --git a/src/k3bsystemproblemdialog.cpp b/src/k3bsystemproblemdialog.cpp
+index d04a0bd..0d0f363 100644
+--- a/src/k3bsystemproblemdialog.cpp
++++ b/src/k3bsystemproblemdialog.cpp
+@@ -175,6 +175,9 @@ void K3b::SystemProblemDialog::checkSystem( QWidget* parent, NotificationLevel l
+     bool showDeviceSettingsButton = false;
+     bool showBinSettingsButton = false;
+ 
++    if (!readCheckSystemConfig())
++        return;
++
+     if( k3bcore->deviceManager()->allDevices().isEmpty() ) {
+         problems.append( K3b::SystemProblem( K3b::SystemProblem::CRITICAL,
+                                            i18n("No optical drive found."),
+@@ -711,13 +714,13 @@ QList<K3b::Device::Device*> K3b::SystemProblemDialog::checkForAutomounting()
+ 
+ bool K3b::SystemProblemDialog::readCheckSystemConfig()
+ {
+-    KConfigGroup cfgGrp( KSharedConfig::openConfig(), "General Options" );
++    KConfigGroup cfgGrp(KSharedConfig::openConfig(), "General Options");
+ 
+-    K3b::Version configVersion( cfgGrp.readEntry( "Last system check version", "0.1" ) );
+-    if( configVersion < k3bcore->version() )
+-        cfgGrp.writeEntry( "check system config", true );
++    K3b::Version configVersion(cfgGrp.readEntry( "Last system check version", "0.1" ));
++    if (configVersion < k3bcore->version())
++        cfgGrp.writeEntry("check system config", true);
+ 
+-    return cfgGrp.readEntry( "check system config", true );
++    return cfgGrp.readEntry("check system config", false);
+ }
+ 
+ 
+-- 
+cgit v0.11.2
+

Fix-do-not-show-again.patch

@@ -0,0 +1,53 @@
+From c1fae2adc90fac9a2af11ec9ef95ecc12033fa41 Mon Sep 17 00:00:00 2001
+From: Leslie Zhai <lesliezhai@llvm.org.cn>
+Date: Mon, 19 Jun 2017 16:42:41 +0800
+Subject: Fix "do not show again" from system configuration problems dialog is
+ not remembered
+
+BUG: 381368
+REVIEW: 130162
+---
+ src/k3bsystemproblemdialog.cpp | 11 +++++------
+ src/k3bsystemproblemdialog.h   |  2 +-
+ 2 files changed, 6 insertions(+), 7 deletions(-)
+
+diff --git a/src/k3bsystemproblemdialog.cpp b/src/k3bsystemproblemdialog.cpp
+index 9dfc50c..126ad31 100644
+--- a/src/k3bsystemproblemdialog.cpp
++++ b/src/k3bsystemproblemdialog.cpp
+@@ -159,14 +159,13 @@ K3b::SystemProblemDialog::SystemProblemDialog( const QList<K3b::SystemProblem>&
+ }
+ 
+ 
+-void K3b::SystemProblemDialog::closeEvent( QCloseEvent* e )
++void K3b::SystemProblemDialog::done(int r)
+ {
+-    if( m_checkDontShowAgain->isChecked() ) {
+-        KConfigGroup grp( KSharedConfig::openConfig(), "General Options" );
+-        grp.writeEntry( "check system config", false );
++    if (m_checkDontShowAgain->isChecked()) {
++        KConfigGroup grp(KSharedConfig::openConfig(), "General Options");
++        grp.writeEntry("check system config", false);
+     }
+-
+-    e->accept();
++    QDialog::done(r);
+ }
+ 
+ 
+diff --git a/src/k3bsystemproblemdialog.h b/src/k3bsystemproblemdialog.h
+index b45f2f8..b406b25 100644
+--- a/src/k3bsystemproblemdialog.h
++++ b/src/k3bsystemproblemdialog.h
+@@ -81,7 +81,7 @@ namespace K3b {
+         static void checkSystem( QWidget* parent = 0, NotificationLevel level = NotifyOnlyErrors );
+ 
+     protected:
+-        void closeEvent( QCloseEvent* );
++        void done(int) Q_DECL_OVERRIDE;
+ 
+     private Q_SLOTS:
+         void slotShowDeviceSettings();
+-- 
+cgit v0.11.2
+

k3b.changes

@@ -1,3 +1,11 @@
+-------------------------------------------------------------------
+Tue Aug  1 11:45:24 UTC 2017 - wbauer@tmo.at
+
+- Add upstream patches to not show the system configuration
+  problems dialog if disabled (kde#381368, boo#1051368)
+  * Fix-do-not-show-again.patch
+  * Add-readCheckSystemConfig.patch
+
 -------------------------------------------------------------------
 Wed Jul 26 15:41:52 UTC 2017 - wbauer@tmo.at
 

k3b.spec

@@ -30,10 +30,14 @@ License:        GPL-2.0+
 Group:          Productivity/Multimedia/CD/Record
 Url:            http://www.kde.org
 Source:         %{name}-%{version}.tar.xz
-# PATCH-FIX-OPENSUSE
+# PATCH-FIX-UPSTREAM
 Patch:          Re-enable-transcode-support.patch
 # PATCH-FIX-OPENSUSE
 Patch1:         Don-t-suggest-to-install-libburn.patch
+# PATCH-FIX-UPSTREAM
+Patch2:         Fix-do-not-show-again.patch
+# PATCH-FIX-UPSTREAM
+Patch3:         Add-readCheckSystemConfig.patch
 
 BuildRequires:  extra-cmake-modules
 BuildRequires:  fdupes
@@ -122,6 +126,8 @@ This package contain files needed for development with k3b.
 %setup -q
 %patch -p1
 %patch1 -p1
+%patch2 -p1
+%patch3 -p1
 
 %build
 CXXFLAGS="%{optflags} -fno-strict-aliasing"