refactor(tvix/derivation): move comma writing business up r/6452

write_input_derivations shouldn't need to write a comma to separate it
from the previous output from write_outputs.
This is better placed in the function calling all of these helper
functions.

Change-Id: I9ccc440e4665b52369ef39e75151b9a29469ce48
Reviewed-on: https://cl.tvl.fyi/c/depot/+/8995
Reviewed-by: raitobezarius <tvl@lahfa.xyz>
Autosubmit: flokli <flokli@flokli.de>
Tested-by: BuildkiteCI

1 files changed, 12 insertions, 0 deletions

diff --git a/tvix/nix-compat/src/derivation/mod.rs b/tvix/nix-compat/src/derivation/mod.rs
index b416e745a5..6a1eee4a81 100644
--- a/tvix/nix-compat/src/derivation/mod.rs
+++ b/tvix/nix-compat/src/derivation/mod.rs
@@ -51,11 +51,23 @@ impl Derivation {
         write::write_char(writer, write::PAREN_OPEN)?;
 
         write::write_outputs(writer, &self.outputs)?;
+        write::write_char(writer, write::COMMA)?;
+
         write::write_input_derivations(writer, &self.input_derivations)?;
+        write::write_char(writer, write::COMMA)?;
+
         write::write_input_sources(writer, &self.input_sources)?;
+        write::write_char(writer, write::COMMA)?;
+
         write::write_system(writer, &self.system)?;
+        write::write_char(writer, write::COMMA)?;
+
         write::write_builder(writer, &self.builder)?;
+        write::write_char(writer, write::COMMA)?;
+
         write::write_arguments(writer, &self.arguments)?;
+        write::write_char(writer, write::COMMA)?;
+
         write::write_enviroment(writer, &self.environment)?;
 
         write::write_char(writer, write::PAREN_CLOSE)?;