* [Buildroot] [PATCH 1/2] package/icu: icu-uc: provide -lstdc++ flag for static build
@ 2015-07-22 21:47 Romain Naour
2015-07-22 21:47 ` [Buildroot] [PATCH 2/2] package/harfbuzz: fix test-unicode static linking issue Romain Naour
2015-07-23 21:34 ` [Buildroot] [PATCH 1/2] package/icu: icu-uc: provide -lstdc++ flag for static build Thomas Petazzoni
0 siblings, 2 replies; 5+ messages in thread
From: Romain Naour @ 2015-07-22 21:47 UTC (permalink / raw)
To: buildroot
icu-uc.pc must provide -lstdc++ for C application linked
statically with libicuuc.a.
Related to:
http://autobuild.buildroot.net/results/210/2107f9dfb39eeb6559fb4271c7af8b39aef521ca/
Signed-off-by: Romain Naour <romain.naour@openwide.fr>
---
.../icu/0005-fix-static-linking-with-icu-uc.patch | 34 ++++++++++++++++++++++
1 file changed, 34 insertions(+)
create mode 100644 package/icu/0005-fix-static-linking-with-icu-uc.patch
diff --git a/package/icu/0005-fix-static-linking-with-icu-uc.patch b/package/icu/0005-fix-static-linking-with-icu-uc.patch
new file mode 100644
index 0000000..965906b
--- /dev/null
+++ b/package/icu/0005-fix-static-linking-with-icu-uc.patch
@@ -0,0 +1,34 @@
+From ffff12fd321c7a056e796e74cc508726b0626ae0 Mon Sep 17 00:00:00 2001
+From: Romain Naour <romain.naour@openwide.fr>
+Date: Wed, 22 Jul 2015 22:43:25 +0200
+Subject: [PATCH] fix static linking with icu-uc
+
+During static linking with a C application and libicuuc.a,
+-lstdc++ is required.
+
+Add -lstdc++ in Libs.private of icu-uc.pc.
+
+Fixes:
+http://autobuild.buildroot.net/results/210/2107f9dfb39eeb6559fb4271c7af8b39aef521ca/
+
+Signed-off-by: Romain Naour <romain.naour@openwide.fr>
+---
+ source/Makefile.in | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/source/Makefile.in b/source/Makefile.in
+index 9db6c52..ca48e16 100644
+--- a/source/Makefile.in
++++ b/source/Makefile.in
+@@ -264,7 +264,7 @@ config/icu-uc.pc: config/icu.pc Makefile icudefs.mk
+ @echo "Description: $(PACKAGE_ICU_DESCRIPTION): Common and Data libraries" >> $@
+ @echo "Name: $(PACKAGE)-uc" >> $@
+ @echo "Libs:" '-L$${libdir}' "${ICULIBS_UC}" "${ICULIBS_DT}" >> $@
+- @echo "Libs.private:" '$${baselibs}' >> $@
++ @echo "Libs.private:" '$${baselibs}' -lstdc++ >> $@
+ @echo $@ updated.
+
+ config/icu-i18n.pc: config/icu.pc Makefile icudefs.mk
+--
+2.4.3
+
--
2.4.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 2/2] package/harfbuzz: fix test-unicode static linking issue
2015-07-22 21:47 [Buildroot] [PATCH 1/2] package/icu: icu-uc: provide -lstdc++ flag for static build Romain Naour
@ 2015-07-22 21:47 ` Romain Naour
2015-07-23 21:36 ` Thomas Petazzoni
2015-07-23 21:34 ` [Buildroot] [PATCH 1/2] package/icu: icu-uc: provide -lstdc++ flag for static build Thomas Petazzoni
1 sibling, 1 reply; 5+ messages in thread
From: Romain Naour @ 2015-07-22 21:47 UTC (permalink / raw)
To: buildroot
When linking test-unicode statically, it needs $(ICU_LIBS)
which contains all required flags.
Especially -lstdc++.
Fixes:
http://autobuild.buildroot.net/results/210/2107f9dfb39eeb6559fb4271c7af8b39aef521ca/
Signed-off-by: Romain Naour <romain.naour@openwide.fr>
---
.../0001-fix-static-linking-with-icu-uc.patch | 33 ++++++++++++++++++++++
1 file changed, 33 insertions(+)
create mode 100644 package/harfbuzz/0001-fix-static-linking-with-icu-uc.patch
diff --git a/package/harfbuzz/0001-fix-static-linking-with-icu-uc.patch b/package/harfbuzz/0001-fix-static-linking-with-icu-uc.patch
new file mode 100644
index 0000000..dd64393
--- /dev/null
+++ b/package/harfbuzz/0001-fix-static-linking-with-icu-uc.patch
@@ -0,0 +1,33 @@
+From 5752d3f17072aa97a056dc081e402f43e1a3ba32 Mon Sep 17 00:00:00 2001
+From: Romain Naour <romain.naour@openwide.fr>
+Date: Wed, 22 Jul 2015 23:26:23 +0200
+Subject: [PATCH] fix static linking with icu-uc
+
+When linking test-unicode statically it needs $(ICU_LIBS)
+which contains all required flags.
+Especially -lstdc++.
+
+Fixes:
+http://autobuild.buildroot.net/results/210/2107f9dfb39eeb6559fb4271c7af8b39aef521ca/
+
+Signed-off-by: Romain Naour <romain.naour@openwide.fr>
+---
+ test/api/Makefile.am | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/test/api/Makefile.am b/test/api/Makefile.am
+index 4ff14fa..314a09f 100644
+--- a/test/api/Makefile.am
++++ b/test/api/Makefile.am
+@@ -34,7 +34,7 @@ test_unicode_CPPFLAGS += $(GLIB_CFLAGS)
+ endif
+ if HAVE_ICU
+ test_unicode_CPPFLAGS += $(ICU_CFLAGS)
+-test_unicode_LDADD += $(top_builddir)/src/libharfbuzz-icu.la
++test_unicode_LDADD += $(top_builddir)/src/libharfbuzz-icu.la $(ICU_LIBS)
+ endif
+
+
+--
+2.4.3
+
--
2.4.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 1/2] package/icu: icu-uc: provide -lstdc++ flag for static build
2015-07-22 21:47 [Buildroot] [PATCH 1/2] package/icu: icu-uc: provide -lstdc++ flag for static build Romain Naour
2015-07-22 21:47 ` [Buildroot] [PATCH 2/2] package/harfbuzz: fix test-unicode static linking issue Romain Naour
@ 2015-07-23 21:34 ` Thomas Petazzoni
1 sibling, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2015-07-23 21:34 UTC (permalink / raw)
To: buildroot
Dear Romain Naour,
On Wed, 22 Jul 2015 23:47:02 +0200, Romain Naour wrote:
> icu-uc.pc must provide -lstdc++ for C application linked
> statically with libicuuc.a.
>
> Related to:
> http://autobuild.buildroot.net/results/210/2107f9dfb39eeb6559fb4271c7af8b39aef521ca/
>
> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
> ---
> .../icu/0005-fix-static-linking-with-icu-uc.patch | 34 ++++++++++++++++++++++
> 1 file changed, 34 insertions(+)
> create mode 100644 package/icu/0005-fix-static-linking-with-icu-uc.patch
Applied, thanks. Please submit the patch upstream!
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 2/2] package/harfbuzz: fix test-unicode static linking issue
2015-07-22 21:47 ` [Buildroot] [PATCH 2/2] package/harfbuzz: fix test-unicode static linking issue Romain Naour
@ 2015-07-23 21:36 ` Thomas Petazzoni
2015-08-06 8:39 ` Romain Naour
0 siblings, 1 reply; 5+ messages in thread
From: Thomas Petazzoni @ 2015-07-23 21:36 UTC (permalink / raw)
To: buildroot
Dear Romain Naour,
On Wed, 22 Jul 2015 23:47:03 +0200, Romain Naour wrote:
> When linking test-unicode statically, it needs $(ICU_LIBS)
> which contains all required flags.
> Especially -lstdc++.
>
> Fixes:
> http://autobuild.buildroot.net/results/210/2107f9dfb39eeb6559fb4271c7af8b39aef521ca/
>
> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
> ---
> .../0001-fix-static-linking-with-icu-uc.patch | 33 ++++++++++++++++++++++
> 1 file changed, 33 insertions(+)
> create mode 100644 package/harfbuzz/0001-fix-static-linking-with-icu-uc.patch
Applied, thanks. Please also submit upstream.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH 2/2] package/harfbuzz: fix test-unicode static linking issue
2015-07-23 21:36 ` Thomas Petazzoni
@ 2015-08-06 8:39 ` Romain Naour
0 siblings, 0 replies; 5+ messages in thread
From: Romain Naour @ 2015-08-06 8:39 UTC (permalink / raw)
To: buildroot
Hi Thomas,
Le 23/07/2015 23:36, Thomas Petazzoni a ?crit :
> Dear Romain Naour,
>
> On Wed, 22 Jul 2015 23:47:03 +0200, Romain Naour wrote:
>> When linking test-unicode statically, it needs $(ICU_LIBS)
>> which contains all required flags.
>> Especially -lstdc++.
>>
>> Fixes:
>> http://autobuild.buildroot.net/results/210/2107f9dfb39eeb6559fb4271c7af8b39aef521ca/
>>
>> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
>> ---
>> .../0001-fix-static-linking-with-icu-uc.patch | 33 ++++++++++++++++++++++
>> 1 file changed, 33 insertions(+)
>> create mode 100644 package/harfbuzz/0001-fix-static-linking-with-icu-uc.patch
>
> Applied, thanks. Please also submit upstream.
Applied upstream:
https://github.com/behdad/harfbuzz/pull/122
Best regards,
Romain
>
> Thomas
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-08-06 8:39 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-22 21:47 [Buildroot] [PATCH 1/2] package/icu: icu-uc: provide -lstdc++ flag for static build Romain Naour
2015-07-22 21:47 ` [Buildroot] [PATCH 2/2] package/harfbuzz: fix test-unicode static linking issue Romain Naour
2015-07-23 21:36 ` Thomas Petazzoni
2015-08-06 8:39 ` Romain Naour
2015-07-23 21:34 ` [Buildroot] [PATCH 1/2] package/icu: icu-uc: provide -lstdc++ flag for static build Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox