about summary refs log tree commit diff
path: root/tvix/nix-compat/src
diff options
context:
space:
mode:
authorFlorian Klink <flokli@flokli.de>2023-07-30T07·37+0200
committerclbot <clbot@tvl.fyi>2023-07-31T21·50+0000
commitaa1982c0852de539eeddd742f7c41bd1a873a53c (patch)
tree8249d13193809bb889950e456dbe18e37063f294 /tvix/nix-compat/src
parent3f856d897713aca25156cc7d471c54845367bfb6 (diff)
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
Diffstat (limited to 'tvix/nix-compat/src')
-rw-r--r--tvix/nix-compat/src/derivation/mod.rs12
-rw-r--r--tvix/nix-compat/src/derivation/write.rs9
2 files changed, 13 insertions, 8 deletions
diff --git a/tvix/nix-compat/src/derivation/mod.rs b/tvix/nix-compat/src/derivation/mod.rs
index b416e745a565..6a1eee4a81c3 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)?;
diff --git a/tvix/nix-compat/src/derivation/write.rs b/tvix/nix-compat/src/derivation/write.rs
index 22985fb4b55f..223ba9ff3fd9 100644
--- a/tvix/nix-compat/src/derivation/write.rs
+++ b/tvix/nix-compat/src/derivation/write.rs
@@ -107,7 +107,6 @@ pub fn write_input_derivations(
     writer: &mut impl Write,
     input_derivations: &BTreeMap<String, BTreeSet<String>>,
 ) -> Result<(), io::Error> {
-    write_char(writer, COMMA)?;
     write_char(writer, BRACKET_OPEN)?;
 
     for (ii, (input_derivation_path, input_derivation)) in input_derivations.into_iter().enumerate()
@@ -142,8 +141,6 @@ pub fn write_input_sources(
     writer: &mut impl Write,
     input_sources: &BTreeSet<String>,
 ) -> Result<(), io::Error> {
-    write_char(writer, COMMA)?;
-
     write_char(writer, BRACKET_OPEN)?;
     write_array_elements(
         writer,
@@ -158,19 +155,16 @@ pub fn write_input_sources(
 }
 
 pub fn write_system(writer: &mut impl Write, platform: &str) -> Result<(), Error> {
-    write_char(writer, COMMA)?;
     write_field(writer, platform, true)?;
     Ok(())
 }
 
 pub fn write_builder(writer: &mut impl Write, builder: &str) -> Result<(), Error> {
-    write_char(writer, COMMA)?;
     write_field(writer, builder, true)?;
     Ok(())
 }
-pub fn write_arguments(writer: &mut impl Write, arguments: &[String]) -> Result<(), io::Error> {
-    write_char(writer, COMMA)?;
 
+pub fn write_arguments(writer: &mut impl Write, arguments: &[String]) -> Result<(), io::Error> {
     write_char(writer, BRACKET_OPEN)?;
     write_array_elements(
         writer,
@@ -188,7 +182,6 @@ pub fn write_enviroment(
     writer: &mut impl Write,
     environment: &BTreeMap<String, BString>,
 ) -> Result<(), io::Error> {
-    write_char(writer, COMMA)?;
     write_char(writer, BRACKET_OPEN)?;
 
     for (i, (k, v)) in environment.into_iter().enumerate() {