-
Notifications
You must be signed in to change notification settings - Fork 77
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
Add Zephyr smp_svr
build testing to Travis-CI
#63
Conversation
40d3a25
to
187f443
Compare
0221ec0
to
ec76a2e
Compare
smp_svr
build testing to Travis-CI
This takes an approach as suggested by #60 (comment). Instead of having multiple copies that need to be constantly synchronized (Possible losing commit history), allow the sample to live outside the tree and check that it is in a build state here. Would be easy to add Mynewt build testing as well, so maybe, There are no tests running as of now, only building the app! |
Hi @utzig we haven’t closed the loop yet on whether we will keep the smp_svr in mcumgr upstream for zephyr as is or just keep the zephyr version of it. So, I would suggest adding builds for both the zephyr version as well as the upstream version for now atleast until we come to a consensus. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am very skeptical of this. It is not going to be possible to maintain master:master compatibility between this repo and zephyrproject-rtos/zephyr, there is gonna be failures.
export ZEPHYR_SDK_INSTALL_DIR="${HOME}/zephyr-sdk" | ||
|
||
repos=( | ||
"https://github.com/zephyrproject-rtos/zephyr" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you are going to use Zephyr, you really should use west
to pull down the repos. How else are you going to be sure that you are pulling the right set?
This builds
smp_svr
directly from the Zephyr tree; the purpose is to assure that the provided sample is still in a good build state when the mcumgr code changes.