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

Move mbed-stubs-platform to the platform directory #14773

Merged
merged 6 commits into from
Jul 7, 2021
Merged

Move mbed-stubs-platform to the platform directory #14773

merged 6 commits into from
Jul 7, 2021

Commits on Jul 6, 2021

  1. CMake: unit-tests: CircularBuffer: Remove trailing whitespace

    @rwalton-arm
    rwalton-arm committed Jul 6, 2021
    Configuration menu
    Copy the full SHA
    3abc788 View commit details
    Browse the repository at this point in the history

  2. CMake: Move platform stubs to the mbed-os/platform directory 

    Move the platform stub library into the platform component directory.
This change is so we can avoid duplicating the mbed-os source tree in a
central UNITTESTS folder.
    @rwalton-arm
    rwalton-arm committed Jul 6, 2021
    Configuration menu
    Copy the full SHA
    40435fd View commit details
    Browse the repository at this point in the history

  3. CMake: unit-tests: Move mbed-headers-platform to platform directory 

    Move the header-only mbed-headers-platform library the unit test stubs
depend on into the platform component directory. This makes the platform
stubs more self contained and improves composition of the library.
    @rwalton-arm
    rwalton-arm committed Jul 6, 2021
    Configuration menu
    Copy the full SHA
    e270474 View commit details
    Browse the repository at this point in the history

  4. CMake: unit-tests: Make platform stub only depend on headers it uses 

    Previously the platform stub library depended on `mbed-headers`, which
is a collection of all available headers in mbed-os. To make it easier
to separate the library, only depend on the headers we're actually using.
    @rwalton-arm
    rwalton-arm committed Jul 6, 2021
    Configuration menu
    Copy the full SHA
    b73b38a View commit details
    Browse the repository at this point in the history

  5. CMake: unit-tests: Split platform headers out of mbed-stubs-headers 

    We have a central collection of "stub headers", which makes reasoning
about dependencies rather difficult, as it forces every stub library to
depend on all available stub headers. The standard approach would be for
each stub library to expose its public headers, and its dependents to
explicitly specify a dependency on the stub library containing the
headers it needs. This is a more modular design than creating a
header-only monolith library. Move the platform stub headers from this
central library into the mbed-stubs-platform library to increase
modularity.

mbed-stubs-connectivity now depends on the mbed-stubs-platform because
it requires some headers which were moved to mbed-stubs-platform.
    @rwalton-arm
    rwalton-arm committed Jul 6, 2021
    Configuration menu
    Copy the full SHA
    021baa6 View commit details
    Browse the repository at this point in the history

  6. CMake: unit-tests: Remove --coverage option from mbed-stubs-platform 

    We set the --coverage flag globally in UNITTESTS/CMakeLists.txt. We
shouldn't need to also set it on mbed-stubs-platform, so remove it.
    @rwalton-arm
    rwalton-arm committed Jul 6, 2021
    Configuration menu
    Copy the full SHA
    55f77ea View commit details
    Browse the repository at this point in the history