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.

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

cpu/native: Move dependencies to Makefile.dep #14109

Merged

Conversation

leandrolanzieri
Copy link
Contributor

Contribution description

This moves the native CPU dependencies to its Makefile.dep.

Testing procedure

Green CI should be enough as periph_uart is used always on startup.c.

Issues/PRs references

Part of #9913

@leandrolanzieri leandrolanzieri added Area: build system Area: Build system Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation labels May 20, 2020
@leandrolanzieri leandrolanzieri added this to the Release 2020.07 milestone May 20, 2020
@leandrolanzieri leandrolanzieri added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label May 20, 2020
Copy link
Contributor

@aabadie aabadie left a comment

Choose a reason for hiding this comment

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

LGTM. The modules loaded are unchanged compared to master:

make -C examples/hello-world/ info-modules --no-print-directory
  • this PR:
auto_init
board
core
core_init
core_msg
core_panic
cpu
native-drivers
periph
periph_common
periph_gpio
periph_init
periph_init_gpio
periph_init_pm
periph_init_uart
periph_pm
periph_uart
stdio_native
sys
  • master:
auto_init
board
core
core_init
core_msg
core_panic
cpu
native-drivers
periph
periph_common
periph_gpio
periph_init
periph_init_gpio
periph_init_pm
periph_init_uart
periph_pm
periph_uart
stdio_native
sys

The list is exactly the same.

ACK

@aabadie aabadie merged commit 4b7ae16 into RIOT-OS:master May 20, 2020
@leandrolanzieri leandrolanzieri deleted the pr/cpu/native/cleanup_dependencies branch May 20, 2020 17:11
@leandrolanzieri
Copy link
Contributor Author

Thanks for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: build system Area: Build system CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants