Skip to content

Add --path option to bsp build to override output directory#2

Open
Copilot wants to merge 5 commits intodevelopmentfrom
copilot/extend-bsp-build-command-path
Open

Add --path option to bsp build to override output directory#2
Copilot wants to merge 5 commits intodevelopmentfrom
copilot/extend-bsp-build-command-path

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 25, 2026

The bsp build command had no way to override the build output path defined in the registry; users were forced to modify registry configuration to change where build artifacts land.

Changes

  • CLI (cli.py): Added --path PATH argument to the build subcommand; value is passed as build_path_override to the manager
  • Manager (bsp_manager.py): Added build_path_override: Optional[str] to build_bsp(), build_by_components(), and _build_resolved(); when provided, it replaces resolved.build_path before directory preparation and KAS invocation
  • Tests: Added coverage for path override in both build entry points and CLI dispatch

Usage

# Named preset with custom output path
bsp build poky-qemux86-64-scarthgap --path /mnt/fast-ssd/build

# Component-based build with custom output path
bsp build --device qemux86-64 --release scarthgap --path /mnt/fast-ssd/build

@miketsukerman
Copy link
Copy Markdown
Owner

@copilot update readme

Copy link
Copy Markdown
Author

Copilot AI commented Mar 25, 2026

@copilot update readme

Done in 0d4d15a. Updated the build command section in the README to document --path along with --device, --release, and --feature options that were previously undocumented.

@miketsukerman
Copy link
Copy Markdown
Owner

@copilot rebase onto development branch

Co-authored-by: miketsukerman <145971+miketsukerman@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented Apr 13, 2026

@copilot rebase onto development branch

Done in a8c98d3. Merged origin/development into this branch and resolved all conflicts between the --path and --deploy features in cli.py, bsp_manager.py, and README.md.

@miketsukerman miketsukerman marked this pull request as ready for review April 13, 2026 10:23
@miketsukerman
Copy link
Copy Markdown
Owner

@copilot resolve the merge conflicts in this pull request

Co-authored-by: miketsukerman <145971+miketsukerman@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented Apr 17, 2026

@copilot resolve the merge conflicts in this pull request

Done in 2b7fba6. Merged latest origin/development (LAVA/gather features) and resolved the only README conflict by combining the --path usage line with the --test usage lines from development.

@miketsukerman
Copy link
Copy Markdown
Owner

@copilot resolve the merge conflicts in this pull request

… options

Co-authored-by: miketsukerman <145971+miketsukerman@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented Apr 17, 2026

@copilot resolve the merge conflicts in this pull request

Done in 7a5beea. Merged the latest origin/development commits that added --target/--task parameters. Resolved conflicts in all 5 affected files (bsp_manager.py, cli.py, README.md, test_bsp_manager.py, test_cli_basic.py) by combining both the --path and --target/--task features. All 279 tests pass.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants