From 448c435b1b5ff747d0d84c49a23435a83f0d40ac Mon Sep 17 00:00:00 2001 From: YoongHM Date: Tue, 29 Sep 2020 22:10:54 +0800 Subject: Fix compilation warnings in Lua and jemalloc dependencies (#7785) - The argument `u` in for `ar` is ignored (and generates warnings since `D` became the default. All it does is avoid updating unchanged objects (shouldn't have any impact on our build) - Enable `LUA_USE_MKSTEMP` to force the use of `mkstemp()` instead of `tmpname()` (which is dead code in redis anyway). - Remove unused variable `c` in `f_parser()` - Removed misleadingly indented space in `luaL_loadfile()` and ``addfield()` Co-authored-by: Oran Agra --- deps/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'deps/Makefile') diff --git a/deps/Makefile b/deps/Makefile index b90bbd2b7..ff16ee9e1 100644 --- a/deps/Makefile +++ b/deps/Makefile @@ -69,13 +69,13 @@ ifeq ($(uname_S),SunOS) LUA_CFLAGS= -D__C99FEATURES__=1 endif -LUA_CFLAGS+= -O2 -Wall -DLUA_ANSI -DENABLE_CJSON_GLOBAL -DREDIS_STATIC='' $(CFLAGS) +LUA_CFLAGS+= -O2 -Wall -DLUA_ANSI -DENABLE_CJSON_GLOBAL -DREDIS_STATIC='' -DLUA_USE_MKSTEMP $(CFLAGS) LUA_LDFLAGS+= $(LDFLAGS) # lua's Makefile defines AR="ar rcu", which is unusual, and makes it more # challenging to cross-compile lua (and redis). These defines make it easier # to fit redis into cross-compilation environments, which typically set AR. AR=ar -ARFLAGS=rcu +ARFLAGS=rc lua: .make-prerequisites @printf '%b %b\n' $(MAKECOLOR)MAKE$(ENDCOLOR) $(BINCOLOR)$@$(ENDCOLOR) -- cgit v1.2.1