Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] bluez: update to 5.60
@ 2021-07-12 23:48 deepcube
  2021-07-17  3:30 ` [PR PATCH] [Updated] " ericonr
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: deepcube @ 2021-07-12 23:48 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 2008 bytes --]

There is a new pull request by deepcube against master on the void-packages repository

https://github.com/deepcube/void-packages emg-bluez-5.60
https://github.com/void-linux/void-packages/pull/31930

bluez: update to 5.60
Note that in some circumstances the user must delete /var/lib/bluetooth
if 5.59 created cache files affected by bluez issue 157[0].  I don't
know if it's acceptable to do this in post_install().  I will note that
archlinux did not remove /var/lib/bluetooth[1].

[0] https://github.com/bluez/bluez/issues/157
[1] https://github.com/archlinux/svntogit-packages/commit/03e8179086ea65f821d799a069ae75ee4e6fadb3

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/31930.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-emg-bluez-5.60-31930.patch --]
[-- Type: text/x-diff, Size: 4531 bytes --]

From c0f3d2e9f9b6a1d15593a1ec5a6417afc4f669a5 Mon Sep 17 00:00:00 2001
From: Evan Gates <evan.gates@gmail.com>
Date: Mon, 12 Jul 2021 16:38:56 -0700
Subject: [PATCH] bluez: update to 5.60

Note that in some circumstances the user must delete /var/lib/bluetooth
if 5.59 created cache files affected by bluez issue 157[0].  I don't
know if it's acceptable to do this in post_install().  I will note that
archlinux did not remove /var/lib/bluetooth[1].

[0] https://github.com/bluez/bluez/issues/157
[1] https://github.com/archlinux/svntogit-packages/commit/03e8179086ea65f821d799a069ae75ee4e6fadb3
---
 ...fkill-Fix-reading-from-rfkill-socket.patch | 83 -------------------
 srcpkgs/bluez/template                        |  4 +-
 2 files changed, 2 insertions(+), 85 deletions(-)
 delete mode 100644 srcpkgs/bluez/patches/0001-rfkill-Fix-reading-from-rfkill-socket.patch

diff --git a/srcpkgs/bluez/patches/0001-rfkill-Fix-reading-from-rfkill-socket.patch b/srcpkgs/bluez/patches/0001-rfkill-Fix-reading-from-rfkill-socket.patch
deleted file mode 100644
index 1bcb6524f5bd..000000000000
--- a/srcpkgs/bluez/patches/0001-rfkill-Fix-reading-from-rfkill-socket.patch
+++ /dev/null
@@ -1,83 +0,0 @@
-From 382646b490c1c336dd32f9186293616ceb4d808b Mon Sep 17 00:00:00 2001
-From: Benjamin Berg <bberg@redhat.com>
-Date: Mon, 3 May 2021 14:45:00 +0200
-Subject: [PATCH 1/2] rfkill: Fix reading from rfkill socket
-
-The kernel will always send exactly one event, but the size of the
-passed struct will depend on the length of the submitted read() and the
-kernel version. i.e. the interface can be extended and we need to expect
-for a read to be longer than expected if we ask for it.
-
-Fix this by only requesting the needed length and explicitly check the
-length against the V1 version of the structure to make the code a bit
-more future proof in case the internal copy of the struct is updated to
-contain new fields.
----
- src/rfkill.c | 24 +++++++++++-------------
- 1 file changed, 11 insertions(+), 13 deletions(-)
-
-diff --git a/src/rfkill.c b/src/rfkill.c
-index ec9fcdfdd..2099c5ac5 100644
---- a/src/rfkill.c
-+++ b/src/rfkill.c
-@@ -53,12 +53,12 @@ struct rfkill_event {
- 	uint8_t  soft;
- 	uint8_t  hard;
- };
-+#define RFKILL_EVENT_SIZE_V1    8
- 
- static gboolean rfkill_event(GIOChannel *chan,
- 				GIOCondition cond, gpointer data)
- {
--	unsigned char buf[32];
--	struct rfkill_event *event = (void *) buf;
-+	struct rfkill_event event = { 0 };
- 	struct btd_adapter *adapter;
- 	char sysname[PATH_MAX];
- 	ssize_t len;
-@@ -69,34 +69,32 @@ static gboolean rfkill_event(GIOChannel *chan,
- 
- 	fd = g_io_channel_unix_get_fd(chan);
- 
--	memset(buf, 0, sizeof(buf));
--
--	len = read(fd, buf, sizeof(buf));
-+	len = read(fd, &event, sizeof(event));
- 	if (len < 0) {
- 		if (errno == EAGAIN)
- 			return TRUE;
- 		return FALSE;
- 	}
- 
--	if (len != sizeof(struct rfkill_event))
-+	if (len < RFKILL_EVENT_SIZE_V1)
- 		return TRUE;
- 
- 	DBG("RFKILL event idx %u type %u op %u soft %u hard %u",
--					event->idx, event->type, event->op,
--						event->soft, event->hard);
-+					event.idx, event.type, event.op,
-+						event.soft, event.hard);
- 
--	if (event->soft || event->hard)
-+	if (event.soft || event.hard)
- 		return TRUE;
- 
--	if (event->op != RFKILL_OP_CHANGE)
-+	if (event.op != RFKILL_OP_CHANGE)
- 		return TRUE;
- 
--	if (event->type != RFKILL_TYPE_BLUETOOTH &&
--					event->type != RFKILL_TYPE_ALL)
-+	if (event.type != RFKILL_TYPE_BLUETOOTH &&
-+					event.type != RFKILL_TYPE_ALL)
- 		return TRUE;
- 
- 	snprintf(sysname, sizeof(sysname) - 1,
--			"/sys/class/rfkill/rfkill%u/name", event->idx);
-+			"/sys/class/rfkill/rfkill%u/name", event.idx);
- 
- 	fd = open(sysname, O_RDONLY);
- 	if (fd < 0)
--- 
-2.31.1
diff --git a/srcpkgs/bluez/template b/srcpkgs/bluez/template
index 09bac4ff5eca..470a7beaef0b 100644
--- a/srcpkgs/bluez/template
+++ b/srcpkgs/bluez/template
@@ -1,6 +1,6 @@
 # Template file for 'bluez'
 pkgname=bluez
-version=5.59
+version=5.60
 revision=1
 build_style=gnu-configure
 configure_args="--with-udevdir=/usr/lib/udev --disable-systemd
@@ -15,7 +15,7 @@ maintainer="Érico Nogueira <ericonr@disroot.org>"
 license="GPL-2.0-or-later, LGPL-2.1-or-later"
 homepage="http://www.bluez.org/"
 distfiles="${KERNEL_SITE}/bluetooth/$pkgname-$version.tar.xz"
-checksum=046b95b386d0bfb2a16311fe799d517ee7735045512d89902c4ed701db477316
+checksum=710999580d01ee59ec585e5e7c07fd94eddedc001aa26fe7464c546f9d945304
 conf_files="/etc/bluetooth/main.conf"
 system_groups="bluetooth"
 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PR PATCH] [Updated] bluez: update to 5.60
  2021-07-12 23:48 [PR PATCH] bluez: update to 5.60 deepcube
@ 2021-07-17  3:30 ` ericonr
  2021-07-17  3:30 ` ericonr
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: ericonr @ 2021-07-17  3:30 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 2012 bytes --]

There is an updated pull request by ericonr against master on the void-packages repository

https://github.com/deepcube/void-packages emg-bluez-5.60
https://github.com/void-linux/void-packages/pull/31930

bluez: update to 5.60
Note that in some circumstances the user must delete /var/lib/bluetooth
if 5.59 created cache files affected by bluez issue 157[0].  I don't
know if it's acceptable to do this in post_install().  I will note that
archlinux did not remove /var/lib/bluetooth[1].

[0] https://github.com/bluez/bluez/issues/157
[1] https://github.com/archlinux/svntogit-packages/commit/03e8179086ea65f821d799a069ae75ee4e6fadb3

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [x] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/31930.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-emg-bluez-5.60-31930.patch --]
[-- Type: text/x-diff, Size: 4956 bytes --]

From aac2bae2ea3836507310fc2e64b24029a4a487d1 Mon Sep 17 00:00:00 2001
From: Evan Gates <evan.gates@gmail.com>
Date: Mon, 12 Jul 2021 16:38:56 -0700
Subject: [PATCH] bluez: update to 5.60

In some circumstances, the user must delete /var/lib/bluetooth if 5.59
created cache files affected by bluez issue #157 [1]. We chose not to
automate this procedure, since it would also delete user configurations.
Instead, we will keep an INSTALL.msg until the next version bump.

[1] https://github.com/bluez/bluez/issues/157
---
 srcpkgs/bluez/INSTALL.msg                     |  5 ++
 ...fkill-Fix-reading-from-rfkill-socket.patch | 83 -------------------
 srcpkgs/bluez/template                        |  4 +-
 3 files changed, 7 insertions(+), 85 deletions(-)
 create mode 100644 srcpkgs/bluez/INSTALL.msg
 delete mode 100644 srcpkgs/bluez/patches/0001-rfkill-Fix-reading-from-rfkill-socket.patch

diff --git a/srcpkgs/bluez/INSTALL.msg b/srcpkgs/bluez/INSTALL.msg
new file mode 100644
index 000000000000..349f14710bec
--- /dev/null
+++ b/srcpkgs/bluez/INSTALL.msg
@@ -0,0 +1,5 @@
+WARNING: if bluez 5.59 had audio issues for you, removing the
+/var/lib/bluetooth directory might be necessary to completely restore
+functionality.
+
+For more information, see https://github.com/bluez/bluez/issues/157
diff --git a/srcpkgs/bluez/patches/0001-rfkill-Fix-reading-from-rfkill-socket.patch b/srcpkgs/bluez/patches/0001-rfkill-Fix-reading-from-rfkill-socket.patch
deleted file mode 100644
index 1bcb6524f5bd..000000000000
--- a/srcpkgs/bluez/patches/0001-rfkill-Fix-reading-from-rfkill-socket.patch
+++ /dev/null
@@ -1,83 +0,0 @@
-From 382646b490c1c336dd32f9186293616ceb4d808b Mon Sep 17 00:00:00 2001
-From: Benjamin Berg <bberg@redhat.com>
-Date: Mon, 3 May 2021 14:45:00 +0200
-Subject: [PATCH 1/2] rfkill: Fix reading from rfkill socket
-
-The kernel will always send exactly one event, but the size of the
-passed struct will depend on the length of the submitted read() and the
-kernel version. i.e. the interface can be extended and we need to expect
-for a read to be longer than expected if we ask for it.
-
-Fix this by only requesting the needed length and explicitly check the
-length against the V1 version of the structure to make the code a bit
-more future proof in case the internal copy of the struct is updated to
-contain new fields.
----
- src/rfkill.c | 24 +++++++++++-------------
- 1 file changed, 11 insertions(+), 13 deletions(-)
-
-diff --git a/src/rfkill.c b/src/rfkill.c
-index ec9fcdfdd..2099c5ac5 100644
---- a/src/rfkill.c
-+++ b/src/rfkill.c
-@@ -53,12 +53,12 @@ struct rfkill_event {
- 	uint8_t  soft;
- 	uint8_t  hard;
- };
-+#define RFKILL_EVENT_SIZE_V1    8
- 
- static gboolean rfkill_event(GIOChannel *chan,
- 				GIOCondition cond, gpointer data)
- {
--	unsigned char buf[32];
--	struct rfkill_event *event = (void *) buf;
-+	struct rfkill_event event = { 0 };
- 	struct btd_adapter *adapter;
- 	char sysname[PATH_MAX];
- 	ssize_t len;
-@@ -69,34 +69,32 @@ static gboolean rfkill_event(GIOChannel *chan,
- 
- 	fd = g_io_channel_unix_get_fd(chan);
- 
--	memset(buf, 0, sizeof(buf));
--
--	len = read(fd, buf, sizeof(buf));
-+	len = read(fd, &event, sizeof(event));
- 	if (len < 0) {
- 		if (errno == EAGAIN)
- 			return TRUE;
- 		return FALSE;
- 	}
- 
--	if (len != sizeof(struct rfkill_event))
-+	if (len < RFKILL_EVENT_SIZE_V1)
- 		return TRUE;
- 
- 	DBG("RFKILL event idx %u type %u op %u soft %u hard %u",
--					event->idx, event->type, event->op,
--						event->soft, event->hard);
-+					event.idx, event.type, event.op,
-+						event.soft, event.hard);
- 
--	if (event->soft || event->hard)
-+	if (event.soft || event.hard)
- 		return TRUE;
- 
--	if (event->op != RFKILL_OP_CHANGE)
-+	if (event.op != RFKILL_OP_CHANGE)
- 		return TRUE;
- 
--	if (event->type != RFKILL_TYPE_BLUETOOTH &&
--					event->type != RFKILL_TYPE_ALL)
-+	if (event.type != RFKILL_TYPE_BLUETOOTH &&
-+					event.type != RFKILL_TYPE_ALL)
- 		return TRUE;
- 
- 	snprintf(sysname, sizeof(sysname) - 1,
--			"/sys/class/rfkill/rfkill%u/name", event->idx);
-+			"/sys/class/rfkill/rfkill%u/name", event.idx);
- 
- 	fd = open(sysname, O_RDONLY);
- 	if (fd < 0)
--- 
-2.31.1
diff --git a/srcpkgs/bluez/template b/srcpkgs/bluez/template
index 09bac4ff5eca..470a7beaef0b 100644
--- a/srcpkgs/bluez/template
+++ b/srcpkgs/bluez/template
@@ -1,6 +1,6 @@
 # Template file for 'bluez'
 pkgname=bluez
-version=5.59
+version=5.60
 revision=1
 build_style=gnu-configure
 configure_args="--with-udevdir=/usr/lib/udev --disable-systemd
@@ -15,7 +15,7 @@ maintainer="Érico Nogueira <ericonr@disroot.org>"
 license="GPL-2.0-or-later, LGPL-2.1-or-later"
 homepage="http://www.bluez.org/"
 distfiles="${KERNEL_SITE}/bluetooth/$pkgname-$version.tar.xz"
-checksum=046b95b386d0bfb2a16311fe799d517ee7735045512d89902c4ed701db477316
+checksum=710999580d01ee59ec585e5e7c07fd94eddedc001aa26fe7464c546f9d945304
 conf_files="/etc/bluetooth/main.conf"
 system_groups="bluetooth"
 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: bluez: update to 5.60
  2021-07-12 23:48 [PR PATCH] bluez: update to 5.60 deepcube
  2021-07-17  3:30 ` [PR PATCH] [Updated] " ericonr
