Skip to content

Conversation

@cfriedt
Copy link
Member

@cfriedt cfriedt commented Sep 25, 2025

Update the hostap module to use standard include paths for the ISO C header files time.h and signal.h

Also includes changes from #96582 to ensure that building works.

Testing done:

west build -p -b native_sim/native tests/net/wifi/configs -T wifi.build.hostapd_ap

Fixes #96566

On an x86_64 Ubuntu host, the following command would result in an error. ```shell west build -p -b native_sim/native tests/net/wifi/configs \ -T wifi.build.hostapd_ap ... In file included from /usr/include/bits/errno.h:26, from /usr/include/errno.h:28, from ~/zephyrproject/zephyr/scripts/native_simulator/\ /common/src/nce.c:24: /usr/include/linux/errno.h:1:10: fatal error: asm/errno.h: \ No such file or directory 1 | #include <asm/errno.h> | ^~~~~~~~~~~~~ compilation terminated. ``` Include `/usr/include/x86_64-linux-gnu` in the search path so that the build succeeds. Signed-off-by: Chris Friedt <cfriedt@tenstorrent.com>
@github-actions
Copy link

github-actions bot commented Sep 25, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hostap zephyrproject-rtos/hostap@0798bf0 (main) zephyrproject-rtos/hostap#105 zephyrproject-rtos/hostap#105/files

DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hostap DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Sep 25, 2025
Update the hostap module to use non-prefixed paths for the ISO C time.h and signal.h headers. Signed-off-by: Chris Friedt <cfriedt@tenstorrent.com>
@cfriedt cfriedt force-pushed the update-hostap-module-to-pr-105 branch from 02ae94d to db27a68 Compare September 25, 2025 16:30
Copy link
Member

@aescolar aescolar left a comment

Choose a reason for hiding this comment

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

Same as #96582 (review)

@cfriedt
Copy link
Member Author

cfriedt commented Sep 25, 2025

Closing in favour of #96587

@cfriedt cfriedt closed this Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: native port Host native arch port (native_sim) area: Wi-Fi Wi-Fi DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hostap

4 participants