Skip to content

Conversation

@m-gorecki
Copy link
Contributor

This command prints full paths to target binaries in form of environmental variables definitions like this:
ELF_PATH=<path_to_elf>
IMG_PATH=<path_to_img>
HEX_PATH=<path_to_hex>

@andrzej-kaczmarek
Copy link
Contributor

andrzej-kaczmarek commented Dec 2, 2025

I'd rename existing and add few more smth like this:

APP_ELF_PATH
APP_HEX_PATH
APP_BIN_PATH
APP_IMG_PATH
BUILD_TARGET_PATH (or BIN_TARGET_PATH?) i.e. bin/targets/blah
BUILD_APP_PATH (or BIN_APP_PATH?) i.e. bin/targets/blah/app/foo/bar/ (where .elf and other files are stored)

@sjanc
Copy link
Contributor

sjanc commented Dec 2, 2025

I'd use BUILD_* not BIN_* for now

Copy link
Contributor

@kasjer kasjer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This command prints full paths to target binaries in form of environmental
variables definitions like this:
APP_ELF_PATH=<path_to_elf>
APP_IMG_PATH=<path_to_img>
APP_HEX_PATH=<path_to_hex>
APP_BIN_PATH=<path_to_bin>
BUILD_TARGET_DIR=<path_to_bin_target_dir>
BUILD_APP_DIR=<path_to_dir_containing_binaries>
@m-gorecki m-gorecki merged commit a62a1c1 into apache:master Dec 3, 2025
70 checks passed
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.

4 participants