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
Refactor hal directory #13489
Refactor hal directory #13489
Conversation
60a03f3
to
fafa0c4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code change is fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
@0xc0170 Could you review and start the CI for this PR Thanks |
Why "hal/include/hal/"? |
CI started |
In CMake for illustration: Include paths are set to: |
We are following this convention for all components/libraries to avoid ambiguity for public headers. With this new include-style (and once the new build system is ready), all public headers will be references as |
Jenkins CI Test : ❌ FAILEDBuild Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
@0xc0170 Greentea test failure related to device issue, could you restart the greentea test
|
@0xc0170 Greentea test failed again but this is a problem with mbed_NUCLEO_F411RE#1 MRFE4 target in Hanna RAAS server so requested Qinghao to put the target into maintenance mode and He did that so and the target is no longer available. Could you restart the CI greentea test |
I restarted the entire pipeline to clear pr-head |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 2 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
@0xc0170 CI passed for this PR, please merge |
Summary of changes
Restructured
hal
as per the new directory structure proposal:Note:
Duplicated below TESTS/host_tests script into hal/tests/TESTS as the current build tool is looking for host_tests in that directory. Already raised an internal ticket to remove this duplicate scripts once build tools can able to pick the scripts from a common location
Impact of changes
None.
Migration actions required
None.
Documentation
To be updated
Pull request type
Test results
Manual testing: (Build for K64F target with GCC_ARM toolchain)
Reviewers
@0xc0170 @ARMmbed/mbed-os-core