* [Buildroot] [PATCH 0 of 2] fix rpm autobuild failure
@ 2013-11-22 21:17 Thomas De Schampheleire
2013-11-22 21:17 ` [Buildroot] [PATCH 1 of 2] beecrypt: include sequence numbers in patch names Thomas De Schampheleire
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Thomas De Schampheleire @ 2013-11-22 21:17 UTC (permalink / raw)
To: buildroot
This series incorporates one patch from Vicente, and provides an alternative
fix to the recent rpm autobuild failure.
Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
---
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1 of 2] beecrypt: include sequence numbers in patch names
2013-11-22 21:17 [Buildroot] [PATCH 0 of 2] fix rpm autobuild failure Thomas De Schampheleire
@ 2013-11-22 21:17 ` Thomas De Schampheleire
2013-11-22 21:17 ` [Buildroot] [PATCH 2 of 2] beecrypt: unconditionally disable openpm Thomas De Schampheleire
2013-11-22 23:21 ` [Buildroot] [PATCH 0 of 2] fix rpm autobuild failure Peter Korsgaard
2 siblings, 0 replies; 4+ messages in thread
From: Thomas De Schampheleire @ 2013-11-22 21:17 UTC (permalink / raw)
To: buildroot
Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
[ThomasDS: minor change in commit message]
Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
---
...beecrypt-0001-cppglue.cxx-cplusplus-only.patch} | 0
...=> beecrypt-0002-icu-check-cross-compile.patch} | 0
2 files changed, 0 insertions(+), 0 deletions(-)
rename package/beecrypt/{beecrypt-cppglue.cxx-cplusplus-only.patch => beecrypt-0001-cppglue.cxx-cplusplus-only.patch} (100%)
rename package/beecrypt/{beecrypt-icu-check-cross-compile.patch => beecrypt-0002-icu-check-cross-compile.patch} (100%)
diff --git a/package/beecrypt/beecrypt-cppglue.cxx-cplusplus-only.patch b/package/beecrypt/beecrypt-0001-cppglue.cxx-cplusplus-only.patch
rename from package/beecrypt/beecrypt-cppglue.cxx-cplusplus-only.patch
rename to package/beecrypt/beecrypt-0001-cppglue.cxx-cplusplus-only.patch
diff --git a/package/beecrypt/beecrypt-icu-check-cross-compile.patch b/package/beecrypt/beecrypt-0002-icu-check-cross-compile.patch
rename from package/beecrypt/beecrypt-icu-check-cross-compile.patch
rename to package/beecrypt/beecrypt-0002-icu-check-cross-compile.patch
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 2 of 2] beecrypt: unconditionally disable openpm
2013-11-22 21:17 [Buildroot] [PATCH 0 of 2] fix rpm autobuild failure Thomas De Schampheleire
2013-11-22 21:17 ` [Buildroot] [PATCH 1 of 2] beecrypt: include sequence numbers in patch names Thomas De Schampheleire
@ 2013-11-22 21:17 ` Thomas De Schampheleire
2013-11-22 23:21 ` [Buildroot] [PATCH 0 of 2] fix rpm autobuild failure Peter Korsgaard
2 siblings, 0 replies; 4+ messages in thread
From: Thomas De Schampheleire @ 2013-11-22 21:17 UTC (permalink / raw)
To: buildroot
beecrypt does not properly handle its dependency on openpm, so that
applications using libbeecrypt are not aware that they also need openmp
(libgomp). This causes error messages during linking, such as (rpm):
..-gcc [..] -lbeecrypt -lbz2 -lz -lpopt -lpthread
libbecrypt.so: undefined reference to `GOMP_sections_end_nowait'
libbecrypt.so: undefined reference to `GOMP_parallel_end'
libbecrypt.so: undefined reference to `GOMP_sections_next'
libbecrypt.so: undefined reference to `GOMP_parallel_sections_start'
Because the openpm dependency of beecrypt is only used by rsa.c, to
parallelize the calculation of RSA cyphers, we can simply disable openmp in
beecrypt instead of trying to fix the way -lgomp is passed.
Fixes http://autobuild.buildroot.net/results/bcf/bcff4b81bfbb1191f97317b0945c74d948c9774b/
Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
---
package/beecrypt/beecrypt.mk | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
Note: this is an alternative fix to the above autobuild than the one Vicente
posted earlier today.
diff --git a/package/beecrypt/beecrypt.mk b/package/beecrypt/beecrypt.mk
--- a/package/beecrypt/beecrypt.mk
+++ b/package/beecrypt/beecrypt.mk
@@ -19,7 +19,8 @@ BEECRYPT_CONF_ENV = \
BEECRYPT_CONF_OPT = \
--without-java \
- --without-python
+ --without-python \
+ --disable-openmp
ifeq ($(BR2_PACKAGE_ICU),y)
# C++ support needs icu
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 0 of 2] fix rpm autobuild failure
2013-11-22 21:17 [Buildroot] [PATCH 0 of 2] fix rpm autobuild failure Thomas De Schampheleire
2013-11-22 21:17 ` [Buildroot] [PATCH 1 of 2] beecrypt: include sequence numbers in patch names Thomas De Schampheleire
2013-11-22 21:17 ` [Buildroot] [PATCH 2 of 2] beecrypt: unconditionally disable openpm Thomas De Schampheleire
@ 2013-11-22 23:21 ` Peter Korsgaard
2 siblings, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2013-11-22 23:21 UTC (permalink / raw)
To: buildroot
>>>>> "Thomas" == Thomas De Schampheleire <patrickdepinguin@gmail.com> writes:
> This series incorporates one patch from Vicente, and provides an alternative
> fix to the recent rpm autobuild failure.
Committed series, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-11-22 23:21 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-22 21:17 [Buildroot] [PATCH 0 of 2] fix rpm autobuild failure Thomas De Schampheleire
2013-11-22 21:17 ` [Buildroot] [PATCH 1 of 2] beecrypt: include sequence numbers in patch names Thomas De Schampheleire
2013-11-22 21:17 ` [Buildroot] [PATCH 2 of 2] beecrypt: unconditionally disable openpm Thomas De Schampheleire
2013-11-22 23:21 ` [Buildroot] [PATCH 0 of 2] fix rpm autobuild failure Peter Korsgaard
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox