From d5b2f8e43fcd596f1f57fa46e5fe4cdbc764e29a Mon Sep 17 00:00:00 2001
From: Austin Seipp <aseipp@pobox.com>
Date: Thu, 5 Oct 2023 11:33:37 -0500
Subject: [PATCH] fix: run 'cargo test' under 'nix flake check' with a
 non-debug profile

Summary: The Nix CI has been failing recently due to (what I assume is) disk
space issues. But only the `flake check` step is failing. Right now, `nix flake
check` runs the Cargo tests with the debug profile to help get more debug info,
which is even heftier in terms of debug info than the normal 'test' profile. For
reference, a single build of 'cargo test' in a clean working copy results in a
15 gigabyte `target/` directory.

Turn off the debug profile for `nix flake check`, which should hopefully stem
the bleeding a bit. I believe the 'test' profile should still have enough
symbols for backtraces, so panics should still be useful.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
Change-Id: Idde10ac15847a1ad1e6f4e48a2497eca
---
 flake.nix | 2 --
 1 file changed, 2 deletions(-)

diff --git a/flake.nix b/flake.nix
index ae48a3176..645cf83f9 100644
--- a/flake.nix
+++ b/flake.nix
@@ -94,8 +94,6 @@
         program = "${self.packages.${system}.jujutsu}/bin/jj";
       };
       checks.jujutsu = self.packages.${system}.jujutsu.overrideAttrs ({ ... }: {
-        cargoBuildType = "debug";
-        cargoCheckType = "debug";
         preCheck = ''
           export RUST_BACKTRACE=1
         '';