asir66 commented on PR #263:
URL:
https://github.com/apache/teaclave-trustzone-sdk/pull/263#issuecomment-3755912453
# Summary of Changes
This refactors the configuration resolution system in cargo-optee and
improves documentation.
## Code Refactoring
1. **Decoupled configuration resolution logic**
- Moved `TaBuildConfig` and `CaBuildConfig` resolution logic to
`config.rs` module
- Separated metadata parsing from priority resolution (CLI > metadata >
default)
- Created `MetadataConfig` struct with `resolve()` method to handle
metadata-only parsing
- Removed redundant `extract_build_config_from_metadata` function
- `MetadataConfig::resolve()` only parses metadata without handling
priorities
2. **Improved code organization**
- Consolidated build configuration resolution in a single module
- Better separation of concerns between metadata parsing and
configuration resolution
## Documentation Improvements
1. **Formatted documentation files**
- Formatted `emulate-and-dev-in-docker-std.md` and
`emulate-and-dev-in-docker.md`
2. **Refactored README.md of cargo-optee**
- Improved Quick Start section for better clarity
- Enhanced Configuration System section with clearer examples
3. **Removed hardcoded configurations**
- Removed hardcoded values from `hello_world-rs` example's `Cargo.toml`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
