Skip to content

Nanopb update version nanopb-0.4.9.1#77

Open
jakubretajczykBD wants to merge 1 commit intoblockydevs:developfrom
jakubretajczykBD:fix/nanopb-update
Open

Nanopb update version nanopb-0.4.9.1#77
jakubretajczykBD wants to merge 1 commit intoblockydevs:developfrom
jakubretajczykBD:fix/nanopb-update

Conversation

@jakubretajczykBD
Copy link
Copy Markdown

@jakubretajczykBD jakubretajczykBD commented Jan 30, 2026

Checklist

  • App update process has been followed
  • Target branch is develop
  • Application version has been bumped
  • Makefile: The Makefile was updated because in the new nanopb version generator/proto/init.py no longer builds nanopb_pb2.py on import, so that file was missing in vendor/nanopb/generator/proto and tests failed. It now explicitly builds nanopb_pb2.py there before generating the app’s Python protobufs when running make python_pb.
  • PIC: Files from the new nanopb version were changed to use Position Independent Code where required for the current target/platform.

Comment thread vendor/nanopb/pb.h

/* This can be defined if the platform is little-endian and has 8-bit bytes.
* Normally it is automatically detected based on __BYTE_ORDER__ macro. */
/* #define PB_LITTLE_ENDIAN_8BIT 1 */

Check notice

Code scanning / CodeQL

Commented-out code Note library

This comment appears to contain commented-out code.
Comment thread vendor/nanopb/pb.h

/* Configure static assert mechanism. Instead of changing these, set your
* compiler to C11 standard mode if possible. */
/* #define PB_C99_STATIC_ASSERT 1 */

Check notice

Code scanning / CodeQL

Commented-out code Note library

This comment appears to contain commented-out code.
Comment thread vendor/nanopb/pb.h
/* Configure static assert mechanism. Instead of changing these, set your
* compiler to C11 standard mode if possible. */
/* #define PB_C99_STATIC_ASSERT 1 */
/* #define PB_NO_STATIC_ASSERT 1 */

Check notice

Code scanning / CodeQL

Commented-out code Note library

This comment appears to contain commented-out code.
@jakubretajczykBD jakubretajczykBD marked this pull request as ready for review February 18, 2026 09:36
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