chore(build): compile protobufs with protox (#2122)

* chore(build): compile protobufs with protox

protox is a pure-rust implementation of the protobuf compiler.
Therefore, it can be managed by cargo.

This removes the implicit dependency on protoc being available
in the environment for the build.

* fix(build): replace copypasta in build script

The paths passed to `compile` aren't actually used by the build.
`skip_protoc_run` prevents that.
That's why a clean build succeeds even with this mistake.

However, the paths are passed to a `cargo:rerun-if-changed` directive.
So this mistake would've caused a failed incremental build if the
protobuf definitions were changed.

1 files changed, 1 insertions, 4 deletions

diff --git a/atuin.nix b/atuin.nix
index 43450b8e..c02a82de 100644
--- a/atuin.nix
+++ b/atuin.nix
@@ -13,7 +13,6 @@
   Security,
   SystemConfiguration,
   AppKit,
-  protobuf,
 }:
 rustPlatform.buildRustPackage {
   name = "atuin";
@@ -28,9 +27,7 @@ rustPlatform.buildRustPackage {
 
   nativeBuildInputs = [installShellFiles];
 
-  buildInputs = lib.optionals stdenv.isDarwin [libiconv Security SystemConfiguration AppKit protobuf];
-
-  env.PROTOC = lib.getExe' protobuf "protoc";
+  buildInputs = lib.optionals stdenv.isDarwin [libiconv Security SystemConfiguration AppKit];
 
   postInstall = ''
     installShellCompletion --cmd atuin \