Skip to content

Commit

Permalink
Fix K32W build rules (ToT CI failure) (#23126)
Browse files Browse the repository at this point in the history
* Use Dict type instead of dict for more python compatibility

* Add se05x and no-ble variants to k32w builds, make use of it in CI since otherwise we run out of flash
  • Loading branch information
andy31415 authored Oct 11, 2022
1 parent af09de0 commit f8703c7
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/examples-k32w.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ jobs:
run: |
scripts/run_in_build_env.sh "\
./scripts/build/build_examples.py \
--target k32w-light \
--target k32w-light-no-ble-se05x \
--target k32w-light-no-ota \
--target k32w-lock-low-power-nologs \
--target k32w-contact-low-power-nologs \
Expand Down
4 changes: 2 additions & 2 deletions scripts/build/build/target.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@

from collections.abc import Iterable
from dataclasses import dataclass
from typing import Any, Optional
from typing import Any, Dict, Optional


@dataclass(init=False)
Expand All @@ -54,7 +54,7 @@ class TargetPart:
name: str

# The build arguments to apply to a builder if this part is active
build_arguments: dict[str, Any]
build_arguments: Dict[str, Any]

# Part should be included if and only if the final string MATCHES the
# given regular expression
Expand Down
2 changes: 2 additions & 0 deletions scripts/build/build/targets.py
Original file line number Diff line number Diff line change
Expand Up @@ -350,6 +350,8 @@ def BuildK32WTarget():
TargetPart('contact', app=K32WApp.CONTACT, release=True),
])

target.AppendModifier(name="se05x", se05x=True)
target.AppendModifier(name="no-ble", disable_ble=True)
target.AppendModifier(name="no-ota", disable_ota=True)
target.AppendModifier(name="low-power", low_power=True).OnlyIfRe("-nologs")
target.AppendModifier(name="nologs", disable_logs=True)
Expand Down

0 comments on commit f8703c7

Please sign in to comment.