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

Remove -lanl when WITH_ADNS is unset #415

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@ffontaine
Contributor

ffontaine commented Apr 3, 2017

Do not add -lanl to BROKER_LIBS for all Linux builds.
Indeed, -lanl is only needed for getaddrinfo_a which is only used in
_mosquitto_try_connect_step1 when WITH_ADNS is set

Signed-off-by: Fabrice Fontaine fontaine.fabrice@gmail.com

Remove -lanl when WITH_ADNS is unset
Do not add -lanl to BROKER_LIBS for all Linux builds.
Indeed, -lanl is only needed for getaddrinfo_a which is only used in
_mosquitto_try_connect_step1 when WITH_ADNS is set

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
@ralight

This comment has been minimized.

Show comment
Hide comment
@ralight

ralight May 5, 2017

Contributor

Thank you for spotting this, and sorry for the delay in responding. I've pushed the commit to the fixes branch so I'm closing this PR.

439f39b

Contributor

ralight commented May 5, 2017

Thank you for spotting this, and sorry for the delay in responding. I've pushed the commit to the fixes branch so I'm closing this PR.

439f39b

@ralight ralight closed this May 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment