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

Allow build wrappers #341

Merged
merged 4 commits into from Nov 11, 2022
Merged

Allow build wrappers #341

merged 4 commits into from Nov 11, 2022

Commits on Nov 11, 2022

  1. Allow build wrappers 

    Closes tianocore#340

Adds two optional environment variables that, if specified, will be
used in lieu of the `build` command and its parameters during build.

1. `EDK_BUILD_CMD` - If present, the absolute path to an application
   to use for the edk build process that will be invoked instead of
   `build`. This is primarily used to allow a tool to wrap around
   `build` such that it will eventually call `build` internally.

2. `EDK_BUILD_PARAMS` - If present, these parameters will be passed
   to the build command. This is primarily used to pair wrapper-
   specific parameters with the wrapper passed in `EDK_BUILD_CMD`.

From a practical standpoint, these are the minimum changes needed
in the build tools to allow the CodeQL CLI to be invoked as a build
wrapper.

Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
    @makubacki
    makubacki committed Nov 11, 2022
    Configuration menu
    Copy the full SHA
    f168877 View commit details
    Browse the repository at this point in the history

  2. environment_variables.md: Move ext dep link back to proper section

    @makubacki
    makubacki committed Nov 11, 2022
    Configuration menu
    Copy the full SHA
    1ad607e View commit details
    Browse the repository at this point in the history

  3. Add build wrapper test to test_uefi_build.py

    @makubacki
    makubacki committed Nov 11, 2022
    Configuration menu
    Copy the full SHA
    c7df65a View commit details
    Browse the repository at this point in the history

  4. Use f-strings in new logging messages

    @makubacki
    makubacki committed Nov 11, 2022
    Configuration menu
    Copy the full SHA
    9f820c4 View commit details
    Browse the repository at this point in the history