[Buildroot] [PATCH 1/2] lua: fix dynamic library install

[Buildroot] [PATCH 1/2] lua: fix dynamic library install

[Fabrice Fontaine]

In 0002-shared-libs-for-lua.patch, revision number is used to set
library name:
TO_SOLIB = liblua.so.$(R)

However, library is built using PKG_VERSION which is passed only during
build step:
$(CC) -o $@.$(PKG_VERSION) -shared -Wl,-soname="$@.$(PKG_VERSION)" $?

As a result, dynamic library is not installed in staging or target paths
since bump to lua 5.3.5

So, instead of replacing R by PKG_VERSION and passing this variable in
all steps, simply update R to 5

Fixes:
 - http://autobuild.buildroot.net/results/28b6672188bb0082ac1467d3b45904880e3634f3

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 .../lua/5.3.5/0003-fix-revision-number.patch  | 31 +++++++++++++++++++
 1 file changed, 31 insertions(+)
 create mode 100644 package/lua/5.3.5/0003-fix-revision-number.patch

diff --git a/package/lua/5.3.5/0003-fix-revision-number.patch b/package/lua/5.3.5/0003-fix-revision-number.patch
new file mode 100644
index 0000000000..ed2e0460eb
--- /dev/null
+++ b/package/lua/5.3.5/0003-fix-revision-number.patch
@@ -0,0 +1,31 @@
+Fix revision number
+
+In 0002-shared-libs-for-lua.patch, revision number is used to set 
+library name:
+TO_SOLIB = liblua.so.$(R)
+
+However, library is built using PKG_VERSION which is passed only during
+build step:
+$(CC) -o $@.$(PKG_VERSION) -shared -Wl,-soname="$@.$(PKG_VERSION)" $?
+
+As a result, dynamic library is not installed in staging or target paths
+since bump to lua 5.3.5
+
+So, instead of replacing R by PKG_VERSION and passing this variable in
+all steps, simply update R to 5
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+
+Index: b/Makefile
+===================================================================
+--- a/Makefile
++++ b/Makefile
+@@ -47,6 +47,6 @@
+
+ # Lua version and release.
+ V= 5.3
+-R= $V.4
++R= $V.5
+
+ # Targets start here.
+ all:    $(PLAT)
-- 
2.17.1

[Buildroot] [PATCH 2/2] haproxy: fix linking with lua

[Fabrice Fontaine]

Use LUA_LIB_NAME to give lua library name otherwise haproxy can't find
it

Fixes:
 - http://autobuild.buildroot.net/results/28b6672188bb0082ac1467d3b45904880e3634f3

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/haproxy/haproxy.mk | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/package/haproxy/haproxy.mk b/package/haproxy/haproxy.mk
index a799e69651..363fc4447f 100644
--- a/package/haproxy/haproxy.mk
+++ b/package/haproxy/haproxy.mk
@@ -25,7 +25,9 @@ endif
 
 ifeq ($(BR2_PACKAGE_LUA_5_3),y)
 HAPROXY_DEPENDENCIES += lua
-HAPROXY_MAKE_OPTS += USE_LUA=1
+HAPROXY_MAKE_OPTS += \
+	LUA_LIB_NAME=lua \
+	USE_LUA=1
 endif
 
 ifeq ($(BR2_PACKAGE_OPENSSL),y)
-- 
2.17.1

[Buildroot] [PATCH 1/2] lua: fix dynamic library install

[François Perrad]

Le ven. 28 sept. 2018 ? 19:35, Fabrice Fontaine <fontaine.fabrice@gmail.com>
a ?crit :

> In 0002-shared-libs-for-lua.patch, revision number is used to set
> library name:
> TO_SOLIB = liblua.so.$(R)
>
> However, library is built using PKG_VERSION which is passed only during
> build step:
> $(CC) -o $@.$(PKG_VERSION) -shared -Wl,-soname="$@.$(PKG_VERSION)" $?
>
> As a result, dynamic library is not installed in staging or target paths
> since bump to lua 5.3.5
>
> So, instead of replacing R by PKG_VERSION and passing this variable in
> all steps, simply update R to 5
>
> Fixes:
>  -
> http://autobuild.buildroot.net/results/28b6672188bb0082ac1467d3b45904880e3634f3
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
>

Acked-by: Francois Perrad <francois.perrad@gadz.org>


> ---
>  .../lua/5.3.5/0003-fix-revision-number.patch  | 31 +++++++++++++++++++
>  1 file changed, 31 insertions(+)
>  create mode 100644 package/lua/5.3.5/0003-fix-revision-number.patch
>
> diff --git a/package/lua/5.3.5/0003-fix-revision-number.patch
> b/package/lua/5.3.5/0003-fix-revision-number.patch
> new file mode 100644
> index 0000000000..ed2e0460eb
> --- /dev/null
> +++ b/package/lua/5.3.5/0003-fix-revision-number.patch
> @@ -0,0 +1,31 @@
> +Fix revision number
> +
> +In 0002-shared-libs-for-lua.patch, revision number is used to set
> +library name:
> +TO_SOLIB = liblua.so.$(R)
> +
> +However, library is built using PKG_VERSION which is passed only during
> +build step:
> +$(CC) -o $@.$(PKG_VERSION) -shared -Wl,-soname="$@.$(PKG_VERSION)" $?
> +
> +As a result, dynamic library is not installed in staging or target paths
> +since bump to lua 5.3.5
> +
> +So, instead of replacing R by PKG_VERSION and passing this variable in
> +all steps, simply update R to 5
> +
> +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +
> +Index: b/Makefile
> +===================================================================
> +--- a/Makefile
> ++++ b/Makefile
> +@@ -47,6 +47,6 @@
> +
> + # Lua version and release.
> + V= 5.3
> +-R= $V.4
> ++R= $V.5
> +
> + # Targets start here.
> + all:    $(PLAT)
> --
> 2.17.1
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180929/9ebe3a88/attachment.html>

