diff mbox

[1/2] package/snappy: fix typo

Message ID 20170730144731.8961-1-bernd.kuhls@t-online.de
State Accepted
Headers show

Commit Message

Bernd Kuhls July 30, 2017, 2:47 p.m. UTC
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/snappy/snappy.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Petazzoni July 31, 2017, 6:50 p.m. UTC | #1
Hello,

On Sun, 30 Jul 2017 16:47:30 +0200, Bernd Kuhls wrote:
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
>  package/snappy/snappy.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Both applied, thanks. It is worth mentioning that the pthread related
tweak is probably no longer needed: both uClibc and musl now use the
"single library" approach, so we no longer need to link libstdc++
explicitly with libpthread.

So perhaps, we could remove this workaround? Of course, this would
require verifying by checking with the problematic toolchain
configurations.

Best regards,

Thomas
Peter Korsgaard Sept. 4, 2017, 8:59 p.m. UTC | #2
>>>>> "Bernd" == Bernd Kuhls <bernd.kuhls@t-online.de> writes:

 > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>

Committed to 2017.02.x, thanks.
diff mbox

Patch

diff --git a/package/snappy/snappy.mk b/package/snappy/snappy.mk
index 2b1da8cf5..81055594c 100644
--- a/package/snappy/snappy.mk
+++ b/package/snappy/snappy.mk
@@ -17,7 +17,7 @@  SNAPPY_INSTALL_STAGING = YES
 SNAPPY_CONF_OPTS = --disable-gtest
 
 # libsnappy links with libstdc++. Some libstdc++/arch variants use
-# pthread symbols for internal locking if built with thead
+# 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.