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

Conditional tools #576

Merged
merged 5 commits into from
Aug 2, 2024
Merged

Conditional tools #576

merged 5 commits into from
Aug 2, 2024

Commits on Aug 2, 2024

  1. input: move tool fingerprint mask into steps 

    To calculate the variant-id, the applicable tools for each step must be
known. The variant-id of a step is calculated already in its
constructor. So far this worked because the list of used tools was
static for each recipe. Now that we want to make tool usage conditional,
we have to pass the tool mask directly into each step.
    @jkloetzke
    jkloetzke committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    9e546cc View commit details
    Browse the repository at this point in the history

  2. input: make tool usage conditional 

    Sometimes a recipe might only require a tool if certain conditions are
met. So far, tools could only be used unconditionally. This adds a new
syntax along the following lines:

  checkoutTools:
      - if: "${TEST_VAR:-}"
        name: graphics::package

Fixes BobBuildTool#572.
    @jkloetzke
    jkloetzke committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    5767a92 View commit details
    Browse the repository at this point in the history

  3. doc: document conditional tool usage

    @jkloetzke
    jkloetzke committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    370b01e View commit details
    Browse the repository at this point in the history

  4. test: add unit test for conditional tools

    @jkloetzke
    jkloetzke committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    6955f6a View commit details
    Browse the repository at this point in the history

  5. input: eliminate some unnecessary env copies

    @jkloetzke
    jkloetzke committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    52f4452 View commit details
    Browse the repository at this point in the history