From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 942AF13997D for ; Tue, 12 Nov 2019 22:09:58 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 93266E0874; Tue, 12 Nov 2019 22:09:57 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 6A291E0874 for ; Tue, 12 Nov 2019 22:09:57 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 4002E34CCDB for ; Tue, 12 Nov 2019 22:09:56 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id 3E5FC8AC for ; Tue, 12 Nov 2019 22:09:54 +0000 (UTC) From: "Andreas Sturmlechner" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Andreas Sturmlechner" Message-ID: <1573596534.a59d298a587a4f21e62c0532578bf0b657b0a1ff.asturm@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: kde-frameworks/solid/files/, kde-frameworks/solid/ X-VCS-Repository: repo/gentoo X-VCS-Files: kde-frameworks/solid/files/solid-5.64.0-udisks2-media-change-detection.patch kde-frameworks/solid/solid-5.64.0-r1.ebuild X-VCS-Directories: kde-frameworks/solid/ kde-frameworks/solid/files/ X-VCS-Committer: asturm X-VCS-Committer-Name: Andreas Sturmlechner X-VCS-Revision: a59d298a587a4f21e62c0532578bf0b657b0a1ff X-VCS-Branch: master Date: Tue, 12 Nov 2019 22:09:54 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: 93b7643b-3322-4862-bb30-fa2c432cd942 X-Archives-Hash: 282d4601f472226309f61fae2093a39a commit: a59d298a587a4f21e62c0532578bf0b657b0a1ff Author: Andreas Sturmlechner gentoo org> AuthorDate: Tue Nov 12 19:47:32 2019 +0000 Commit: Andreas Sturmlechner gentoo org> CommitDate: Tue Nov 12 22:08:54 2019 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=a59d298a kde-frameworks/solid: Fix ext. optical drives media change detection KDE-Bug: https://bugs.kde.org/show_bug.cgi?id=394348 Package-Manager: Portage-2.3.79, Repoman-2.3.18 Signed-off-by: Andreas Sturmlechner gentoo.org> ...lid-5.64.0-udisks2-media-change-detection.patch | 65 ++++++++++++++++++++++ kde-frameworks/solid/solid-5.64.0-r1.ebuild | 38 +++++++++++++ 2 files changed, 103 insertions(+) diff --git a/kde-frameworks/solid/files/solid-5.64.0-udisks2-media-change-detection.patch b/kde-frameworks/solid/files/solid-5.64.0-udisks2-media-change-detection.patch new file mode 100644 index 00000000000..57fd17f7b61 --- /dev/null +++ b/kde-frameworks/solid/files/solid-5.64.0-udisks2-media-change-detection.patch @@ -0,0 +1,65 @@ +From 4aa39aed1bd6c81dbf0d6dc9d312af5340f7caab Mon Sep 17 00:00:00 2001 +From: Rok Mandeljc +Date: Tue, 12 Nov 2019 10:20:38 -0700 +Subject: [udisks2] fix media change detection for external optical drives + +Summary: +If an external optical drive is connected after Solid does its initial introspection, the resulting UDI does not get a mediaChanged signal/slot connection, and thus fails to react to the media change. Consequently, disc content is not properly updated after the medium is ejected, causing bug #394348. + +This patch assumes that an addition of "org.freedesktop.UDisks2.Block" interface means that a new device has been added; in this case, it performs Device::mightBeOpticalDisc() check to add the slotMediaChanged() connection. + +BUG: 394348 +FIXED-IN: 5.65 + +Test Plan: +Test steps: +1. Start KDE Plasma session. Make sure K3b is installed. +2. Connect an external USB optical drive. +3. Insert the disc +4. Observe Device action notifications for the inserted disc. +5. Eject the disc. +6. Observe Device action notifications. + +Behavior before patch: after disc is ejected, a "Copy with K3b" action remains available for the drive +See: https://bugs.kde.org/show_bug.cgi?id=394348 + +Behavior after patch: after disc is ejected, no actions remain available for the drive + +Reviewers: bruns, broulik, dfaure, #frameworks, ngraham + +Reviewed By: ngraham + +Subscribers: ngraham, bugseforuns, kde-frameworks-devel + +Tags: #frameworks + +Differential Revision: https://phabricator.kde.org/D20508 +--- + src/solid/devices/backends/udisks2/udisksmanager.cpp | 12 ++++++++++++ + 1 file changed, 12 insertions(+) + +diff --git a/src/solid/devices/backends/udisks2/udisksmanager.cpp b/src/solid/devices/backends/udisks2/udisksmanager.cpp +index 4982aee..3b76423 100644 +--- a/src/solid/devices/backends/udisks2/udisksmanager.cpp ++++ b/src/solid/devices/backends/udisks2/udisksmanager.cpp +@@ -191,6 +191,18 @@ void Manager::slotInterfacesAdded(const QDBusObjectPath &object_path, const Vari + + qCDebug(UDISKS2) << udi << "has new interfaces:" << interfaces_and_properties.keys(); + ++ // If device gained an org.freedesktop.UDisks2.Block interface, we ++ // should check if it is an optical drive, in order to properly ++ // register mediaChanged event handler with newly-plugged external ++ // drives ++ if (interfaces_and_properties.contains("org.freedesktop.UDisks2.Block")) { ++ Device device(udi); ++ if (device.mightBeOpticalDisc()) { ++ QDBusConnection::systemBus().connect(UD2_DBUS_SERVICE, udi, DBUS_INTERFACE_PROPS, "PropertiesChanged", this, ++ SLOT(slotMediaChanged(QDBusMessage))); ++ } ++ } ++ + updateBackend(udi); + + // new device, we don't know it yet +-- +cgit v1.1 diff --git a/kde-frameworks/solid/solid-5.64.0-r1.ebuild b/kde-frameworks/solid/solid-5.64.0-r1.ebuild new file mode 100644 index 00000000000..6eea94371fe --- /dev/null +++ b/kde-frameworks/solid/solid-5.64.0-r1.ebuild @@ -0,0 +1,38 @@ +# Copyright 1999-2019 Gentoo Authors +# Distributed under the terms of the GNU General Public License v2 + +EAPI=7 + +QTMIN=5.12.3 +VIRTUALX_REQUIRED="test" +inherit ecm kde.org + +DESCRIPTION="Provider for platform independent hardware discovery, abstraction and management" +LICENSE="LGPL-2.1+" +KEYWORDS="~amd64 ~arm ~arm64 ~ppc64 ~x86" +IUSE="nls" + +BDEPEND=" + nls? ( >=dev-qt/linguist-tools-${QTMIN}:5 ) +" +RDEPEND=" + >=dev-qt/qtdbus-${QTMIN}:5 + >=dev-qt/qtdeclarative-${QTMIN}:5 + >=dev-qt/qtwidgets-${QTMIN}:5 + >=dev-qt/qtxml-${QTMIN}:5 + sys-fs/udisks:2 + virtual/libudev:= +" +DEPEND="${RDEPEND} + test? ( >=dev-qt/qtconcurrent-${QTMIN}:5 ) +" + +PATCHES=( "${FILESDIR}/${P}-udisks2-media-change-detection.patch" ) + +pkg_postinst() { + ecm_pkg_postinst + + if [[ -z "${REPLACING_VERSIONS}" ]] && ! has_version "app-misc/media-player-info" ; then + elog "For media player support, install app-misc/media-player-info" + fi +}