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

build: add option to disable bsd library inclusion #2587

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Ansuel
Copy link

@Ansuel Ansuel commented Oct 28, 2023

It might be needed to disable bsd inclusion and fallback to the compat functions even if bsd headers are detected.

This is the case when multiple library are cross-compiled and someone wants to explicitly compile aircrack-ng without linking to bsd library.

With the current implementation, if a bsd header is detected, the bsd library is always linked even if unwanted. Add option to configure this with the combo --with-libbsd=yes|no|auto with auto set by default.

Also add an extra featurw with introducing the possibility of requiring the bsd library and fail the configure phase.

It might be needed to disable bsd inclusion and fallback to the compat
functions even if bsd headers are detected.

This is the case when multiple library are cross-compiled and someone
wants to explicitly compile aircrack-ng without linking to bsd library.

With the current implementation, if a bsd header is detected, the bsd
library is always linked even if unwanted. Add option to configure this
with the combo --with-libbsd=yes|no|auto with auto set by default.

Also add an extra featurw with introducing the possibility of requiring
the bsd library and fail the configure phase.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Musl or recent glibc added support for these additional string function,
strlcat and strlcpy hence the compat function are not needed and the
builtin version can be used instead.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
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.

None yet

1 participant