[Buildroot] [PATCH 1/1] grantlee: fix lpthread build error

[Zoltan Gyarmati <mr.zoltan.gyarmati@gmail.com>]

On 11/13/2012 09:00 PM, Arnout Vandecappelle wrote:
> On 13/11/12 13:05, Zoltan at osuosl.org wrote:
>> From: "Zoltan Gyarmati"<mr.zoltan.gyarmati@gmail.com>
>>
>>
>> Signed-off-by: Zoltan Gyarmati<mr.zoltan.gyarmati@gmail.com>
>
> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> Tested-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
>  (it already worked for me, but at least this patch doesn't break things)
>
>  However, your patch still misses the following things:
>
> - The commit message is too vague.  It should describe the problem in 
> more detail,
> including the root cause.  For example:
>
> grantlee: fix -lpthread build error
>
> Build fails because of missing -lpthread with toolchains that need it. It
> gives an error log like:
> Linking CXX executable testgenerictypes_exec
> /home/buildroot12git/output/host/usr/lib/gcc/arm-buildroot-linux-uclibcgnueabi/4.6.3/../../../../arm-buildroot-linux-uclibcgnueabi/bin/ld: 
> /home/buildroot12git/output/host/usr/arm-buildroot-linux-uclibcgnueabi/sysroot/usr/lib/libQtCore.so: 
> undefined reference to symbol 'pthread_cancel'
> /home/buildroot12git/output/host/usr/lib/gcc/arm-buildroot-linux-uclibcgnueabi/4.6.3/../../../../arm-buildroot-linux-uclibcgnueabi/bin/ld: 
> note: 'pthread_cancel' is defined in DSO 
> /home/buildroot12git/output/host/usr/arm-buildroot-linux-uclibcgnueabi/sysroot/lib/libpthread.so.0 
> so try adding it to the linker command line
> /home/buildroot12git/output/host/usr/arm-buildroot-linux-uclibcgnueabi/sysroot/lib/libpthread.so.0: 
> could not read symbols: Invalid operation
> collect2: ld returned 1 exit status
> make[3]: *** [templates/tests/testgenerictypes_exec] Error 1
> make[3]: Leaving directory 
> `/home/buildroot12git/output/build/grantlee-0.2.0'
> make[2]: *** 
> [templates/tests/CMakeFiles/testgenerictypes_exec.dir/all] Error 2
> make[2]: Leaving directory 
> `/home/buildroot12git/output/build/grantlee-0.2.0'
> make[1]: *** [all] Error 2
> make[1]: Leaving directory 
> `/home/buildroot12git/output/build/grantlee-0.2.0'
> make: *** 
> [/home/buildroot12git/output/build/grantlee-0.2.0/.stamp_built] Error
>
>
> - The patch itself should contain an equally detailed description (can 
> be a copy
> of the commit message).  It should also have your Signed-off-by line.
>
>
>  Both of these are minor things, however.
>
>  Regards,
>  Arnout
>
>> ---
>>   .../grantlee-0.2.0-fix-lphread-link-error.patch    |   13 
>> +++++++++++++
>>   1 file changed, 13 insertions(+)
>>   create mode 100644 
>> package/grantlee/grantlee-0.2.0-fix-lphread-link-error.patch
>>
>> diff --git 
>> a/package/grantlee/grantlee-0.2.0-fix-lphread-link-error.patch 
>> b/package/grantlee/grantlee-0.2.0-fix-lphread-link-error.patch
>> new file mode 100644
>> index 0000000..8293172
>> --- /dev/null
>> +++ b/package/grantlee/grantlee-0.2.0-fix-lphread-link-error.patch
>> @@ -0,0 +1,13 @@
>> +diff -rupN grantlee-0.2.0_orig/templates/tests/CMakeLists.txt 
>> grantlee-0.2.0/templates/tests/CMakeLists.txt
>> +--- grantlee-0.2.0_orig/templates/tests/CMakeLists.txt 2012-11-13 
>> 12:11:44.583379820 +0100
>> ++++ grantlee-0.2.0/templates/tests/CMakeLists.txt    2012-11-13 
>> 11:35:18.000000000 +0100
>> +@@ -68,7 +68,8 @@ macro(GRANTLEE_TEMPLATES_UNIT_TESTS)
>> +                   ${_testresource_rcc_src}
>> +     )
>> +     add_test(${_testname} ${_testname}_exec )
>> +-    target_link_libraries(${_testname}_exec ${QT_QTTEST_LIBRARY} 
>> grantlee_core )
>> ++    find_package (Threads)
>> ++    target_link_libraries(${_testname}_exec ${QT_QTTEST_LIBRARY} 
>> ${CMAKE_THREAD_LIBS_INIT}  grantlee_core )
>> +
>> +     set_property(GLOBAL APPEND PROPERTY TEST_COVERAGE 
>> "${CMAKE_CURRENT_BINARY_DIR}/${_testname}_exec" )
>> +   endforeach(_testname)
>

Hi Arnout,

Thanks for checking! Should i resend it with corrections, or it will be 
ok like this?


-- 
br,
Zoltan Gyarmati
mail: mr.zoltan.gyarmati at gmail.com
freenode nick: zgyarmati