Skip to content

Revert pykwalify to jsonschema#103661

Closed
bjarki-andreasen wants to merge 2 commits intozephyrproject-rtos:mainfrom
bjarki-andreasen:revert-pykwalify-to-jsonschema
Closed

Revert pykwalify to jsonschema#103661
bjarki-andreasen wants to merge 2 commits intozephyrproject-rtos:mainfrom
bjarki-andreasen:revert-pykwalify-to-jsonschema

Conversation

@bjarki-andreasen
Copy link
Copy Markdown
Contributor

Recent commits moving from pykwalify to jsonschema have resulted in the west tool and zephyr being out of sync, see zephyrproject-rtos/west#904 (comment)

Following the getting started guide now will result in

Traceback (most recent call last):
  File "/home/nordic/Documents/zephyr-2/.venv/lib/python3.11/site-packages/west/commands.py", line 565, in __call__
    mod = _commands_module_from_file(self.py_file)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/nordic/Documents/zephyr-2/.venv/lib/python3.11/site-packages/west/commands.py", line 718, in _commands_module_from_file
    spec.loader.exec_module(mod)
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/nordic/Documents/zephyr-2/zephyr/scripts/west_commands/packages.py", line 20, in <module>
    import zephyr_module
  File "/home/nordic/Documents/zephyr-2/zephyr/scripts/zephyr_module.py", line 27, in <module>
    import jsonschema
ModuleNotFoundError: No module named 'jsonschema'

as west packages pip --install is called

therefore revert the changes until this issue is fixed as it is imperative that the getting started guide works.

…r blobs"

This reverts commit 9fa30fc.

Signed-off-by: Bjarki Arge Andreasen <bjarki.andreasen@nordicsemi.no>
This reverts commit 58bf2dc.

Signed-off-by: Bjarki Arge Andreasen <bjarki.andreasen@nordicsemi.no>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 6, 2026

@carlescufi carlescufi added the Hotfix Fix for issues blocking development, i.e. upstream CI issues, tests failing in upstream CI , etc. label Feb 6, 2026
@pdgendt pdgendt added the DNM This PR should not be merged (Do Not Merge) label Feb 6, 2026
@pdgendt
Copy link
Copy Markdown
Contributor

pdgendt commented Feb 6, 2026

Alternative fix: #103671

@nashif nashif removed the Hotfix Fix for issues blocking development, i.e. upstream CI issues, tests failing in upstream CI , etc. label Feb 6, 2026
@bjarki-andreasen
Copy link
Copy Markdown
Contributor Author

Alternate fix will likely be applied

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Build System DNM This PR should not be merged (Do Not Merge)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants