[vlc-devel] [PATCH 4/5] contrib: lua: rewrite prefix in pkg-config file
robux4 at ycbcr.xyz
Tue Apr 7 08:14:06 CEST 2020
Can't change_prefix.sh be used for that ?
On 2020-04-06 15:56, Alexandre Janniaux wrote:
> This patch enable the writing of the pkg-config file for all platforms
> and enforce the correct prefix= value in the file. Previously the
> pkg-config file was never installed for linux platforms so it was using
> the file detection in the configure.ac, which is the last method of
> If a newer lua was detected that was not in the contrib file
> autodetection, it would build the lua contrib but still configure with
> the newer one, which means that contrib includes will be used and then
> it will be linked to the detected external lua.
> In particular, on Archlinux, when using the default lua (5.3) version,
> the contrib would be built but it will still link lua5.3 which has no
> symbol for the luaL_register function used when header version is less
> than or equal to 5.2.
> With previous change, as `lua >= 5.1` is tested first, contrib will be
> used first if built whatever the version of lua on the platform, so in
> the worst case we only build lua from the contrib too often (like
> previously) instead of failing link on some platforms too.
> contrib/src/lua/rules.mak | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
> diff --git a/contrib/src/lua/rules.mak b/contrib/src/lua/rules.mak
> index bf5671572c..87de70b7f4 100644
> --- a/contrib/src/lua/rules.mak
> +++ b/contrib/src/lua/rules.mak
> @@ -83,9 +83,9 @@ endif
> cd $< && $(HOSTVARS) $(MAKE) install INSTALL_TOP="$(PREFIX)"
> ifdef HAVE_WIN32
> cd $< && $(RANLIB) "$(PREFIX)/lib/liblua.a"
> mkdir -p -- "$(PREFIX)/lib/pkgconfig"
> - cp $</etc/lua.pc "$(PREFIX)/lib/pkgconfig/"
> + sed "s#^prefix=.*#prefix=$(PREFIX)#" $</etc/lua.pc > "$(PREFIX)/lib/pkgconfig/lua.pc"
> touch $@
> .sum-luac: .sum-lua
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
More information about the vlc-devel