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

bump tinydtls to 9d6cf54, update bindgen, bump crate version to 0.2.0 #3

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

pulsastrix
Copy link
Member

bindgen 0.59.X depended on a crate with a cargo audit warning, updating it to fix that.

Also made some minor changes to build script and tests to accomodate for bindgen and tinydtls changes.

Crate version has to be updated to 0.2.0 because tinydtls added an additional field to dtls_handler_t, which is a breaking change (unsure if there are more breaking changes, but that one was immediately noticable because the tests wouldn't compile anymore).

bindgen 0.59.X depended on a crate with a cargo audit warning, updating
it to fix that.

Also made some minor changes to build script and tests to accomodate for
bindgen and tinydtls changes.
@pulsastrix pulsastrix self-assigned this Jun 11, 2024
@pulsastrix pulsastrix requested a review from JKRhb June 11, 2024 15:41
Copy link
Member

@JKRhb JKRhb left a comment

Choose a reason for hiding this comment

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

Besides the question above: LGTM :)

Quick sidenote: I guess at some point, the GitLab CI pipeline could be ported to GitHub Actions, right?

@pulsastrix
Copy link
Member Author

Quick sidenote: I guess at some point, the GitLab CI pipeline could be ported to GitHub Actions, right?

Yes, that would probably be a good idea.
Copying over some of the stuff in libcoap-rs and making some small adjustments should probably suffice.

@pulsastrix pulsastrix merged commit 9cea60c into main Jun 11, 2024
@pulsastrix pulsastrix deleted the bump-tinydtls-9d6cf54 branch June 11, 2024 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants