From fbb1077e87eaead96814653646155b295d7f0b66 Mon Sep 17 00:00:00 2001
From: Iban Eguia <razican@protonmail.ch>
Date: Tue, 20 Oct 2020 12:08:50 +0200
Subject: [PATCH] Another fix for the historical results generation (#899)

---
 boa_tester/src/results.rs | 25 ++++++++++++++++---------
 1 file changed, 16 insertions(+), 9 deletions(-)

diff --git a/boa_tester/src/results.rs b/boa_tester/src/results.rs
index a787938ddc..68cca7244e 100644
--- a/boa_tester/src/results.rs
+++ b/boa_tester/src/results.rs
@@ -70,9 +70,6 @@ pub(crate) fn write_json(
             branch = "pull".to_owned();
         }
 
-        // We make sure we are using the latest commit information in GitHub pages:
-        update_gh_pages_repo(path);
-
         let path = if branch.is_empty() {
             path.to_path_buf()
         } else {
@@ -81,6 +78,9 @@ pub(crate) fn write_json(
             folder
         };
 
+        // We make sure we are using the latest commit information in GitHub pages:
+        update_gh_pages_repo(path.as_path(), verbose);
+
         if verbose != 0 {
             println!("Writing the results to {}...", path.display());
         }
@@ -140,7 +140,7 @@ fn get_test262_commit() -> Box<str> {
 }
 
 /// Updates the GitHub pages repository by pulling latest changes before writing the new things.
-fn update_gh_pages_repo(path: &Path) {
+fn update_gh_pages_repo(path: &Path, verbose: u8) {
     if env::var("GITHUB_REF").is_ok() {
         use std::process::Command;
 
@@ -151,11 +151,18 @@ fn update_gh_pages_repo(path: &Path) {
             .expect("could not update GitHub Pages");
 
         // Copy the full results file
-        fs::copy(
-            Path::new("../gh-pages/test262/refs/heads/master/").join(RESULTS_FILE_NAME),
-            path.join(RESULTS_FILE_NAME),
-        )
-        .expect("could not copy the master results file");
+        let from = Path::new("../gh-pages/test262/refs/heads/master/").join(RESULTS_FILE_NAME);
+        let to = path.join(RESULTS_FILE_NAME);
+
+        if verbose != 0 {
+            println!(
+                "Copying the {} file to {} in order to add the results",
+                from.display(),
+                to.display()
+            );
+        }
+
+        fs::copy(from, to).expect("could not copy the master results file");
     }
 }