Re: [PATCH 1/1] prtr-luausb Makefile: honor CFLAGS, CPPFLAGS

[Daurnimator <quae@...>]

On 24 Mar 2016 4:25 am, "Heinrich Schuchardt" <xypron.glpk@gmx.de> wrote:
>
> Some Linux distribution install libraries and include files in unusual
> places, e.g. Debian has these files:
>
> /usr/lib/x86_64-linux-gnu/liblua5.2.so.0
> /usr/include/lua5.2/lua.h
>
> With the patch the user can export CFLAGS and CPPFLAGS to support
> compilation of luausb on these systems, e.g.
>
> export LUA_VERSION="5.2"
> export CFLAGS="-I/usr/include/lua5.2/ -llua5.2"
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
>  Makefile | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index 02992d8..94b4aa1 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -13,8 +13,8 @@ LUA_VERSION?=5.3
>  PREFIX?=/usr
>  INSTALL_LUA=$(PREFIX)/share/lua/$(LUA_VERSION)
>  INSTALL_BIN=$(PREFIX)/lib/lua/$(LUA_VERSION)
> -CPPFLAGS=-Wall -Wextra -Werror -O2
> -CFLAGS=-fvisibility=hidden
> +CPPFLAGS+=-Wall -Wextra -Werror -O2
> +CFLAGS+=-fvisibility=hidden
>
>  ifeq ($(OS),Linux)
>  CPPFLAGS+=-I/usr/include/libusb-1.0 "-DLUAUSB_API=__attribute__((visibility(\"default\")))"
> --
> 2.1.4
>
>

Did you mean to send this to the libusb list? This is the lua list.