[offline_compiler/tests] Add -lexecinfo to resolve `backtrace` and `backtrace_symbols`

[krupenik]

[JacekDanecki]

I don't think libexecinfo library is available in Linux distributions used by Neo CI. It's only available on Alpine and few other distros, see: https://repology.org/project/libexecinfo/versions If you want to make specific changes for Alpine, you can modify function get_os_release_info and add support for Alpine, and then use it in cmake file similar like in package.cmake

[krupenik]

I don't think libexecinfo library is available in Linux distributions used by Neo CI. It's only available on Alpine and few other distros, see: https://repology.org/project/libexecinfo/versions If you want to make specific changes for Alpine, you can modify function get_os_release_info and add support for Alpine, and then use it in cmake file similar like in package.cmake

thanks, looking into it

[krupenik]

@JacekDanecki what's your preferred approach, scope -lexecinfo to Alpine/Adelie or replace with libunwind (https://repology.org/project/libunwind/versions)?

[JacekDanecki]

I'd prefer preparation changes specific for Alpine, as it'll not have impact on binaries created for another distributions. libunwind is interesting proposal, but using it we could add some old bugs to Neo code, as for example: https://github.com/libunwind/libunwind/issues/94. This was fixed in libunwind 1.5-rc2, which is not available in all distributions yet.

[JablonskiMateusz]

@krupenik could you confirm if it is still needed?

[krupenik]

@krupenik could you confirm if it is still needed?

to the best of my knowledge — yes, for non-glibc builds it is still needed.

[JablonskiMateusz]

@krupenik could you confirm if it is still needed?

to the best of my knowledge — yes, for non-glibc builds it is still needed.

Could you detect the need on cmake level? based on some package detection add this execinfo dependency

[krupenik]

yeah, will try. I’m in Ukraine, so this might get postponed a bit, sorry :-)

On 6 Jul 2022, at 11:38, Mateusz Jablonski @.***> wrote:

 @krupenik could you confirm if it is still needed?

to the best of my knowledge — yes, for non-glibc builds it is still needed.

Could you detect the need on cmake level? based on some package detection add this execinfo dependency

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.