Skip to content

Bump nanopb submodule to 0.4.9.2 - #218

Merged
tyeth merged 1 commit into
masterfrom
nanopb-0.4.9.2
Sep 30, 2026
Merged

tyeth merged 1 commit into
masterfrom
nanopb-0.4.9.2

Conversation

@tyeth

@tyeth tyeth commented Sep 16, 2026 •

Copy link
Copy Markdown
Member

Bumps the nanopb submodule from a 2021 0.4.5-dev commit to the nanopb-0.4.9.2 release and drops setuptools from the Build workflow.

Why: master CI is red with ModuleNotFoundError: No module named 'pkg_resources' (e.g. https://github.com/adafruit/Wippersnapper_Protobuf/actions/runs/35118154038). The old generator imports pkg_resources unconditionally; setuptools 81+ removed it. The 0.4.9.2 generator uses importlib.resources, so no setuptools is needed.

Generated Arduino output: regenerated locally with 0.4.9.2 and compared against the current wrappers in https://github.com/adafruit/Adafruit_Wippersnapper_Arduino main. Tags, sizes, init macros and field lists are identical; the only additions are the *_ENUMTYPE convenience macros. Cosmetic differences: no timestamp line, shorter include guards, proto comments emitted, .pb.c includes its header by bare name.

Companion Arduino PR: adafruit/Adafruit_Wippersnapper_Arduino#993 (for adafruit/Adafruit_Wippersnapper_Arduino#920).

api-v2 equivalent: #219

Fixes the pkg_resources ImportError in the Build and wrapper-generation workflows.
Ref adafruit/Adafruit_Wippersnapper_Arduino#920

@tyeth tyeth left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tyeth
tyeth requested a review from brentru September 16, 2026 16:52

@brentru brentru left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tyeth LGTM

@tyeth

tyeth commented Sep 30, 2026

Copy link
Copy Markdown
Member Author

Tested for v1 against prod in
adafruit/Adafruit_Wippersnapper_Arduino#993 (comment)

@tyeth
tyeth merged commit a4683ef into master Sep 30, 2026
3 checks passed
tyeth added a commit to tyeth/adafruit-Adafruit_Wippersnapper_Arduino that referenced this pull request Oct 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants