[Buildroot] [PATCH 1/1] package/wireshark: link with -latomic if needed

[Arnout Vandecappelle]

On 03/08/2021 10:26, Fabrice Fontaine wrote:
> Fix the following build failure raised since bump to version 3.4.7 in
> commit bb75c4b541fac144b53b63248e235f22ba1d25ad:
> 
> /tmp/instance-5/output-1/host/opt/ext-toolchain/bin/../lib/gcc/sparc-buildroot-linux-uclibc/9.3.0/../../../../sparc-buildroot-linux-uclibc/bin/ld: ui/qt/CMakeFiles/qtui.dir/sequence_diagram.cpp.o: undefined reference to symbol '__atomic_compare_exchange_4@@LIBATOMIC_1.0'
> /tmp/instance-5/output-1/host/opt/ext-toolchain/bin/../lib/gcc/sparc-buildroot-linux-uclibc/9.3.0/../../../../sparc-buildroot-linux-uclibc/bin/ld: /tmp/instance-5/output-1/host/sparc-buildroot-linux-uclibc/sysroot/lib/libatomic.so.1: error adding symbols: DSO missing from command line
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/6617ee0e0046a0452a1515b89e9c704b1c125ec4
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>

 I double-checked that wireshark's CMakeLists.txt doesn't do anything evil with
CMAKE_EXE_LINKER_FLAGS - it actually does, but only for MSVC.

 Applied to master, thanks.

 Regards,
 Arnout

> ---
>  package/wireshark/wireshark.mk | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/package/wireshark/wireshark.mk b/package/wireshark/wireshark.mk
> index e852083de6..6ee4b20391 100644
> --- a/package/wireshark/wireshark.mk
> +++ b/package/wireshark/wireshark.mk
> @@ -43,6 +43,10 @@ endef
>  
>  WIRESHARK_PRE_BUILD_HOOKS += WIRESHARK_BUILD_LEMON_TOOL
>  
> +ifeq ($(BR2_TOOLCHAIN_HAS_LIBATOMIC),y)
> +WIRESHARK_CONF_OPTS += -DCMAKE_EXE_LINKER_FLAGS=-latomic
> +endif
> +
>  ifeq ($(BR2_GCC_ENABLE_LTO),y)
>  WIRESHARK_CONF_OPTS += -DENABLE_LTO=ON
>  else
>