Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/bluez5_utils: enable asha when building audio plugins
@ 2024-10-03 16:59 James Hilliard
  2024-10-03 20:06 ` Thomas Petazzoni via buildroot
  0 siblings, 1 reply; 3+ messages in thread
From: James Hilliard @ 2024-10-03 16:59 UTC (permalink / raw)
  To: buildroot; +Cc: James Hilliard, Marcin Bis

Symbols from the asha plugin are required when building a2dp
support as part of the audio plugins, so always enable asha
when audio plugins are selected.

Fixes:
 - http://autobuild.buildroot.net/results/5a3/5a306356f5f190a14a0a2294a4428544b1a018bb
 - http://autobuild.buildroot.net/results/353/3539c83deaa45a3548bcd488659148c62af3fc73

Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
---
 package/bluez5_utils/bluez5_utils.mk | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/package/bluez5_utils/bluez5_utils.mk b/package/bluez5_utils/bluez5_utils.mk
index 13a8ce2f0b..10bb067dd3 100644
--- a/package/bluez5_utils/bluez5_utils.mk
+++ b/package/bluez5_utils/bluez5_utils.mk
@@ -28,7 +28,6 @@ BLUEZ5_UTILS_CONF_OPTS = \
 	--disable-lsan \
 	--disable-ubsan \
 	--disable-pie \
-	--disable-asha \
 	--with-dbusconfdir=/etc
 
 ifeq ($(BR2_PACKAGE_BLUEZ5_UTILS_OBEX),y)
@@ -68,6 +67,7 @@ endif
 ifeq ($(BR2_PACKAGE_BLUEZ5_UTILS_PLUGINS_AUDIO),y)
 BLUEZ5_UTILS_CONF_OPTS += \
 	--enable-a2dp \
+	--enable-asha \
 	--enable-avrcp \
 	--enable-bap \
 	--enable-mcp \
@@ -75,6 +75,7 @@ BLUEZ5_UTILS_CONF_OPTS += \
 else
 BLUEZ5_UTILS_CONF_OPTS += \
 	--disable-a2dp \
+	--disable-asha \
 	--disable-avrcp \
 	--disable-bap \
 	--disable-mcp \
-- 
2.34.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH 1/1] package/bluez5_utils: enable asha when building audio plugins
  2024-10-03 16:59 [Buildroot] [PATCH 1/1] package/bluez5_utils: enable asha when building audio plugins James Hilliard
@ 2024-10-03 20:06 ` Thomas Petazzoni via buildroot
  2024-10-03 20:31   ` James Hilliard
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Petazzoni via buildroot @ 2024-10-03 20:06 UTC (permalink / raw)
  To: James Hilliard; +Cc: Marcin Bis, buildroot

Hello James,

On Thu,  3 Oct 2024 10:59:26 -0600
James Hilliard <james.hilliard1@gmail.com> wrote:

> Symbols from the asha plugin are required when building a2dp
> support as part of the audio plugins, so always enable asha
> when audio plugins are selected.
> 
> Fixes:
>  - http://autobuild.buildroot.net/results/5a3/5a306356f5f190a14a0a2294a4428544b1a018bb
>  - http://autobuild.buildroot.net/results/353/3539c83deaa45a3548bcd488659148c62af3fc73
> 
> Signed-off-by: James Hilliard <james.hilliard1@gmail.com>

Could you please indicate since when this problem is occurring. It
seems to be appearing since the recent bump, but is it really the case?

A bug fix submitted to Buildroot should *always* carry an explanation
of since when the problem started occurring.

Thanks!

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH 1/1] package/bluez5_utils: enable asha when building audio plugins
  2024-10-03 20:06 ` Thomas Petazzoni via buildroot
@ 2024-10-03 20:31   ` James Hilliard
  0 siblings, 0 replies; 3+ messages in thread
From: James Hilliard @ 2024-10-03 20:31 UTC (permalink / raw)
  To: Thomas Petazzoni; +Cc: Marcin Bis, buildroot


[-- Attachment #1.1: Type: text/plain, Size: 1165 bytes --]

On Thu, Oct 3, 2024 at 2:06 PM Thomas Petazzoni <
thomas.petazzoni@bootlin.com> wrote:

> Hello James,
>
> On Thu,  3 Oct 2024 10:59:26 -0600
> James Hilliard <james.hilliard1@gmail.com> wrote:
>
> > Symbols from the asha plugin are required when building a2dp
> > support as part of the audio plugins, so always enable asha
> > when audio plugins are selected.
> >
> > Fixes:
> >  -
> http://autobuild.buildroot.net/results/5a3/5a306356f5f190a14a0a2294a4428544b1a018bb
> >  -
> http://autobuild.buildroot.net/results/353/3539c83deaa45a3548bcd488659148c62af3fc73
> >
> > Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
>
> Could you please indicate since when this problem is occurring. It
> seems to be appearing since the recent bump, but is it really the case?
>

Looks like it was introduced in ce4e5fd129b557ac0411d0a0704be43717310d6d


>
> A bug fix submitted to Buildroot should *always* carry an explanation
> of since when the problem started occurring.
>
> Thanks!
>
> Thomas
> --
> Thomas Petazzoni, co-owner and CEO, Bootlin
> Embedded Linux and Kernel engineering and training
> https://bootlin.com
>

[-- Attachment #1.2: Type: text/html, Size: 2139 bytes --]

[-- Attachment #2: Type: text/plain, Size: 150 bytes --]

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

end of thread, other threads:[~2024-10-03 20:31 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-03 16:59 [Buildroot] [PATCH 1/1] package/bluez5_utils: enable asha when building audio plugins James Hilliard
2024-10-03 20:06 ` Thomas Petazzoni via buildroot
2024-10-03 20:31   ` James Hilliard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox