Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs: add ARM64 cross-compile instructions for Windows and Linux #1116

Conversation

micahsnyder
Copy link
Contributor

No description provided.

@ragusaa
Copy link
Contributor

ragusaa commented Dec 22, 2023

I followed the instructions for not having a sysroot, and am getting link errors for stuff that's in the curses library.
/usr/lib/gcc-cross/aarch64-linux-gnu/11/../../../../aarch64-linux-gnu/bin/ld: CMakeFiles/clamdtop.dir/clamdtop.c.o: in function resize': clamdtop.c:(.text+0xc0): undefined reference to stdscr'

@micahsnyder
Copy link
Contributor Author

@ragusaa Ah right... On some distros they split up ncurses into libncurses and libtinfo. stdscr is one they moved to the other library: https://stackoverflow.com/a/11269810

I think you can change the CURSES_LIBRARY variable in that CMAKE_TOOLCHAIN_ARM64.cmake to have both, like this:

set(CURSES_LIBRARY "/usr/lib/aarch64-linux-gnu/libncurses.a;/usr/lib/aarch64-linux-gnu/libtinfo.a")

You should not need to separately sudo apt-get install -y libtinfo:arm64. I think the ncurses package should have brought it along: https://packages.ubuntu.com/jammy/libncurses-dev

Copy link
Contributor

@ragusaa ragusaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see comments

@micahsnyder micahsnyder force-pushed the CLAM-2463-CLAM-2088-cross-compile-instructions branch from dc78308 to 02e8ecb Compare December 22, 2023 17:45
@micahsnyder micahsnyder merged commit b869154 into Cisco-Talos:main Mar 26, 2024
13 of 24 checks passed
@micahsnyder micahsnyder deleted the CLAM-2463-CLAM-2088-cross-compile-instructions branch March 26, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants