Skip to content

Commit

Permalink
base: Add nrf-regtool to the list of Python packages installed
Browse files Browse the repository at this point in the history
After a discussion in Discord
(https://discord.com/channels/720317445772017664/1196766845332619305/1196766847962464276)
we concluded that there is no easy way of adding vendor-specific Python
packages that are required to be executed during the build but should
not be present in the standard requirements-*.txt files since they are
specific to a single vendor.

For now solve the issue by including the package by name in the Docker
file, to have the package available for the PR that introduced the first
IC that requires this package, the nRF54H20 (zephyrproject-rtos/zephyr#68043).

Signed-off-by: Carles Cufi <[email protected]>
  • Loading branch information
carlescufi authored and stephanosio committed Jan 29, 2024
1 parent 73f57c8 commit 23e5da1
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion Dockerfile.base
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,8 @@ RUN python3 -m pip install -U --no-cache-dir pip && \
-r https://raw.githubusercontent.com/zephyrproject-rtos/zephyr/main/scripts/requirements.txt \
-r https://raw.githubusercontent.com/zephyrproject-rtos/mcuboot/main/scripts/requirements.txt \
GitPython imgtool junitparser numpy protobuf PyGithub \
pylint sh statistics west && \
pylint sh statistics west \
nrf-regtool && \
pip3 check

# Clean up stale packages
Expand Down

0 comments on commit 23e5da1

Please sign in to comment.