Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge lists from config file with lists provided in CLI options #32

Open
Aposhian opened this issue Dec 14, 2021 · 1 comment
Open

Merge lists from config file with lists provided in CLI options #32

Aposhian opened this issue Dec 14, 2021 · 1 comment

Comments

@Aposhian
Copy link

Currently, I specify options to a list like:

{
    "build": {
        "mixin": [ "ccache", "rel-with-deb-info" ]
    }
}

then it works just fine, but if I add an additional option to the list via the CLI like:

colcon build --mixin asan-gcc

then the whole list is ignored from my config file, which is rather inconvenient: you must use either all of the defaults and no more, or none of the defaults.

If the lists from the config files were simply merged with options provided on the CLI, then my use case would be easier, but the only way to disable the defaults would be to use --ignore-user-meta, which will apply to all of your defaults, not just one list.

@christophebedard
Copy link

Some context on why merging options is not really straightforward can be found here: colcon/colcon-mixin#18 (comment).

It might be possible to improve this to do the merging carefully on a case-by-case (option by option) basis, see colcon/colcon-mixin#18 (comment).

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

No branches or pull requests

2 participants