@ 2021-07-17  3:30 ` ericonr
  2021-07-17  3:40 ` ericonr
  2021-07-17  4:25 ` [PR PATCH] [Merged]: " ericonr
  3 siblings, 0 replies; 5+ messages in thread
From: ericonr @ 2021-07-17  3:30 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 236 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/31930#issuecomment-881809736

Comment:
Ok, I have added some stuff, will merge in a bit :)

Thanks a lot for tracking all this down!

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: bluez: update to 5.60
  2021-07-12 23:48 [PR PATCH] bluez: update to 5.60 deepcube
  2021-07-17  3:30 ` [PR PATCH] [Updated] " ericonr
  2021-07-17  3:30 ` ericonr
@ 2021-07-17  3:40 ` ericonr
  2021-07-17  4:25 ` [PR PATCH] [Merged]: " ericonr
  3 siblings, 0 replies; 5+ messages in thread
From: ericonr @ 2021-07-17  3:40 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 291 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/31930#issuecomment-881810618

Comment:
Btw, `post_install` is for the package still being built; if you want to run a command on a device during installation, you need a `INSTALL` script :)

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PR PATCH] [Merged]: bluez: update to 5.60
  2021-07-12 23:48 [PR PATCH] bluez: update to 5.60 deepcube
                   ` (2 preceding siblings ...)
  2021-07-17  3:40 ` ericonr
@ 2021-07-17  4:25 ` ericonr
  3 siblings, 0 replies; 5+ messages in thread
From: ericonr @ 2021-07-17  4:25 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1848 bytes --]

There's a merged pull request on the void-packages repository

bluez: update to 5.60
https://github.com/void-linux/void-packages/pull/31930

Description:
Note that in some circumstances the user must delete /var/lib/bluetooth
if 5.59 created cache files affected by bluez issue 157[0].  I don't
know if it's acceptable to do this in post_install().  I will note that
archlinux did not remove /var/lib/bluetooth[1].

[0] https://github.com/bluez/bluez/issues/157
[1] https://github.com/archlinux/svntogit-packages/commit/03e8179086ea65f821d799a069ae75ee4e6fadb3

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [x] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-07-17  4:25 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-12 23:48 [PR PATCH] bluez: update to 5.60 deepcube
2021-07-17  3:30 ` [PR PATCH] [Updated] " ericonr
2021-07-17  3:30 ` ericonr
2021-07-17  3:40 ` ericonr
2021-07-17  4:25 ` [PR PATCH] [Merged]: " ericonr

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).