diff options
| author | YoongHM <yoonghm@users.noreply.github.com> | 2020-09-29 22:10:54 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-09-29 17:10:54 +0300 |
| commit | 448c435b1b5ff747d0d84c49a23435a83f0d40ac (patch) | |
| tree | 60bcd4d536a287b3f89bfc55d93195afd2529bfc /deps/Makefile | |
| parent | ef33252c434ff25991ff75e2c7e4646bc9ad5ff7 (diff) | |
| download | redis-448c435b1b5ff747d0d84c49a23435a83f0d40ac.tar.gz | |
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 <oran@redislabs.com>
Diffstat (limited to 'deps/Makefile')
| -rw-r--r-- | deps/Makefile | 4 |
1 files changed, 2 insertions, 2 deletions
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) |