From f5ccb65f26b18e576d1eb50e4a981230adb9101d Mon Sep 17 00:00:00 2001 From: Florian Klink Date: Wed, 1 Feb 2023 11:45:44 +0100 Subject: feat(tvix/nix-compat/derivation): Display -> to_aterm_string() Instead of implementing `std::fmt::Display for Derivation` and relying on the `to_string` method, introduce a `to_aterm_string()` method, which does the same thing, but makes it clearer what we're producing, rather than just calling `to_string()``. Change-Id: I21823de9096a0f2c2eb6f4591e48c1aa9fd94161 Reviewed-on: https://cl.tvl.fyi/c/depot/+/7998 Autosubmit: flokli Reviewed-by: tazjin Tested-by: BuildkiteCI --- tvix/nix-compat/src/derivation/mod.rs | 26 +++++++++++++++++--------- 1 file changed, 17 insertions(+), 9 deletions(-) (limited to 'tvix/nix-compat/src/derivation/mod.rs') diff --git a/tvix/nix-compat/src/derivation/mod.rs b/tvix/nix-compat/src/derivation/mod.rs index cecaa3f42d80..f500161f9181 100644 --- a/tvix/nix-compat/src/derivation/mod.rs +++ b/tvix/nix-compat/src/derivation/mod.rs @@ -40,6 +40,9 @@ pub struct Derivation { } impl Derivation { + /// write the Derivation to the given [std::fmt::Write], in ATerm format. + /// + /// The only errors returns are these when writing to the passed writer. pub fn serialize(&self, writer: &mut impl std::fmt::Write) -> Result<(), std::fmt::Error> { writer.write_str(write::DERIVATION_PREFIX)?; writer.write_char(write::PAREN_OPEN)?; @@ -57,6 +60,18 @@ impl Derivation { Ok(()) } + /// return the ATerm serialization as a string. + pub fn to_aterm_string(&self) -> String { + let mut buffer = String::new(); + + // invoke serialize and write to the buffer. + // Note we only propagate errors writing to the writer in serialize, + // which won't panic for the string we write to. + self.serialize(&mut buffer).unwrap(); + + buffer + } + /// Returns the fixed output path and its hash // (if the Derivation is fixed output), /// or None if there is no fixed output. @@ -116,7 +131,7 @@ impl Derivation { // it as a hex-encoded string (prefixed with sha256:). let aterm_digest = { let mut derivation_hasher = Sha256::new(); - derivation_hasher.update(self.to_string()); + derivation_hasher.update(self.to_aterm_string()); derivation_hasher.finalize() }; @@ -171,7 +186,7 @@ impl Derivation { }; // write the ATerm of that to the hash function - hasher.update(replaced_derivation.to_string()); + hasher.update(replaced_derivation.to_aterm_string()); hasher.finalize() } @@ -281,10 +296,3 @@ impl Derivation { Ok(()) } } - -impl std::fmt::Display for Derivation { - /// Formats the Derivation in ATerm representation. - fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result { - self.serialize(f) - } -} -- cgit 1.4.1