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

Added missing "extern" to declarations: #29

Closed
wants to merge 1 commit into from
Closed

Conversation

aJunk
Copy link

@aJunk aJunk commented Jun 12, 2020

LOG_DLEP_RADIO in dlep_internal.h
LOG_NL80211 in nl80211_internal.h
LOG_MPR in mpr_internal.h
LOG_NHDP, LOG_NHDP_R, LOG_NHDP_W in nhdp_internal.h
LOG_OLSRV2, LOG_OLSRV2_R, LOG_OLSRV2_ROUTING, LOG_OLSRV2_W in olsrv2_internal.h

These caused symbol conflicts in GCC 10 as well as in GCC <10 when linking with '-fno-common'.
(see issue #28)

LOG_DLEP_RADIO in dlep_internal.h
LOG_NL80211 in nl80211_internal.h
LOG_MPR in mpr_internal.h
LOG_NHDP, LOG_NHDP_R, LOG_NHDP_W in nhdp_internal.h
LOG_OLSRV2, LOG_OLSRV2_R, LOG_OLSRV2_ROUTING, LOG_OLSRV2_W in olsrv2_internal.h
@mathiashro
Copy link
Contributor

@aJunk , do you see anything open? Recently there where some PR merges in regards to the missing "extern" statements.

@aJunk
Copy link
Author

aJunk commented Jul 26, 2022

@aJunk , do you see anything open? Recently there where some PR merges in regards to the missing "extern" statements.

Looks good to me now!

@aJunk aJunk closed this Jul 26, 2022
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