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

boot: zephyr: Fix build issue for multiple conf files #795

Merged

Conversation

nvlsianpu
Copy link
Collaborator

Change fixes build issue that occurs if CONF_FILE contains multiple
file paths instead of single file path.

Signed-off-by: Marek Pieta Marek.Pieta@nordicsemi.no
Signed-off-by: Andrzej Puzdrowski andrzej.puzdrowski@nordicsemi.no

Change fixes build issue that occurs if CONF_FILE contains multiple
file paths instead of single file path.

Signed-off-by: Marek Pieta <Marek.Pieta@nordicsemi.no>
Signed-off-by: Andrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no>
@nvlsianpu nvlsianpu requested a review from utzig September 10, 2020 09:56
@nvlsianpu
Copy link
Collaborator Author

cc @MarekPieta

@utzig
Copy link
Member

utzig commented Sep 10, 2020

This fixes my local issues building with west, but I wonder why the Zephyr buildkite runs have not failed, any idea?

@utzig
Copy link
Member

utzig commented Sep 10, 2020

This fixes my local issues building with west, but I wonder why the Zephyr buildkite runs have not failed, any idea?

At a glance it doesn't look like any of the Zephyr tests build MCUboot itself, they only use some of its provided API, so there's not that much point in submitting a build PR there.

@nvlsianpu
Copy link
Collaborator Author

User discovered this on disco_l475_iot1 which is not used in zephyr CI.

@nvlsianpu nvlsianpu merged commit c1cdcae into mcu-tools:master Sep 10, 2020
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.

3 participants