Skip to content

Commit

Permalink
Support building chip-tool with python3.12 (#33242) (#33770)
Browse files Browse the repository at this point in the history
This updates 2 python packages to versions compatible with python3.12,
fixing the following errors when running
`./scripts/examples/gn_build_example.sh examples/chip-tool BUILDFOLDER`:

1. `pyyaml` 6.0 build error: `AttributeError: cython_sources`, fixed
   [here](yaml/pyyaml#702)
2. `construct` import error:

   ```
           File "/tmp/pip-install-ifgk4tul/construct_6d60304f85e249759f6c184ea89f1317/construct/core.py", line 3, in <module>
          import struct, io, binascii, itertools, collections, pickle, sys, os, tempfile, hashlib, importlib, imp
      ModuleNotFoundError: No module named 'imp'
   ```

   this was fixed
   [here](construct/construct@91cc0c6),
   in version `v2.10.57` of `construct`. I'm just
   updating to latest though.

Now running `./scripts/examples/gn_build_example.sh examples/chip-tool
BUILDFOLDER` succeeds when the host python is python3.12.

Co-authored-by: Noah Pendleton <[email protected]>
  • Loading branch information
raju-apple and noahp committed Jun 6, 2024
1 parent ba510d4 commit 7dbf446
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
5 changes: 2 additions & 3 deletions scripts/setup/constraints.txt
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ colorama==0.4.6
# west
coloredlogs==15.0.1
# via -r requirements.all.txt
construct==2.10.54
construct==2.10.70
# via
# -r requirements.esp32.txt
# esp-coredump
Expand Down Expand Up @@ -208,7 +208,7 @@ python-socketio==4.6.1
# via -r requirements.esp32.txt
pytz==2022.7.1
# via pandas
pyyaml==6.0
pyyaml==6.0.1
# via
# esptool
# idf-component-manager
Expand Down Expand Up @@ -296,4 +296,3 @@ setuptools==68.0.0
# Higher versions depend on proto-plus, which break
# nanopb code generation (due to name conflict of the 'proto' module)
google-api-core==2.17.0

2 changes: 1 addition & 1 deletion scripts/setup/requirements.esp32.txt
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ pygdbmi<=0.9.0.2
reedsolo>=1.5.3,<=1.5.4
bitstring>=3.1.6,<4
ecdsa>=0.16.0
construct==2.10.54
construct>=2.10.70
python-socketio<5
itsdangerous<2.1 ; python_version < "3.11"
esp_idf_monitor==1.1.1
Expand Down

0 comments on commit 7dbf446

Please sign in to comment.