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

zephyr: Aligning source directory tree with apache/mynewt-core #8

Merged
merged 1 commit into from
Oct 18, 2019
Merged

zephyr: Aligning source directory tree with apache/mynewt-core #8

merged 1 commit into from
Oct 18, 2019

Conversation

de-nordic
Copy link
Collaborator

Goal of this commit is to make file structure tree, of this fork of
TinyCBOR, to be more aligned to the upstream:
https://github.com/apache/mynewt-core/tree/master/encoding/tinycbor

Signed-off-by: Dominik Ermel dominik.ermel@nordicsemi.no

Goal of this commit is to make file structure tree, of this fork of
TinyCBOR, to be more aligned to the upstream:
https://github.com/apache/mynewt-core/tree/master/encoding/tinycbor

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
@carlescufi
Copy link
Member

cc @vrahane @mlaz @ccollins476ad

Copy link
Collaborator

@mlaz mlaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@nvlsianpu
Copy link
Collaborator

nvlsianpu commented Oct 16, 2019

@de-nordic Can you provide path to the zephyr west.yaml https://docs.zephyrproject.org/latest/guides/modules.html#changes-to-existing-modules and any code adaptation needed?
(zephyr/include/mgmt/buf.h)

@de-nordic
Copy link
Collaborator Author

@nvlsianpu synchronous modification of zephyr source code is not needed for now, as there is redundant path provided that makes old includes work within zephyr (in CMakeLists.txt). Such changes may be completed lately. Redundant path may be removed lately as there is more work needed within TinyCBOR code alignment.

@de-nordic
Copy link
Collaborator Author

@nvlsianpu west.yml update: zephyrproject-rtos/zephyr#19876

@de-nordic de-nordic changed the title zephyr: Aligning source directory tree with apache/mynewt-core [DNM] zephyr: Aligning source directory tree with apache/mynewt-core Oct 17, 2019
Copy link
Collaborator

@nvlsianpu nvlsianpu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good. I also verified that it actually works well as part of smp_svr application (so tinycbor user was mcumgr) and mcuboot.

@nvlsianpu
Copy link
Collaborator

@carlescufi this is ready to be merged

@carlescufi carlescufi changed the title [DNM] zephyr: Aligning source directory tree with apache/mynewt-core zephyr: Aligning source directory tree with apache/mynewt-core Oct 18, 2019
@carlescufi carlescufi merged commit 0fc68fc into zephyrproject-rtos:zephyr Oct 18, 2019
de-nordic added a commit to de-nordic/zephyr that referenced this pull request Oct 21, 2019
Reviewed here: zephyrproject-rtos/tinycbor#8

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
galak pushed a commit to zephyrproject-rtos/zephyr that referenced this pull request Oct 22, 2019
Reviewed here: zephyrproject-rtos/tinycbor#8

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
@de-nordic de-nordic deleted the file-structure-tree-align branch April 3, 2020 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants