Skip to content

Commit

Permalink
zephyr: remove posix prefix on standard headers
Browse files Browse the repository at this point in the history
The upstream PR is zephyrproject-rtos/zephyr#43987

Basing off of that so that we can roll-back the many changes like
the one below in `.upstream` thrift.

```
 #ifdef CONFIG_SOC_POSIX
 #include <unistd.h>
 #else
 #include <posix/unistd.h>
 #endif
```

Fixes #61

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
  • Loading branch information
cfriedt committed Mar 22, 2022
1 parent a368a71 commit 39229e2
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 8 deletions.
6 changes: 0 additions & 6 deletions tests/lib/thrift/hello/src/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,16 +8,10 @@
#include <memory>
#include <stdexcept>

#ifdef CONFIG_SOC_POSIX
#include <pthread.h>
#include <unistd.h>
#include <signal.h>
#include <sys/socket.h>
#else
#include <posix/pthread.h>
#include <posix/unistd.h>
#include <posix/sys/socket.h>
#endif

#include "Hello.h"
#include "HelloHandler.h"
Expand Down
4 changes: 3 additions & 1 deletion west.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,5 +21,7 @@ manifest:
revision: master
- name: zephyr
remote: zephyrproject-rtos
revision: main
# Remove the need for 'posix/' prefix when including std headers
# https://github.com/zephyrproject-rtos/zephyr/pull/43987
revision: pull/43987/head
import: true

0 comments on commit 39229e2

Please sign in to comment.