mathstuf / rust-keyutils

Rust interface to the Linux keyring
BSD 3-Clause "New" or "Revised" License
17 stars 9 forks source link

Cleanup keyutils types and api.rs #29

Closed josephlr closed 5 years ago

josephlr commented 5 years ago

This patchset makes many changes in preparation for fixing issue #24 (to no longer need libkeyutils). Doing everything in one PR would be very large, so this PR is just all of the internal cleanup that doesn't change the external API of keyutils.

Specifically:

Again, note that we continue linking libkeyutils, and this only ends up with a very tiny change to the external API:

mathstuf commented 5 years ago

Thanks for all the work this past week! Once this is rebased, I'll take a closer look at it now that the other PRs have been merged.

josephlr commented 5 years ago

Thanks for all the work this past week! Once this is rebased, I'll take a closer look at it now that the other PRs have been merged.

Done, and let me know if you want to split this up. I've also revised the PR's description to be more accurate.

mathstuf commented 5 years ago

Thanks!