Skip to content
This repository was archived by the owner on Mar 16, 2026. It is now read-only.

config.py: do variable expansion for qemu_base and qemu_extra options#19

Merged
il-steffen merged 1 commit intoIntelLabs:masterfrom
il-steffen:feat/expand_qemu_args
Oct 14, 2022
Merged

config.py: do variable expansion for qemu_base and qemu_extra options#19
il-steffen merged 1 commit intoIntelLabs:masterfrom
il-steffen:feat/expand_qemu_args

Conversation

@il-steffen
Copy link
Copy Markdown
Contributor

We previously called variable expansion only for arguments that are parsed as files (parse_is_file()). This patch generalizes FullPath() to ExpandVars() and also expands any corresponding config entry.

Also removes the abspath() from argparse parsing, so that cmdline options and config options are treated more similar.

…tions

We previously called variable expansion only for arguments that are
parsed as files (parse_is_file()). This patch generalizes FullPath() to
ExpandVars() and also expands any corresponding config entry.

Also removes the abspath() from argparse parsing, so that
cmdline options and config options are treated more similar.
@il-steffen il-steffen requested a review from Wenzel October 13, 2022 11:46
@il-steffen il-steffen merged commit f2a76c6 into IntelLabs:master Oct 14, 2022
@il-steffen il-steffen deleted the feat/expand_qemu_args branch October 30, 2022 00:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant