Skip to content

Commit 7d3aa19

Browse files
committed
Revert "modules: nanopb: Move pip dependencies to optional module"
This reverts commit 2661483. A bug in upstream nanopb, which this commit was pulling as part of the submanifest update, seems to be causing test failures on big-endian platforms. Upstream bug: nanopb/nanopb#1039 Signed-off-by: Benjamin Cabé <[email protected]>
1 parent 6c7b38f commit 7d3aa19

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

doc/services/serialization/nanopb.rst

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@ Additionally, Nanopb is an optional module and needs to be added explicitly to t
4545
4646
west config manifest.project-filter -- +nanopb
4747
west update
48-
west packages pip --install
4948
5049
Configuration
5150
*************

scripts/requirements-extras.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,10 @@ Pillow>=10.0
2121
# can be used to sign a Zephyr application binary for consumption by a bootloader
2222
imgtool>=2.1.0
2323

24+
# used by nanopb module to generate sources from .proto files
25+
grpcio-tools>=1.47.0
26+
protobuf>=3.20.3
27+
2428
# used by scripts/release/bug_bash.py for generating top ten bug squashers
2529
PyGithub
2630

submanifests/optional.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ manifest:
2323
groups:
2424
- optional
2525
- name: nanopb
26-
revision: 0a3b5050a28faeca3cfa2f566e043807431c8d7f
26+
revision: 98bf4db69897b53434f3d0ba72e0a3ab1a902824
2727
path: modules/lib/nanopb
2828
remote: upstream
2929
groups:

0 commit comments

Comments
 (0)