Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package bluez for openSUSE:Factory checked 
in at 2021-12-01 20:46:21
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/bluez (Old)
 and      /work/SRC/openSUSE:Factory/.bluez.new.31177 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "bluez"

Wed Dec  1 20:46:21 2021 rev:183 rq:934690 version:5.62

Changes:
--------
--- /work/SRC/openSUSE:Factory/bluez/bluez.changes      2021-10-30 
23:13:09.763027761 +0200
+++ /work/SRC/openSUSE:Factory/.bluez.new.31177/bluez.changes   2021-12-02 
02:10:31.663527587 +0100
@@ -1,0 +2,13 @@
+Tue Nov 23 09:52:32 UTC 2021 - Bj??rn Lie <bjorn....@gmail.com>
+
+- Stop nuking the obex service, we support user systemd services
+  just fine now. Following this, no longer hack the dbus service,
+  leave it as a systemd service as upstream intended.
+- Split out obex in own package with it's needed enabledment as a
+  systemd user service.
+- Add 0001-obex-Use-GLib-helper-function-to-manipulate-paths.patch:
+  obex: Use GLib helper function to manipulate paths. Instead of
+  trying to do it by hand. This also makes sure that relative paths
+  aren't used by the agent. Patch from fedora.
+
+-------------------------------------------------------------------

New:
----
  0001-obex-Use-GLib-helper-function-to-manipulate-paths.patch

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

Other differences:
------------------
++++++ bluez.spec ++++++
--- /var/tmp/diff_new_pack.tfWu3v/_old  2021-12-02 02:10:32.275525721 +0100
+++ /var/tmp/diff_new_pack.tfWu3v/_new  2021-12-02 02:10:32.279525708 +0100
@@ -53,6 +53,9 @@
 #
 # Move 43xx firmware path for RPi3 bluetooth support bsc#1140688
 Patch10:        RPi-Move-the-43xx-firmware-into-lib-firmware.patch
+#
+# PATCH-FIX-UPSTREAM 
0001-obex-Use-GLib-helper-function-to-manipulate-paths.patch -- obex: Use GLib 
helper function to manipulate paths
+Patch11:        
https://src.fedoraproject.org/rpms/bluez/raw/rawhide/f/0001-obex-Use-GLib-helper-function-to-manipulate-paths.patch
 # Upstream suggests to use btmon instead of hcidump and does not want those 
patches
 # => PATCH-FIX-OPENSUSE for those two :-)
 # fix some memory leak with malformed packet (reported upstream but not yet 
fixed)
@@ -184,15 +187,20 @@
 to use the modern tools instead.
 %endif
 
+%package obexd
+Summary:        Object Exchange daemon for sharing content
+License:        GPL-2.0-or-later
+Group:          Hardware/Mobile
+Requires:       bluez = %{version}
+
+%description obexd
+Object Exchange daemon for sharing content.
+
 %prep
 %setup -q
 %autopatch -p1
 mkdir dbus-apis
 cp -a doc/*.txt dbus-apis/
-# FIXME: Change the dbus service to be a real service, not systemd launched
-sed -i "s:Exec=/bin/false:Exec=%{_libexecdir}/bluetooth/obexd:g" 
obexd/src/org.bluez.obex.service
-sed -i "/SystemdService=.*/d" obexd/src/org.bluez.obex.service
-# END FIXME
 
 # for auto-enable subpackage
 sed -i '/^#AutoEnable=false/aAutoEnable=true' src/main.conf
@@ -242,10 +250,6 @@
 # no idea why this is suddenly necessary...
 install --mode 0755 -d %{buildroot}%{_localstatedir}/lib/bluetooth
 
-# FIXME: Do not delete the systemd service once we support systemd 
user/session services
-rm %{buildroot}%{_userunitdir}/obex.service
-# end FIXME
-
 ## same as in fedora...
 # "make install" fails to install gatttool, used with Bluetooth Low Energy
 # boo#970628
@@ -318,6 +322,18 @@
 %post -n libbluetooth3 -p /sbin/ldconfig
 %postun -n libbluetooth3 -p /sbin/ldconfig
 
+%pre obexd
+%systemd_user_pre obex.service
+
+%post obexd
+%systemd_user_post obex.service
+
+%preun obexd
+%systemd_user_preun obex.service
+
+%postun obexd
+%systemd_user_postun obex.service
+
 %files
 %doc AUTHORS ChangeLog README dbus-apis src/main.conf
 %if %{with mesh}
@@ -339,7 +355,6 @@
 %{_bindir}/mesh-cfgtest
 %{_mandir}/man8/bluetooth-meshd.8%{?ext_man}
 %endif
-%{_libexecdir}/bluetooth/obexd
 %{_bindir}/bluetoothctl
 %{_bindir}/btmon
 %if %{with mesh}
@@ -364,11 +379,15 @@
 %{_unitdir}/bluetooth-mesh.service
 %endif
 %{_datadir}/dbus-1/system-services/org.bluez.service
-%{_datadir}/dbus-1/services/org.bluez.obex.service
 # not packaged, boo#1151518
 ###%%{_datadir}/dbus-1/system-services/org.bluez.mesh.service
 %{_datadir}/zsh/site-functions/_bluetoothctl
 
+%files obexd
+%{_libexecdir}/bluetooth/obexd
+%{_datadir}/dbus-1/services/org.bluez.obex.service
+%{_userunitdir}/obex.service
+
 %if %{with bluez_deprecated}
 %files deprecated
 %{_bindir}/gatttool

++++++ 0001-obex-Use-GLib-helper-function-to-manipulate-paths.patch ++++++
>From 90b72b787a6ae6b9b0bf8ece238e108e8607a433 Mon Sep 17 00:00:00 2001
From: Bastien Nocera <had...@hadess.net>
Date: Sat, 9 Nov 2013 18:13:43 +0100
Subject: [PATCH 1/2] obex: Use GLib helper function to manipulate paths

Instead of trying to do it by hand. This also makes sure that
relative paths aren't used by the agent.
---
 obexd/src/manager.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/obexd/src/manager.c b/obexd/src/manager.c
index f84384ae4..285c07c37 100644
--- a/obexd/src/manager.c
+++ b/obexd/src/manager.c
@@ -650,14 +650,14 @@ static void agent_reply(DBusPendingCall *call, void 
*user_data)
                                DBUS_TYPE_STRING, &name,
                                DBUS_TYPE_INVALID)) {
                /* Splits folder and name */
-               const char *slash = strrchr(name, '/');
+               gboolean is_relative = !g_path_is_absolute(name);
                DBG("Agent replied with %s", name);
-               if (!slash) {
-                       agent->new_name = g_strdup(name);
+               if (is_relative) {
+                       agent->new_name = g_path_get_basename(name);
                        agent->new_folder = NULL;
                } else {
-                       agent->new_name = g_strdup(slash + 1);
-                       agent->new_folder = g_strndup(name, slash - name);
+                       agent->new_name = g_path_get_basename(name);
+                       agent->new_folder = g_path_get_dirname(name);
                }
        }
 
-- 
2.14.1

Reply via email to