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

[Telink] use src/app/chip_data_model.cmake for data model inclusions #23225

Closed
andy31415 opened this issue Oct 17, 2022 · 2 comments
Closed

[Telink] use src/app/chip_data_model.cmake for data model inclusions #23225

andy31415 opened this issue Oct 17, 2022 · 2 comments

Comments

@andy31415
Copy link
Contributor

andy31415 commented Oct 17, 2022 •
edited
Loading

Reproduction steps

Telink CMakeFiles directly include /src/app/util/* and zzz_generated paths for includes.

This makes any common refactors tedious. Mbed and NRF already use a common src/app/chip_data_model.cmake

Platform

Telink
@s07641069 s07641069 mentioned this issue Oct 19, 2022
Merged
@s07641069
Copy link
Contributor

@andy31415 Please check if I did everything what you mean. If something missed just inform me.

@s07641069 s07641069 mentioned this issue Nov 3, 2022
Merged
@Damian-Nordic
Copy link
Contributor

This is already fixed.

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
@andy31415 @s07641069 @Damian-Nordic