-
Notifications
You must be signed in to change notification settings - Fork 344
添加mcoredump软件包索引 #1920
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
添加mcoredump软件包索引 #1920
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds a new package index for the MCoreDump software component, which is a fault dump utility specifically designed for embedded ARM Cortex-M systems. The package automatically generates ELF format core dump files for offline debugging and fault analysis.
- Adds package configuration files for MCoreDump
- Integrates MCoreDump into the existing tools package system
- Provides comprehensive configuration options for memory management and filesystem support
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| tools/MCoreDump/package.json | Defines package metadata, repository information, and version details |
| tools/MCoreDump/Kconfig | Provides configuration options for architecture selection, memory buffer sizing, and filesystem integration |
| tools/Kconfig | Integrates the MCoreDump package into the main tools menu |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
|
||
| config PKG_MCOREDUMP_VER_NUM | ||
| hex | ||
| default 0x99999 if PKG_USING_MCOREDUMP_LATEST_VERSION |
Copilot
AI
Aug 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The condition references 'PKG_USING_MCOREDUMP_LATEST_VERSION' but the actual config name defined above is 'PKG_USING_MCORE_DUMP_LATEST_VERSION'. This will cause the default value to never be applied.
| default 0x99999 if PKG_USING_MCOREDUMP_LATEST_VERSION | |
| default "latest" if PKG_USING_MCORE_DUMP_LATEST_VERSION | |
| config PKG_MCOREDUMP_VER_NUM | |
| hex | |
| default 0x99999 if PKG_USING_MCORE_DUMP_LATEST_VERSION |
|
|
||
| config PKG_MCOREDUMP_VER_NUM | ||
| hex | ||
| default 0x99999 if PKG_USING_MCOREDUMP_LATEST_VERSION |
Copilot
AI
Aug 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The condition references 'PKG_USING_MCOREDUMP_LATEST_VERSION' but the actual config name defined above is 'PKG_USING_MCORE_DUMP_LATEST_VERSION'. This will cause the default value to never be applied.
| default 0x99999 if PKG_USING_MCOREDUMP_LATEST_VERSION | |
| default "latest" if PKG_USING_MCORE_DUMP_LATEST_VERSION | |
| config PKG_MCOREDUMP_VER_NUM | |
| hex | |
| default 0x99999 if PKG_USING_MCORE_DUMP_LATEST_VERSION |
No description provided.