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.

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

modules: nanopb: Move pip dependencies to optional module #82360

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pdgendt
Copy link
Collaborator

@pdgendt pdgendt commented Nov 30, 2024

Try #81878 again after being reverted by #82351

Upstream has reverted the breaking change on Big endian platforms.

Add a big endian test platform for validating serialization.

Add a big endian test platform for validating serialization.

Signed-off-by: Pieter De Gendt <[email protected]>
The west packages extension can be used to install module dependencies.

Upstream nanopb has added the pip package dependencies to the
zephyr/module.yml file. Remove in-tree pip package dependencies.

Signed-off-by: Pieter De Gendt <[email protected]>
@zephyrbot
Copy link
Collaborator

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
nanopb zephyrproject-rtos/nanopb@98bf4db zephyrproject-rtos/nanopb@1e77a68 (master) zephyrproject-rtos/[email protected]

Additional metadata changed:

Name URL Submodules West cmds module.yml
nanopb

DNM label due to: 1 project with metadata changes

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@zephyrbot zephyrbot added manifest manifest-nanopb DNM This PR should not be merged (Do Not Merge) labels Nov 30, 2024
@pdgendt pdgendt marked this pull request as ready for review November 30, 2024 14:05
@pdgendt pdgendt removed the DNM This PR should not be merged (Do Not Merge) label Nov 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants