Skip to content

Conversation

@Salamandar
Copy link

Closes #68

paths-relative-to can be CWD, THIS_FILE, or a literal path. It defaults
to CWD.

The firmware configs will be patched accordingly: binary paths will be
prefixed by either the current working directory (this is the old
behaviour), the directory containing the firmware config file, or the
literal path.

Let me know if you prefer the CWD behaviour to do nothing (like the current behaviour) instead of prefixing with os.getcwd().

Copy link
Collaborator

@rgantois rgantois left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! I've requested a few changes. There's also a linter error, so you might want to run "ruff check" on your sources.

… final fw_configs

fw_configs will always be a dictionnary, but individual ones might not.
configs

paths-relative-to can be CWD, THIS_FILE, or a literal path. It defaults
to CWD.

The firmware configs will be patched accordingly: binary paths will be
prefixed by either the current working directory (this is the old
behaviour), the directory containing the firmware config file, or the
literal path.

Signed-off-by: Félix Piédallu <[email protected]>
This new option of the configuration file indicates that paths to images
are relative to a specific path.

Signed-off-by: Félix Piédallu <[email protected]>
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.

snagrecover firmwares.yml: firmware paths should allow relative paths from firmwares.yml

2 participants