Merge pull request #59 from matrix-org/kegan/build-rust-before-run

Remove compiled rust bindings

Author: kegsay

.github/workflows/tests.yaml

@@ -69,10 +69,7 @@ jobs:
 
       - name: Build Rust FFI bindings
         run: |
-          sed -i.bak 's/uniffi =.*/uniffi = "0\.25\.3"/' rust-sdk/Cargo.toml
-          sed -i.bak 's^uniffi_bindgen =.*^uniffi_bindgen = { git = "https:\/\/github.com\/mozilla\/uniffi-rs", rev = "0a03b713306d6ce3de033157fc2ce92a238c2e24" }^' rust-sdk/Cargo.toml
-          (cd rust-sdk && cargo build -p matrix-sdk-ffi)
-          ls -alh rust-sdk/target/debug
+          ./install_uniffi_bindgen_go.sh && ./rebuild_rust_sdk.sh ./rust-sdk
 
       # Temporary: as it takes 3m to build the complement synapse image >:(
       - name: Pull synapse service v1.94.0 and mitmproxy
@@ -119,7 +116,6 @@ jobs:
         run: |
           export LIBRARY_PATH="$(pwd)/rust-sdk/target/debug"
           export LD_LIBRARY_PATH="$(pwd)/rust-sdk/target/debug"
-          env
           go test $(go list ./internal/... | grep -v 'internal/tests') -timeout 60s
 
       - name: "Building RPC client"

README.md

@@ -14,6 +14,8 @@ Complement Crypto can be compiled and run in different modes depending on which
 to test JS SDK then you do not need to compile rust code or run rust tests, and vice versa. Conversely, if you want to test
 interoperability between the two SDKs then you need to compile both SDKs.
 
+This means you MUST compile at least 1 SDK in order for Complement-Crypto to compile and run.
+
 #### JS SDK
 
 Run `./rebuild_js_sdk.sh` according to its help page:
@@ -23,9 +25,10 @@ Usage: ./rebuild_js_sdk.sh [version]
   [version]: the yarn/npm package to use. This is fed directly into 'yarn add' so branches/commits can be used
 
 Examples:
-  Install a released version: ./rebuild_js_sdk.sh [email protected]
-  Install develop branch:     ./rebuild_js_sdk.sh matrix-js-sdk@https://github.com/matrix-org/matrix-js-sdk#develop
-  Install specific commit:    ./rebuild_js_sdk.sh matrix-js-sdk@https://github.com/matrix-org/matrix-js-sdk#36c958642cda08d32bc19c2303ebdfca470d03c1
+  Install a released version:    ./rebuild_js_sdk.sh [email protected]
+  Install develop branch:        ./rebuild_js_sdk.sh matrix-js-sdk@https://github.com/matrix-org/matrix-js-sdk#develop
+  Install specific commit:       ./rebuild_js_sdk.sh matrix-js-sdk@https://github.com/matrix-org/matrix-js-sdk#36c958642cda08d32bc19c2303ebdfca470d03c1
+  Install from a local checkout: ./rebuild_js_sdk.sh matrix-js-sdk@file:/path/to/local/js/sdk"
 ```
 
 #### Rust SDK