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

mcuboot build is broken #31640

Closed
gmarull opened this issue Jan 26, 2021 · 5 comments · Fixed by #31710
Closed

mcuboot build is broken #31640

gmarull opened this issue Jan 26, 2021 · 5 comments · Fixed by #31710
Assignees
Labels
area: MCUBoot bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Milestone

Comments

@gmarull
Copy link
Member

gmarull commented Jan 26, 2021

Describe the bug

mcuboot build is broken with current Zephyr master.

To Reproduce

west build -b nrf52840dk_nrf52840 -d build_bootloader -s ../bootloader/mcuboot/boot/zephyr

Expected behavior

Build succeeds

Impact

mcuboot bootloader can't be built.

Logs and console output

Relevant CMake errors:

error: LOG_MINIMAL (defined at subsys/logging/Kconfig.mode:51) is assigned in a configuration file,
but is not directly user-configurable (has no prompt). It gets its value indirectly from other
symbols. See http://docs.zephyrproject.org/latest/reference/kconfig/CONFIG_LOG_MINIMAL.html and/or
look up LOG_MINIMAL in the menuconfig/guiconfig interface. The Application Development Primer,
Setting Configuration Values, and Kconfig - Tips and Best Practices sections of the manual might be
helpful too.

Environment (please complete the following information):

  • OS: Ubuntu 20.04
  • Toolchain: Zephyr SDK 0.11.4
  • Commit SHA or Version used: 4720585

Additional context

Reverting #31404 fixes the problem

@gmarull gmarull added the bug The issue is a bug, or the PR is fixing a bug label Jan 26, 2021
@henrikbrixandersen
Copy link
Member

Reverting #31404 should not be necessary. The MCUboot Zephyr just needs to be updated to use the new Kconfig options.

@gmarull
Copy link
Member Author

gmarull commented Jan 26, 2021

@henrikbrixandersen I agree, but until then Zephyr master should be in a working state

@gmarull
Copy link
Member Author

gmarull commented Jan 26, 2021

Opened mcu-tools/mcuboot#932

@nandojve
Copy link
Member

Question: How is possible MCUboot be broken at master? Are tests running to avoid this?
This is not an exception, I believe, and I would like to see something addressed to avoid this in Future.

@nvlsianpu
Copy link
Collaborator

Question: How is possible MCUboot be broken at master? Are tests running to avoid this?
This is not an exception, I believe, and I would like to see something addressed to avoid this in Future.

@nandojve #29397
saw that problem, not fixed so fare

nvlsianpu added a commit to nvlsianpu/zephyr that referenced this issue Jan 27, 2021
MCUboot version with bugfix for following issue:
CONFIG_LOG_MINIMAL was replaced by
CONFIG_LOG_MODE_MINIMAL by
zephyrproject-rtos#31404 which
broke MCUboot default build.

fixes zephyrproject-rtos#31640

Signed-off-by: Andrzej Puzdrowski <[email protected]>
@nvlsianpu nvlsianpu added the priority: medium Medium impact/importance bug label Jan 27, 2021
nashif pushed a commit that referenced this issue Jan 27, 2021
MCUboot version with bugfix for following issue:
CONFIG_LOG_MINIMAL was replaced by
CONFIG_LOG_MODE_MINIMAL by
#31404 which
broke MCUboot default build.

fixes #31640

Signed-off-by: Andrzej Puzdrowski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: MCUBoot bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants