Skip to content

Conversation

@holtrop
Copy link

@holtrop holtrop commented Sep 12, 2025

Description

Create initial Rust wrapper structure

Generate bindings to C library with bindgen
Add github CI workflow to build Rust wrapper

Testing

Tested that new github workflow executes to build Rust wrapper.

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@holtrop holtrop self-assigned this Sep 12, 2025
@holtrop holtrop force-pushed the rust-wrapper branch 2 times, most recently from 90c9b1f to e42f467 Compare September 12, 2025 18:58
Generate bindings to C library with bindgen
Add github CI workflow to build Rust wrapper
@holtrop
Copy link
Author

holtrop commented Sep 12, 2025

retest this please (hudson.remoting.RequestAbortedException)

@holtrop holtrop marked this pull request as ready for review September 12, 2025 21:05
@holtrop holtrop requested review from wolfSSL-Bot and removed request for wolfSSL-Bot September 12, 2025 21:06
@holtrop holtrop assigned wolfSSL-Bot and unassigned holtrop Sep 12, 2025
Copy link
Contributor

@JacobBarthelmeh JacobBarthelmeh left a comment

Choose a reason for hiding this comment

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

The wrapper needs added to include.am files. To add the files to a bundle created with make dist

@holtrop
Copy link
Author

holtrop commented Sep 17, 2025

The wrapper needs added to include.am files. To add the files to a bundle created with make dist

Thanks. I always forget that!

I've added it now.

I wonder if we could have make dist pull from git ls-files as a base rather than needing to specify all these paths manually?

@holtrop
Copy link
Author

holtrop commented Sep 17, 2025

retest this please (AgentOfflineException)

@JacobBarthelmeh JacobBarthelmeh merged commit 05bcd82 into wolfSSL:master Sep 17, 2025
325 of 326 checks passed
@holtrop holtrop deleted the rust-wrapper branch September 18, 2025 16:46
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.

4 participants