diff options
author | Florian Klink <flokli@flokli.de> | 2023-12-16T22·16+0200 |
---|---|---|
committer | flokli <flokli@flokli.de> | 2023-12-22T16·55+0000 |
commit | a5167c508cf2ed92f8a39696a6b4376cf25ee872 (patch) | |
tree | 5ffb2f8d0d331b6fea1aeb4f6391e0408df6d234 /tvix/store/Cargo.toml | |
parent | 52cad8619511b97c4bcd5768ce9b3579ff665505 (diff) |
chore(tvix): move store/fs to castore/fs r/7256
With the recent introduction of the RootNodes trait, there's nothing in the fs module pulling in tvix-store dependencies, so it can live in tvix-castore. This allows other crates to make use of TvixStoreFS, without having to pull in tvix-store. For example, a tvix-build using a fuse mountpoint at /nix/store doesn't need a PathInfoService to hold the root nodes that should be present, but just a list. tvix-store now has a pathinfoservice/fs module, which contains the necessary glue logic to implement the RootNodes trait for a PathInfoService. To satisfy Rust orphan rules for trait implementations, we had to add a small wrapper struct. It's mostly hidden away by the make_fs helper function returning a TvixStoreFs. It can't be entirely private, as its still leaking into the concrete type of TvixStoreFS. tvix-store still has `fuse` and `virtiofs` features, but they now simply enable these features in the `tvix-castore` crate they depend on. The tests for the fuse functionality stay in tvix-store for now, as they populate the root nodes through a PathInfoService. Once above mentioned "list of root nodes" implementation exists, we might want to shuffle this around one more time. Fixes b/341. Change-Id: I989f664827a5a361b23b34368d242d10c157c756 Reviewed-on: https://cl.tvl.fyi/c/depot/+/10378 Autosubmit: flokli <flokli@flokli.de> Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
Diffstat (limited to 'tvix/store/Cargo.toml')
-rw-r--r-- | tvix/store/Cargo.toml | 51 |
1 files changed, 5 insertions, 46 deletions
diff --git a/tvix/store/Cargo.toml b/tvix/store/Cargo.toml index 044b305b96ae..0a8e690e8250 100644 --- a/tvix/store/Cargo.toml +++ b/tvix/store/Cargo.toml @@ -14,7 +14,6 @@ data-encoding = "2.3.3" futures = "0.3.28" lazy_static = "1.4.0" nix-compat = { path = "../nix-compat", features = ["async"] } -parking_lot = "0.12.1" pin-project-lite = "0.2.13" prost = "0.12.1" sha2 = "0.10.6" @@ -35,42 +34,10 @@ async-recursion = "1.0.5" reqwest = { version = "0.11.22", features = ["rustls-tls", "stream"], default-features = false } xz2 = "0.1.7" -[dependencies.fuse-backend-rs] -optional = true -version = "0.11.0" - -[dependencies.vhost] -optional = true -version = "0.6" - -[dependencies.vhost-user-backend] -optional = true -version = "0.8" - -[dependencies.virtio-queue] -optional = true -version = "0.7" - -[dependencies.vm-memory] -optional = true -version = "0.10" - -[dependencies.vmm-sys-util] -optional = true -version = "0.11" - -[dependencies.virtio-bindings] -optional = true -version = "0.2.1" - [dependencies.tonic-reflection] optional = true version = "0.10.2" -[dependencies.libc] -optional = true -version = "0.2.144" - [build-dependencies] prost-build = "0.12.1" tonic-build = "0.10.2" @@ -80,19 +47,11 @@ test-case = "2.2.2" tempfile = "3.3.0" tokio-retry = "0.3.0" +[dev-dependencies.libc] +version = "0.2.144" + [features] default = ["fuse", "tonic-reflection"] -fs = ["dep:libc", "dep:fuse-backend-rs"] -virtiofs = [ - "fs", - "dep:vhost", - "dep:vhost-user-backend", - "dep:virtio-queue", - "dep:vm-memory", - "dep:vmm-sys-util", - "dep:virtio-bindings", - "fuse-backend-rs?/vhost-user-fs", # impl FsCacheReqHandler for SlaveFsCacheReq - "fuse-backend-rs?/virtiofs", -] -fuse = ["fs"] +fuse = ["tvix-castore/fuse"] +virtiofs = ["tvix-castore/virtiofs"] tonic-reflection = ["dep:tonic-reflection", "tvix-castore/tonic-reflection"] |