Date: Thu, 3 Aug 2017 15:21:20 +0200
From: Natanael Copa <ncopa@alpinelinux.org>
To: Nathan Angelacos <nangel@alpinelinux.org>
Cc: alpine-aports@lists.alpinelinux.org
Subject: Re: [alpine-aports] [PATCH] community/lua-hiredis
Message-ID: <20170803152120.227f7e18@ncopa-desktop.copa.dup.pw>
In-Reply-To: <20170731235439.9492-1-nangel@alpinelinux.org>
References: <20170731235439.9492-1-nangel@alpinelinux.org>
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Mon, 31 Jul 2017 19:54:39 -0400
Nathan Angelacos <nangel@alpinelinux.org> wrote:

>  APKBUILD: hiredis.so now loads correctly
>  0003-: spell compatibility correctly
>  0004-: add backward compatibility define for luaL_checkint for Lua 5.3+
> ---

...

> @@ -53,21 +57,16 @@ _split() {
>  	cd "$srcdir"/$pkgname-$_ver || return 1
>  	pkgdesc="$pkgdesc (Lua $_ver)"
>  	install_if="lua$_ver $pkgname=$pkgver-r$pkgrel"
> -	${CC:-gcc} ${CFLAGS} $(pkg-config --cflags hiredis) \
> -		$(pkg-config --cflags lua$_ver) -shared -fPIC \
> +	${CC:-gcc} src/lua-hiredis.c ${CFLAGS} -shared -fPIC \
> +		$(pkg-config --cflags hiredis lua$_ver) \
> +		$(pkg-config --libs hiredis lua$_ver) \
>  		-I/usr/include/lua$_ver -I/usr/include/hiredis \
> -		-lhiredis -o hiredis.so src/lua-hiredis.c \
> -		|| return 1
> +		-o hiredis.so || return 1 
I fixed a whitespace damage (space before EOL) and pushed.

Thanks!



---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---