All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] package/trousers: needs host-pkgconf
@ 2016-08-21 21:36 Romain Naour
  2016-08-21 21:36 ` [Buildroot] [PATCH 2/2] package/trousers: fix musl build Romain Naour
  2016-08-22 14:46 ` [Buildroot] [PATCH 1/2] package/trousers: needs host-pkgconf Thomas Petazzoni
  0 siblings, 2 replies; 3+ messages in thread
From: Romain Naour @ 2016-08-21 21:36 UTC (permalink / raw)
  To: buildroot

Otherwise the build fail with this error:
configure.in:53: error: possibly undefined macro: AC_MSG_ERROR
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
configure.in:95: error: possibly undefined macro: AM_CONDITIONAL

Also, there is a PKG_CHECK_MODULES used for GTK.

Signed-off-by: Romain Naour <romain.naour@gmail.com>
---
 package/trousers/trousers.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/trousers/trousers.mk b/package/trousers/trousers.mk
index be6ce39..3d9e0ba 100644
--- a/package/trousers/trousers.mk
+++ b/package/trousers/trousers.mk
@@ -12,7 +12,7 @@ TROUSERS_LICENSE_FILES = LICENSE
 TROUSERS_INSTALL_STAGING = YES
 # Need autoreconf because of a patch touching configure.in and Makefile.am
 TROUSERS_AUTORECONF = YES
-TROUSERS_DEPENDENCIES = openssl
+TROUSERS_DEPENDENCIES = host-pkgconf openssl
 
 ifeq ($(BR2_PACKAGE_LIBICONV),y)
 TROUSERS_DEPENDENCIES += libiconv
-- 
2.5.5

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

* [Buildroot] [PATCH 2/2] package/trousers: fix musl build
  2016-08-21 21:36 [Buildroot] [PATCH 1/2] package/trousers: needs host-pkgconf Romain Naour
@ 2016-08-21 21:36 ` Romain Naour
  2016-08-22 14:46 ` [Buildroot] [PATCH 1/2] package/trousers: needs host-pkgconf Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Romain Naour @ 2016-08-21 21:36 UTC (permalink / raw)
  To: buildroot

The musl libc doesn't provide <bits/local_lim.h>, use <limits.h>
instead.

Fixes:
http://autobuild.buildroot.net/results/f71/f71edcf08f31029ef642415b8351ed14767c8e0e

Signed-off-by: Romain Naour <romain.naour@gmail.com>
---
 ..._param-include-limits.h-to-fix-musl-build.patch | 29 ++++++++++++++++++++++
 1 file changed, 29 insertions(+)
 create mode 100644 package/trousers/0003-tsp_tcsi_param-include-limits.h-to-fix-musl-build.patch

diff --git a/package/trousers/0003-tsp_tcsi_param-include-limits.h-to-fix-musl-build.patch b/package/trousers/0003-tsp_tcsi_param-include-limits.h-to-fix-musl-build.patch
new file mode 100644
index 0000000..f88e47a
--- /dev/null
+++ b/package/trousers/0003-tsp_tcsi_param-include-limits.h-to-fix-musl-build.patch
@@ -0,0 +1,29 @@
+From 66bbe0a87818a1e3c721780dc33b2d81fdf4a774 Mon Sep 17 00:00:00 2001
+From: Romain Naour <romain.naour@gmail.com>
+Date: Sun, 21 Aug 2016 23:32:10 +0200
+Subject: [PATCH] tsp_tcsi_param: include limits.h to fix musl build
+
+The musl libc doesn't provide <bits/local_lim.h>, use <limits.h>
+instead.
+
+Signed-off-by: Romain Naour <romain.naour@gmail.com>
+---
+ src/tspi/tsp_tcsi_param.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/tspi/tsp_tcsi_param.c b/src/tspi/tsp_tcsi_param.c
+index 670f86f..8f2b4e4 100644
+--- a/src/tspi/tsp_tcsi_param.c
++++ b/src/tspi/tsp_tcsi_param.c
+@@ -11,7 +11,7 @@
+ #include <stdlib.h>
+ #include <string.h>
+ #include <stdio.h>
+-#include <bits/local_lim.h>
++#include <limits.h>
+ #include "trousers/tss.h"
+ #include "trousers/trousers.h"
+ #include "trousers_types.h"
+-- 
+2.5.5
+
-- 
2.5.5

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

* [Buildroot] [PATCH 1/2] package/trousers: needs host-pkgconf
  2016-08-21 21:36 [Buildroot] [PATCH 1/2] package/trousers: needs host-pkgconf Romain Naour
  2016-08-21 21:36 ` [Buildroot] [PATCH 2/2] package/trousers: fix musl build Romain Naour
@ 2016-08-22 14:46 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2016-08-22 14:46 UTC (permalink / raw)
  To: buildroot

Hello,

On Sun, 21 Aug 2016 23:36:48 +0200, Romain Naour wrote:
> Otherwise the build fail with this error:
> configure.in:53: error: possibly undefined macro: AC_MSG_ERROR
>       If this token and others are legitimate, please use m4_pattern_allow.
>       See the Autoconf documentation.
> configure.in:95: error: possibly undefined macro: AM_CONDITIONAL
> 
> Also, there is a PKG_CHECK_MODULES used for GTK.
> 
> Signed-off-by: Romain Naour <romain.naour@gmail.com>
> ---
>  package/trousers/trousers.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Both applied to master, thanks! Please don't forget to send the
<limits.h> build fix upstream!

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

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

end of thread, other threads:[~2016-08-22 14:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-21 21:36 [Buildroot] [PATCH 1/2] package/trousers: needs host-pkgconf Romain Naour
2016-08-21 21:36 ` [Buildroot] [PATCH 2/2] package/trousers: fix musl build Romain Naour
2016-08-22 14:46 ` [Buildroot] [PATCH 1/2] package/trousers: needs host-pkgconf Thomas Petazzoni

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.