* [Buildroot] [PATCH v2] package/lua-curl: add variant definition of lcurl_url_t
@ 2019-02-24 19:51 Francois Perrad
2019-02-24 21:01 ` Thomas Petazzoni
0 siblings, 1 reply; 2+ messages in thread
From: Francois Perrad @ 2019-02-24 19:51 UTC (permalink / raw)
To: buildroot
Fixes: http://autobuild.buildroot.org/results/8d76ad49837b368a7bba3c3dfd445a035471268a/
Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
...dd-variant-definition-of-lcurl_url_t.patch | 40 +++++++++++++++++++
1 file changed, 40 insertions(+)
create mode 100644 package/lua-curl/0001-add-variant-definition-of-lcurl_url_t.patch
diff --git a/package/lua-curl/0001-add-variant-definition-of-lcurl_url_t.patch b/package/lua-curl/0001-add-variant-definition-of-lcurl_url_t.patch
new file mode 100644
index 000000000..0fdd0760d
--- /dev/null
+++ b/package/lua-curl/0001-add-variant-definition-of-lcurl_url_t.patch
@@ -0,0 +1,40 @@
+From 28ac0c5e642c9f7e3091a60268745f7cab4cc92b Mon Sep 17 00:00:00 2001
+From: Francois Perrad <francois.perrad@gadz.org>
+Date: Sat, 23 Feb 2019 15:23:26 +0100
+Subject: [PATCH] add variant definition of lcurl_url_t
+
+like in lceasy.h
+
+Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
+---
+ src/lcurlapi.h | 9 +++++++++
+ 1 file changed, 9 insertions(+)
+
+diff --git a/Lua-cURLv3-0.3.9/src/lcurlapi.h b/Lua-cURLv3-0.3.9/src/lcurlapi.h
+index 4dd4672..862c94f 100644
+--- a/Lua-cURLv3-0.3.9/src/lcurlapi.h
++++ b/Lua-cURLv3-0.3.9/src/lcurlapi.h
+@@ -19,11 +19,20 @@
+
+ #if LCURL_CURL_VER_GE(7,62,0)
+
++#if LCURL_CC_SUPPORT_FORWARD_TYPEDEF
+ typedef struct lcurl_url_tag {
+ CURLU *url;
+
+ int err_mode;
+ }lcurl_url_t;
++#else
++struct lcurl_url_tag {
++ CURLU *url;
++
++ int err_mode;
++};
++#define lcurl_url_t struct lcurl_url_tag
++#endif
+
+ int lcurl_url_create(lua_State *L, int error_mode);
+
+--
+2.17.1
+
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [Buildroot] [PATCH v2] package/lua-curl: add variant definition of lcurl_url_t
2019-02-24 19:51 [Buildroot] [PATCH v2] package/lua-curl: add variant definition of lcurl_url_t Francois Perrad
@ 2019-02-24 21:01 ` Thomas Petazzoni
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2019-02-24 21:01 UTC (permalink / raw)
To: buildroot
Hello Fran?ois,
On Sun, 24 Feb 2019 20:51:32 +0100
Francois Perrad <fperrad@gmail.com> wrote:
> Fixes: http://autobuild.buildroot.org/results/8d76ad49837b368a7bba3c3dfd445a035471268a/
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> ---
> ...dd-variant-definition-of-lcurl_url_t.patch | 40 +++++++++++++++++++
> 1 file changed, 40 insertions(+)
> create mode 100644 package/lua-curl/0001-add-variant-definition-of-lcurl_url_t.patch
Your commit messages are really too terse, they should contain more
details about the problem. I've extended the commit log and applied to
master.
Also, I noticed that packages built by the luarocks infrastructure have
code being built during the "installation to target" step, and nothing
built in the normal "build" step. Is this expected ? Can this be fixed ?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-02-24 21:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-24 19:51 [Buildroot] [PATCH v2] package/lua-curl: add variant definition of lcurl_url_t Francois Perrad
2019-02-24 21:01 ` Thomas Petazzoni
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.