-
Notifications
You must be signed in to change notification settings - Fork 374
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
tinydtls: Update to most recent version, simplify usage #657
tinydtls: Update to most recent version, simplify usage #657
Conversation
9c2d286
to
8b70829
Compare
8b70829
to
bc204ab
Compare
Do we need a new CQ for this, the existing one is for version 0.8.2 |
You are talking about tinydtls dependency right ? As now tinydtls is an eclipse project, AFAIK no CQ is needed. You could ask why does it exist a CQ for the version 0.8.2 ? |
Just to mention: |
@boaks do you know the idea behind |
I don't know. So asking @obgm is the best. |
The reason for these branches is that it follows the usual best practices for git workflows with a stable main branch, and a develop branch where the next stable version is being prepared. |
Thanks to @boaks I think we are getting closer to this initial release within the next weeks. |
Would you (tinydtls) prefer Wakaama to update now (and again in a few weeks) or wait for the final release? |
+1 for the final release |
That depends on what you want. I usually try the "newest", but with care, mainly in order to find left or new bugs. |
I suppose updating now is a good idea, especially as the release process will take some time. We could provide a new release candidate soon if you prefer to have some synchronization point. |
@obgm, I think I get it, you're master branch is a kind of maintenance branch. Currently tinydtls is just used for example / demo, right ? E.g. Leshan (LWM2M in java) development branch ( |
Yes, its works-with. side note: I intend to include the DTLS enabled client binary in the integration testing, but even then, tinydtls will not be a dependency of the Wakaama library itself. |
While (again) not an official release, this will advance us by 274 commits: ``` $ git rev-list --count 178b945e5497b813b1239e2fbf57e34b7488745f..HEAD 274 ```
bc204ab
to
7c0275c
Compare
Just updated this PR on the most recent Wakaama master and with the most recent commit of tindydtls (there is no release yet). Suggesting to merge this PR in this current state. |
I consider, that when the PR for RFC5746 is merged into tinydtls/main, a update would make again sense. |
Just realized the thumbs up from @boaks. I consider this an approval and therefore merge this PR. 👯 |
While (again) not an official release, this will advance us by 274 commits:
Smoke tested like this (after registering here and here):