[Buildroot] [PATCH v2 1/2] package/ofono: remove forced autoreconf

[Buildroot] [PATCH v2 1/2] package/ofono: remove forced autoreconf

[Nicolas Serafini]

The patch that required an autoreconf is no longer there.

Signed-off-by: Nicolas Serafini <nicolas.serafini@sensefly.com>
---
 package/ofono/ofono.mk | 2 --
 1 file changed, 2 deletions(-)

diff --git a/package/ofono/ofono.mk b/package/ofono/ofono.mk
index ce05dfcadf..756998687b 100644
--- a/package/ofono/ofono.mk
+++ b/package/ofono/ofono.mk
@@ -49,7 +49,5 @@ else
 OFONO_CONF_OPTS += --disable-bluetooth
 endif
 
-# required by 0003-build-Add-check-for-explicit_bzero-support.patch
-OFONO_AUTORECONF = YES
 
 $(eval $(autotools-package))
-- 
2.23.0

[Buildroot] [PATCH v2 2/2] package/ofono: use target ell library if enabled

[Nicolas Serafini]

Signed-off-by: Nicolas Serafini <nicolas.serafini@sensefly.com>

---
Changes v1 -> v2:
  - Remove forced autoreconf into another patch
  - Add missing OFONO_DEPENDENCIES
  - Add disable option if ell is not selected

Signed-off-by: Nicolas Serafini <nicolas.serafini@sensefly.com>
---
 package/ofono/ofono.mk | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/package/ofono/ofono.mk b/package/ofono/ofono.mk
index 756998687b..00c5ffbbe4 100644
--- a/package/ofono/ofono.mk
+++ b/package/ofono/ofono.mk
@@ -49,5 +49,11 @@ else
 OFONO_CONF_OPTS += --disable-bluetooth
 endif
 
+ifeq ($(BR2_PACKAGE_ELL),y)
+OFONO_DEPENDENCIES += ell
+OFONO_CONF_OPTS += --enable-external-ell
+else
+OFONO_CONF_OPTS += --disable-external-ell
+endif
 
 $(eval $(autotools-package))
-- 
2.23.0

[Buildroot] [PATCH v2 2/2] package/ofono: use target ell library if enabled

[Thomas Petazzoni]

Hello Nicolas,

On Tue, 22 Oct 2019 13:44:08 +0000
Nicolas Serafini <nicolas.serafini@sensefly.com> wrote:

> +ifeq ($(BR2_PACKAGE_ELL),y)
> +OFONO_DEPENDENCIES += ell
> +OFONO_CONF_OPTS += --enable-external-ell
> +else
> +OFONO_CONF_OPTS += --disable-external-ell
> +endif

If I understand correctly the code, ell is in fact always needed by
ofono. If ell is not available externally, it uses a bundled version of
ell.

In this case, Buildroot prefers to always use the external library, so
could you turn ell into a mandatory dependency of ofono, and pass
--enable-external-ell unconditionally?

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [PATCH v2 2/2] package/ofono: use target ell library if enabled

[Nicolas Serafini]

Hello Thomas,

On Tue, 22 Oct 2019 22:18:00 +0200 Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote:

>Hello Nicolas,
>
>On Tue, 22 Oct 2019 13:44:08 +0000
>Nicolas Serafini <nicolas.serafini@sensefly.com> wrote:
>
>> +ifeq ($(BR2_PACKAGE_ELL),y)
>> +OFONO_DEPENDENCIES += ell
>> +OFONO_CONF_OPTS += --enable-external-ell
>> +else
>> +OFONO_CONF_OPTS += --disable-external-ell
>> +endif
>
>If I understand correctly the code, ell is in fact always needed by
>ofono. If ell is not available externally, it uses a bundled version of
>ell.

Yes, that's right.

>
>In this case, Buildroot prefers to always use the external library, so
>could you turn ell into a mandatory dependency of ofono, and pass
>--enable-external-ell unconditionally?

I also prefer to use only the external library.
In case of a bug it avoids maintaining a patch in two different packages.
I'm preparing a v3.

Thanks

Nicolas

>
>Thanks!
>
>Thomas

[Buildroot] [PATCH v2 1/2] package/ofono: remove forced autoreconf

[Thomas Petazzoni]

On Tue, 22 Oct 2019 13:44:07 +0000
Nicolas Serafini <nicolas.serafini@sensefly.com> wrote:

> The patch that required an autoreconf is no longer there.
> 
> Signed-off-by: Nicolas Serafini <nicolas.serafini@sensefly.com>
> ---
>  package/ofono/ofono.mk | 2 --
>  1 file changed, 2 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com