[Buildroot] [PATCH 1/2] lua: fix dynamic library install

[Peter Korsgaard]

>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > In 0002-shared-libs-for-lua.patch, revision number is used to set
 > library name:
 > TO_SOLIB = liblua.so.$(R)

 > However, library is built using PKG_VERSION which is passed only during
 > build step:
 > $(CC) -o $@.$(PKG_VERSION) -shared -Wl,-soname="$@.$(PKG_VERSION)" $?

 > As a result, dynamic library is not installed in staging or target paths
 > since bump to lua 5.3.5

 > So, instead of replacing R by PKG_VERSION and passing this variable in
 > all steps, simply update R to 5

 > Fixes:
 >  - http://autobuild.buildroot.net/results/28b6672188bb0082ac1467d3b45904880e3634f3

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed, thanks.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH 2/2] haproxy: fix linking with lua

[Peter Korsgaard]

>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > Use LUA_LIB_NAME to give lua library name otherwise haproxy can't find
 > it

 > Fixes:
 >  - http://autobuild.buildroot.net/results/28b6672188bb0082ac1467d3b45904880e3634f3

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed, thanks.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH 1/2] lua: fix dynamic library install

[François Perrad]

Le sam. 29 sept. 2018 ? 09:39, Fran?ois Perrad <francois.perrad@gadz.org> a
?crit :

>
>
> Le ven. 28 sept. 2018 ? 19:35, Fabrice Fontaine <
> fontaine.fabrice at gmail.com> a ?crit :
>
>> In 0002-shared-libs-for-lua.patch, revision number is used to set
>> library name:
>> TO_SOLIB = liblua.so.$(R)
>>
>> However, library is built using PKG_VERSION which is passed only during
>> build step:
>> $(CC) -o $@.$(PKG_VERSION) -shared -Wl,-soname="$@.$(PKG_VERSION)" $?
>>
>> As a result, dynamic library is not installed in staging or target paths
>> since bump to lua 5.3.5
>>
>> So, instead of replacing R by PKG_VERSION and passing this variable in
>> all steps, simply update R to 5
>>
>> Fixes:
>>  -
>> http://autobuild.buildroot.net/results/28b6672188bb0082ac1467d3b45904880e3634f3
>>
>> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
>>
>
> Acked-by: Francois Perrad <francois.perrad@gadz.org>
>
>

This commit is useful in branch 2018.08.x too.

Fran?ois

---
>>  .../lua/5.3.5/0003-fix-revision-number.patch  | 31 +++++++++++++++++++
>>  1 file changed, 31 insertions(+)
>>  create mode 100644 package/lua/5.3.5/0003-fix-revision-number.patch
>>
>> diff --git a/package/lua/5.3.5/0003-fix-revision-number.patch
>> b/package/lua/5.3.5/0003-fix-revision-number.patch
>> new file mode 100644
>> index 0000000000..ed2e0460eb
>> --- /dev/null
>> +++ b/package/lua/5.3.5/0003-fix-revision-number.patch
>> @@ -0,0 +1,31 @@
>> +Fix revision number
>> +
>> +In 0002-shared-libs-for-lua.patch, revision number is used to set
>> +library name:
>> +TO_SOLIB = liblua.so.$(R)
>> +
>> +However, library is built using PKG_VERSION which is passed only during
>> +build step:
>> +$(CC) -o $@.$(PKG_VERSION) -shared -Wl,-soname="$@.$(PKG_VERSION)" $?
>> +
>> +As a result, dynamic library is not installed in staging or target paths
>> +since bump to lua 5.3.5
>> +
>> +So, instead of replacing R by PKG_VERSION and passing this variable in
>> +all steps, simply update R to 5
>> +
>> +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
>> +
>> +Index: b/Makefile
>> +===================================================================
>> +--- a/Makefile
>> ++++ b/Makefile
>> +@@ -47,6 +47,6 @@
>> +
>> + # Lua version and release.
>> + V= 5.3
>> +-R= $V.4
>> ++R= $V.5
>> +
>> + # Targets start here.
>> + all:    $(PLAT)
>> --
>> 2.17.1
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20181005/95a0fab6/attachment.html>

[Buildroot] [PATCH 1/2] lua: fix dynamic library install

[Peter Korsgaard]

>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > In 0002-shared-libs-for-lua.patch, revision number is used to set
 > library name:
 > TO_SOLIB = liblua.so.$(R)

 > However, library is built using PKG_VERSION which is passed only during
 > build step:
 > $(CC) -o $@.$(PKG_VERSION) -shared -Wl,-soname="$@.$(PKG_VERSION)" $?

 > As a result, dynamic library is not installed in staging or target paths
 > since bump to lua 5.3.5

 > So, instead of replacing R by PKG_VERSION and passing this variable in
 > all steps, simply update R to 5

 > Fixes:
 >  - http://autobuild.buildroot.net/results/28b6672188bb0082ac1467d3b45904880e3634f3

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed to 2018.08.x, thanks.

-- 
Bye, Peter Korsgaard