All of lore.kernel.org
 help / color / mirror / Atom feed
From: "François Perrad" <francois.perrad@gadz.org>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/2] lua: fix dynamic library install
Date: Fri, 5 Oct 2018 21:04:01 +0200	[thread overview]
Message-ID: <CAB0FRsvnbxk=HSbAw8DJenYW8jBj1YvYGfEJWFDNtF9dS-RUFQ@mail.gmail.com> (raw)
In-Reply-To: <CAB0FRsu4My55Udwq9NnQJ+zAcQi9MS+qYVi7Z3f3e16aeAk58Q@mail.gmail.com>

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>

  reply	other threads:[~2018-10-05 19:04 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-28 17:35 [Buildroot] [PATCH 1/2] lua: fix dynamic library install Fabrice Fontaine
2018-09-28 17:35 ` [Buildroot] [PATCH 2/2] haproxy: fix linking with lua Fabrice Fontaine
2018-10-01 13:17   ` Peter Korsgaard
2018-09-29  7:39 ` [Buildroot] [PATCH 1/2] lua: fix dynamic library install François Perrad
2018-10-05 19:04   ` François Perrad [this message]
2018-10-01 13:17 ` Peter Korsgaard
2018-10-05 20:04 ` Peter Korsgaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAB0FRsvnbxk=HSbAw8DJenYW8jBj1YvYGfEJWFDNtF9dS-RUFQ@mail.gmail.com' \
    --to=francois.perrad@gadz.org \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.