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.

Sign up for GitHub

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

Jump to bottom

Missing extern "C" in "components/esp_wifi/include/esp_wifi_default.h" (IDFGH-2370) #4497

Closed
svenbieg opened this issue Dec 16, 2019 · 5 comments
Closed

Missing extern "C" in "components/esp_wifi/include/esp_wifi_default.h" (IDFGH-2370) #4497

svenbieg opened this issue Dec 16, 2019 · 5 comments

Comments

@svenbieg
Copy link

I am calling esp_netif_create_default_wifi_ap and esp_netif_create_default_wifi_sta from a cpp-file. Usually there is an #ifdef __cplusplus extern "C" statement at the beginning of the header.
@github-actions github-actions bot changed the title Missing extern "C" in "components/esp_wifi/include/esp_wifi_default.h" Missing extern "C" in "components/esp_wifi/include/esp_wifi_default.h" (IDFGH-2370) Dec 16, 2019
@igrr igrr mentioned this issue Dec 17, 2019
Closed
@svenbieg
Copy link
Author

Same as IDFGH-2241

@anderstogern anderstogern mentioned this issue Dec 20, 2019
Closed
@anderstogern
Copy link

It seems this issue is still around!

With latest master the extern C deifnition is still missing. Adding it manually fixes the problem and compilation completes successfully.

#4389 focuses on the order of the initializers and refers to this as a separate issue, so I think that this issue should be reopened.

@svenbieg svenbieg reopened this Dec 22, 2019
@Alvin1Zhang
Copy link
Collaborator

@svenbieg @anderstogern Thanks for reporting, please help check whether the issue still happens at your side, thanks.

@svenbieg
Copy link
Author

svenbieg commented Jan 9, 2020

It's working now, thank You!

@Alvin1Zhang
Copy link
Collaborator

@svenbieg Thanks for the updates.

0xFEEDC0DE64 pushed a commit to 0xFEEDC0DE64/esp-idf that referenced this issue May 5, 2021
@me-no-dev @geeksville
RE: Change check for BT_ENABLE to really be a check for BLUEDROID_ENA…
591c438 
…BLED (espressif#4497)

* Change check for CONFIG_BT_ENABLE to really be a check for CONFIG_BLUEDROID_ENABLED

Which is really what should have been tested.  This allows use of
the Arduino layer with the newer Nimble stack for those that don't want
to use Bluedroid.

In support of meshtastic/firmware#266

* Change check for CONFIG_BT_ENABLE to really be a check for CONFIG_BLUEDROID_ENABLED

Which is really what should have been tested.  This allows use of
the Arduino layer with the newer Nimble stack for those that don't want
to use Bluedroid.

In support of meshtastic/firmware#266

* wifi prov changes

* merge fixes

Co-authored-by: geeksville <kevinh@geeksville.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@anderstogern @svenbieg @Alvin1Zhang