All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
To: buildroot@buildroot.org
Cc: "Wojciech M . Zabolotny" <wzab01@gmail.com>,
	Fabrice Fontaine <fontaine.fabrice@gmail.com>
Subject: [Buildroot] [PATCH 2/4] package/jack2: libsndfile is optional, not mandatory
Date: Wed, 27 Jul 2022 15:26:28 +0200	[thread overview]
Message-ID: <20220727132630.1323856-2-fontaine.fabrice@gmail.com> (raw)
In-Reply-To: <20220727132630.1323856-1-fontaine.fabrice@gmail.com>

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/jack2/Config.in | 1 -
 package/jack2/jack2.mk  | 9 ++++++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/package/jack2/Config.in b/package/jack2/Config.in
index e02449ba87..f5ac0399eb 100644
--- a/package/jack2/Config.in
+++ b/package/jack2/Config.in
@@ -5,7 +5,6 @@ config BR2_PACKAGE_JACK2
 	depends on BR2_INSTALL_LIBSTDCPP
 	depends on !BR2_STATIC_LIBS
 	depends on BR2_TOOLCHAIN_HAS_SYNC_4
-	select BR2_PACKAGE_LIBSNDFILE
 	select BR2_PACKAGE_ALSA_LIB
 	select BR2_PACKAGE_ALSA_LIB_HWDEP
 	select BR2_PACKAGE_ALSA_LIB_SEQ
diff --git a/package/jack2/jack2.mk b/package/jack2/jack2.mk
index 4059fa689c..3b45474cc4 100644
--- a/package/jack2/jack2.mk
+++ b/package/jack2/jack2.mk
@@ -9,7 +9,7 @@ JACK2_SITE = $(call github,jackaudio,jack2,v$(JACK2_VERSION))
 JACK2_LICENSE = GPL-2.0+ (jack server), LGPL-2.1+ (jack library)
 JACK2_LICENSE_FILES = COPYING
 JACK2_CPE_ID_VENDOR = jackaudio
-JACK2_DEPENDENCIES = libsndfile alsa-lib
+JACK2_DEPENDENCIES = alsa-lib
 JACK2_INSTALL_STAGING = YES
 
 JACK2_CONF_OPTS = --alsa
@@ -26,6 +26,13 @@ else
 JACK2_CONF_OPTS += --samplerate=no
 endif
 
+ifeq ($(BR2_PACKAGE_LIBSNDFILE),y)
+JACK2_DEPENDENCIES += libsndfile
+JACK2_CONF_OPTS += --sndfile=yes
+else
+JACK2_CONF_OPTS += --sndfile=no
+endif
+
 ifeq ($(BR2_PACKAGE_OPUS),y)
 JACK2_DEPENDENCIES += opus
 JACK2_CONF_OPTS += --opus=yes
-- 
2.35.1

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

  reply	other threads:[~2022-07-27 13:26 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-27 13:26 [Buildroot] [PATCH 1/4] package/jack2: libsamplerate is optional, not mandatoy Fabrice Fontaine
2022-07-27 13:26 ` Fabrice Fontaine [this message]
2022-08-30 16:09   ` [Buildroot] [PATCH 2/4] package/jack2: libsndfile is optional, not mandatory Peter Korsgaard
2022-07-27 13:26 ` [Buildroot] [PATCH 3/4] package/jack2: add systemd optional dependency Fabrice Fontaine
2022-08-30 16:09   ` Peter Korsgaard
2022-07-27 13:26 ` [Buildroot] [PATCH 4/4] package/jack2: disable example tools Fabrice Fontaine
2022-08-30 16:10   ` Peter Korsgaard
2022-07-27 14:54 ` [Buildroot] [PATCH 1/4] package/jack2: libsamplerate is optional, not mandatoy Thomas Petazzoni via buildroot
2022-08-30 16:09 ` Peter Korsgaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220727132630.1323856-2-fontaine.fabrice@gmail.com \
    --to=fontaine.fabrice@gmail.com \
    --cc=buildroot@buildroot.org \
    --cc=wzab01@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.