snappy: fix static build with threads
PATCH 1/1 snappy: fix static build with threads
Commit 0e2ee0e6e8076 (package.snappy: fix static builds with threads) added LIBS=-pthread to configure options to fix static build with threads on some architectures. This works for autotools packages. Then, commit d26282b3aa14cb (package/snappy: bump version, switch to cmake) switched to cmake that does not understand the 'LIBS' parameter.
Remove this LIBS parameter to fix static build with threads. Build for Cortex-M4 target indicates that the original issue is fixed now.
Fixes:
http://autobuild.buildroot.net/results/8dc/8dc9e147e5f41a979aafdfb1ab98121e9cd491b8/
http://autobuild.buildroot.net/results/9e8/9e8405360d1da6824a741fb4ac45495f9e806358/
http://autobuild.buildroot.net/results/500/50095cae347eb95a6912df33f455b6284fee73d4/
--- package/snappy/snappy.mk | 9 --------- 1 file changed, 9 deletions(-)
package/snappy/snappy.mk +0 -9
@@ -11,13 +11,4 @@ SNAPPY_LICENSE_FILES = COPYING SNAPPY_INSTALL_STAGING = YES SNAPPY_CONF_OPTS = -DSNAPPY_BUILD_TESTS=OFF -# libsnappy links with libstdc++. Some libstdc++/arch variants use -# pthread symbols for internal locking if built with thread -# support. libstdc++ does not have a .pc file, and its .la file does -# not mention -pthread. So, static linkig to libstdc++ will fail if -# -pthread is not explicity linked to. Only do that for static builds. -ifeq ($(BR2_STATIC_LIBS)$(BR2_TOOLCHAIN_HAS_THREADS),yy) -SNAPPY_CONF_OPTS += LIBS=-pthread -endif - $(eval $(cmake-package))
Signature · 1 line
2.13.2
PATCH 1/1 snappy: fix static build with threads
Baruch, All,
On 2017-08-09 14:35 +0300, Baruch Siach spake thusly:
Quoted 8 lines
Commit
0e2ee0e6e8076(package.snappy: fix static builds with threads) added LIBS=-pthread to configure options to fix static build with threads on some architectures. This works for autotools packages. Then, commitd26282b3aa14cb(package/snappy: bump version, switch to cmake) switched to cmake that does not understand the 'LIBS' parameter.Remove this LIBS parameter to fix static build with threads. Build for Cortex-M4 target indicates that the original issue is fixed now.
We already have https://patchwork.ozlabs.org/patch/797578/
in which I replied that the underlying reason that the trick is no longer needed is that we now explicitly disable building the test suite, and so the static linking issue no longer happens.
It's not that the cortex-m4 issue has been fixed, and I think it has not. Instead, we now explicitly disable building the offending code.
I'm marking your patch as rejected, because we already have Bernd's.
Thanks! :-)
Regards,
Yann E. MORIN.
Quoted 33 lines
Fixes:
http://autobuild.buildroot.net/results/8dc/8dc9e147e5f41a979aafdfb1ab98121e9cd491b8/
http://autobuild.buildroot.net/results/9e8/9e8405360d1da6824a741fb4ac45495f9e806358/
http://autobuild.buildroot.net/results/500/50095cae347eb95a6912df33f455b6284fee73d4/Cc Yann E. MORIN <[email protected]>Cc Bernd Kuhls <[email protected]>Signed-off-by Baruch Siach <[email protected]>--- package/snappy/snappy.mk | 9 --------- 1 file changed, 9 deletions(-)package/snappy/snappy.mk +0
-9@@ -11,13 +11,4 @@ SNAPPY_LICENSE_FILES = COPYING SNAPPY_INSTALL_STAGING = YES SNAPPY_CONF_OPTS = -DSNAPPY_BUILD_TESTS=OFF -# libsnappy links with libstdc++. Some libstdc++/arch variants use -# pthread symbols for internal locking if built with thread -# support. libstdc++ does not have a .pc file, and its .la file does -# not mention -pthread. So, static linkig to libstdc++ will fail if -# -pthread is not explicity linked to. Only do that for static builds. -ifeq ($(BR2_STATIC_LIBS)$(BR2_TOOLCHAIN_HAS_THREADS),yy) -SNAPPY_CONF_OPTS += LIBS=-pthread -endif - $(eval $(cmake-package))Signature · 1 line
2.13.2
Signature · 6 lines
.-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------'