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.

Sign up for GitHub

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

Jump to bottom

File no longer at this location #30261

Closed
JeroenAka opened this issue Nov 26, 2020 · 0 comments · Fixed by #30264
Closed

File no longer at this location #30261

JeroenAka opened this issue Nov 26, 2020 · 0 comments · Fixed by #30264
Assignees
@carlescufi
Labels
area: Console area: mcumgr bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@JeroenAka
Copy link

zephyr/drivers/console/uart_console.c

Line 35 in c08ef38

#include "mgmt/serial.h"

The file reference here is no longer at this location but at mgmt/mcumgr/serial.h
@carlescufi carlescufi added the bug The issue is a bug, or the PR is fixing a bug label Nov 26, 2020
@carlescufi carlescufi self-assigned this Nov 26, 2020
carlescufi added a commit to carlescufi/zephyr that referenced this issue Nov 26, 2020
@carlescufi
drivers: console: Fix path to mcumgr header
a5c82ad 
The mcumgr header files are now in include/mgmt/mcumgr/, so reflect this
change in the UART console implementation.

Fixes zephyrproject-rtos#30261.

Signed-off-by: Carles Cufi <[email protected]>
@carlescufi carlescufi mentioned this issue Nov 26, 2020
Merged
@nashif nashif added the priority: low Low impact/importance bug label Nov 27, 2020
carlescufi added a commit that referenced this issue Nov 27, 2020
@carlescufi
drivers: console: Fix path to mcumgr header
ebf2e76 
The mcumgr header files are now in include/mgmt/mcumgr/, so reflect this
change in the UART console implementation.

Fixes #30261.

Signed-off-by: Carles Cufi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@carlescufi carlescufi

Labels
area: Console area: mcumgr bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

drivers: console: Fix path to mcumgr header carlescufi/zephyr
3 participants
@nashif @JeroenAka @carlescufi