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

Update DM XML generation script to use new path format #37148

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

cecille
Copy link
Contributor

@cecille cecille commented Jan 21, 2025

The path format for spec parsing was changed earlier. This script is only run on-demand to generate new DM XMLs for release versions of the spec, so wasn't updated earlier.

Testing

Manual testing - the script doesn't run as part of the normal operation since it relies on the spec repo, which is not accessible to the SDK. Script was used to generate the 1.4 DM update here: #37146

Copy link

semanticdiff-com bot commented Jan 21, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  scripts/spec_xml/generate_spec_xml.py  17% smaller

@mergify mergify bot removed the conflict label Jan 21, 2025
Copy link

PR #37148: Size comparison from 2fed176 to 1df0f61

Full report (3 builds for cc32xx, stm32)
platform target config section 2fed176 1df0f61b change % change
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 538433 538433 0 0.0
RAM 205192 205192 0 0.0
lock CC3235SF_LAUNCHXL FLASH 572297 572297 0 0.0
RAM 205344 205344 0 0.0
stm32 light STM32WB5MM-DK FLASH 482608 482608 0 0.0
RAM 144672 144672 0 0.0

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