[Buildroot] [PATCH v3 1/2] package/lua-lunix: fix build with recent gcc

Arnout Vandecappelle arnout at mind.be
Thu Aug 5 17:05:47 UTC 2021



On 04/08/2021 15:46, Francois Perrad wrote:
> Fixes: http://autobuild.buildroot.net/results/749a11d5289c6fec3b2f236b9073fc1ab730d090/
> Signed-off-by: Francois Perrad <francois.perrad at gadz.org>
> ---
>  package/lua-lunix/0003-force-no-sysctl.patch | 32 ++++++++++++++++++++
>  1 file changed, 32 insertions(+)
>  create mode 100644 package/lua-lunix/0003-force-no-sysctl.patch
> 
> diff --git a/package/lua-lunix/0003-force-no-sysctl.patch b/package/lua-lunix/0003-force-no-sysctl.patch
> new file mode 100644
> index 000000000..3467e2713
> --- /dev/null
> +++ b/package/lua-lunix/0003-force-no-sysctl.patch
> @@ -0,0 +1,32 @@
> +From d9b65e306478f5d2356ef90eab8191984534238a Mon Sep 17 00:00:00 2001
> +From: Francois Perrad <francois.perrad at gadz.org>
> +Date: Wed, 4 Aug 2021 15:27:05 +0200
> +Subject: [PATCH] force no sysctl
> +
> +Signed-off-by: Francois Perrad <francois.perrad at gadz.org>
> +---
> + lunix-rel-20170920/lunix-20170920-1.rockspec | 9 +++++++++
> + 1 file changed, 9 insertions(+)
> +
> +diff --git a/lunix-rel-20170920/lunix-20170920-1.rockspec b/lunix-rel-20170920/lunix-20170920-1.rockspec
> +index 76cafeb..e42fd34 100644
> +--- a/lunix-rel-20170920/lunix-20170920-1.rockspec
> ++++ b/lunix-rel-20170920/lunix-20170920-1.rockspec
> +@@ -35,5 +35,14 @@ build = {
> + 				};
> + 			};
> + 		};
> ++		linux = {
> ++			modules = {
> ++				["unix"] = {
> ++					defines = {nil, nil, nil;
> ++						"HAVE_SYS_SYSCTL_H=0";

 That's a really dirty way of setting a define...

 Unless this patch is accepted upstream (unlikely, given that there was no
maintainer activity on [1] since more than a month), a much better way IMHO is
to set it from the .mk file:

LUA_LUNIX_BUILD_OPTS = CFLAGS="$(TARGET_CFLAGS) -DHAVE_SYS_SYSCTL_H=0"

(This may not work though, depending on the order of the flags added by luarocks.)


 Regards,
 Arnout

> ++					};
> ++				};
> ++			};
> ++		};
> + 	};
> + }
> +-- 
> +2.30.2
> +
> 


More information about the buildroot mailing list