Skip to content

zephyr build warnings: zephyr.h is deprecated, include <zephyr/kernel.h> instead #47

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
Closed
DhruvaG2000 opened this issue Sep 13, 2022 · 0 comments · Fixed by #49
Closed

zephyr build warnings: zephyr.h is deprecated, include <zephyr/kernel.h> instead #47

DhruvaG2000 opened this issue Sep 13, 2022 · 0 comments · Fixed by #49
Labels
bug Something isn't working

Comments

@DhruvaG2000
Copy link
Collaborator

Whenever I try to build a sample using latest zephyr, I get a bunch of the following errors:

/home/dhruva/zephyrproject/zephyr/include/zephyr/zephyr.h:13:2: warning: #warning "<zephyr/zephyr.h> is deprecated, include <zephyr/kernel.h> instead" [-Wcpp]
   13 | #warning "<zephyr/zephyr.h> is deprecated, include <zephyr/kernel.h> instead"
      |  ^~~~~~~
[51/164] Building CXX object zephyr/CMakeFiles/zephyr.dir/home/dhruva/zephyrproject/modules/lib/Arduino-Zephyr-API/cores/arduino/zephyrCommon.cpp.obj
In file included from /home/dhruva/zephyrproject/modules/lib/Arduino-Zephyr-API/cores/arduino/./Arduino.h:10,
                 from /home/dhruva/zephyrproject/modules/lib/Arduino-Zephyr-API/cores/arduino/zephyrCommon.cpp:7:
/home/dhruva/zephyrproject/zephyr/include/zephyr/zephyr.h:13:2: warning: #warning "<zephyr/zephyr.h> is deprecated, include <zephyr/kernel.h> instead" [-Wcpp]
   13 | #warning "<zephyr/zephyr.h> is deprecated, include <zephyr/kernel.h> instead"
@DhruvaG2000 DhruvaG2000 added the bug Something isn't working label Sep 13, 2022
DhruvaG2000 added a commit that referenced this issue Sep 17, 2022
@DhruvaG2000
variants: bunch all includes in common file
6846c25 
Also fix #47

Signed-off-by: Dhruva Gole <[email protected]>
@DhruvaG2000 DhruvaG2000 mentioned this issue Sep 17, 2022
Merged
DhruvaG2000 added a commit that referenced this issue Sep 17, 2022
@DhruvaG2000
variants: fix zephyr include warnings
fd7394e 
fix #47

Signed-off-by: Dhruva Gole <[email protected]>
DhruvaG2000 added a commit that referenced this issue Sep 17, 2022
@DhruvaG2000
variants: fix zephyr include warnings
ea681ae 
fix #47

Signed-off-by: Dhruva Gole <[email protected]>
DhruvaG2000 added a commit that referenced this issue Sep 22, 2022
@DhruvaG2000
include kernel.h instead of zephyr.h
3a5430a 
- zephyr.h is already deprecated, hence using kernel.h instead of it.
- fix #47
- affects cores/ and variants/ dirs

Signed-off-by: Dhruva Gole <[email protected]>
DhruvaG2000 added a commit that referenced this issue Sep 22, 2022
@DhruvaG2000
include kernel.h instead of zephyr.h
5a76d45 
- zephyr.h is already deprecated, hence using kernel.h instead of it.
- fix #47
- affects cores/ and variants/ dirs

Signed-off-by: Dhruva Gole <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

variants: bunch all includes in common file zephyrproject-rtos/gsoc-2022-arduino-core
1 participant
@DhruvaG2000