From 9225ef0d3119967a042ac147243739ea130f0c19 Mon Sep 17 00:00:00 2001
From: Lars Wirzenius <liw@liw.fi>
Date: Fri, 12 Jan 2024 15:54:42 +0200
Subject: [PATCH] pass in RunLog, not filename for it

RunLog knows how to construct a whole log, rather than just writing
individual messages to it.

Signed-off-by: Lars Wirzenius <liw@liw.fi>
---
 src/bin/radicle-native-ci.rs | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/bin/radicle-native-ci.rs b/src/bin/radicle-native-ci.rs
index 3c91c1b..1975912 100644
--- a/src/bin/radicle-native-ci.rs
+++ b/src/bin/radicle-native-ci.rs
@@ -243,7 +243,7 @@ struct RunnerBuilder<'a> {
     commit: Option<Oid>,
     src: Option<PathBuf>,
     log: Option<&'a mut LogFile>,
-    run_log: Option<PathBuf>,
+    run_log: Option<RunLog>,
     timeout: Option<usize>,
     builder: Option<&'a mut RunInfoBuilder>,
 }
@@ -279,8 +279,8 @@ impl<'a> RunnerBuilder<'a> {
         self
     }
 
-    fn run_log(mut self, path: &Path) -> Self {
-        self.run_log = Some(path.into());
+    fn run_log(mut self, run_log: RunLog) -> Self {
+        self.run_log = Some(run_log);
         self
     }
 
@@ -295,8 +295,6 @@ impl<'a> RunnerBuilder<'a> {
     }
 
     fn build(self) -> Result<Runner<'a>, NativeError> {
-        let run_log = self.run_log.ok_or(NativeError::Unset("run_log"))?;
-        let run_log = RunLog::new(&run_log);
         Ok(Runner {
             run_id: self.run_id.ok_or(NativeError::Unset("run_id"))?,
             storage: self.storage.ok_or(NativeError::Unset("storage"))?,
@@ -304,7 +302,7 @@ impl<'a> RunnerBuilder<'a> {
             commit: self.commit.ok_or(NativeError::Unset("commit"))?,
             src: self.src.ok_or(NativeError::Unset("src"))?,
             log: self.log.ok_or(NativeError::Unset("log"))?,
-            run_log,
+            run_log: self.run_log.ok_or(NativeError::Unset("run_log"))?,
             timeout: self.timeout,
             builder: self.builder.ok_or(NativeError::Unset("builder"))?,
         })