Skip to content
This repository has been archived by the owner on Sep 21, 2024. It is now read-only.

feat: Add SphereFS read/write to FFI #141

Merged
merged 9 commits into from
Nov 14, 2022
Merged

Conversation

cdata
Copy link
Collaborator

@cdata cdata commented Nov 13, 2022

This change introduces reading and writing-related FFI functions and structs to enable basic access to local sphere data. A test has been implemented to demonstrate the FFI in action on the Swift side.

This change also adds automated Swift testing to our Github Actions workflow. A specialization of the Apple library build that only targets MacOS x86_64 has been created so that we can run Swift tests on a library created on-demand for the current branch without wasting runner minutes. An extension to the release flow has been included that does everything except commit the work needed for #135 .

New brand images have also been added to the README as of this change.

BREAKING CHANGE: Some FFI interfaces now have simplified interfaces.

Fixes #133

Copy link
Contributor

@jsantell jsantell left a comment

Choose a reason for hiding this comment

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

Amazing work @cdata! Some points for discussion on what do we want our FFI guidelines to be w/r/t naming, returned allocated primitives, error handling, and documenting fallibility and ownership

rust/noosphere/src/ffi/fs.rs Show resolved Hide resolved
rust/noosphere/src/ffi/fs.rs Show resolved Hide resolved
rust/noosphere/src/ffi/fs.rs Show resolved Hide resolved
rust/noosphere/src/ffi/fs.rs Show resolved Hide resolved
rust/noosphere/src/ffi/noosphere.rs Show resolved Hide resolved
design/explainer.md Show resolved Hide resolved
design/explainer.md Show resolved Hide resolved
design/images/noosphere-dark.svg Show resolved Hide resolved
rust/noosphere/src/ffi/fs.rs Show resolved Hide resolved
rust/noosphere/src/ffi/fs.rs Show resolved Hide resolved
This change also adds automated Swift testing to our Github Actions workflow.

BREAKING CHANGE: Some FFI interfaces now have simplified interfaces.
The new test-specific build will cut down Github Actions runner usage
time by ~60% for Apple-related work.
@cdata cdata force-pushed the feature/ffi-read-write-sphere branch from b5faa63 to 0e9e0d9 Compare November 14, 2022 21:48
@cdata cdata requested a review from jsantell November 14, 2022 21:50
@cdata cdata merged commit 26e34ac into main Nov 14, 2022
@cdata cdata deleted the feature/ffi-read-write-sphere branch November 14, 2022 22:12
@github-actions github-actions bot mentioned this pull request Jul 1, 2023
@github-actions github-actions bot mentioned this pull request Jul 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FFI should support reading and writing content
3 participants