From a1bd95797b1e51b56ffea872952fa54b95fcfde0 Mon Sep 17 00:00:00 2001 From: Jaap Frolich Date: Fri, 7 Mar 2025 11:28:09 +0100 Subject: [PATCH] remove dirty tracking optimization for incremental build --- src/build.rs | 21 ++------------------- 1 file changed, 2 insertions(+), 19 deletions(-) diff --git a/src/build.rs b/src/build.rs index 4cd48792..ccc6e01c 100644 --- a/src/build.rs +++ b/src/build.rs @@ -12,7 +12,6 @@ use crate::build::compile::{mark_modules_with_deleted_deps_dirty, mark_modules_w use crate::helpers::emojis::*; use crate::helpers::{self, get_workspace_root}; use crate::sourcedirs; -use ahash::AHashSet; use anyhow::{anyhow, Result}; use build_types::*; use console::style; @@ -275,7 +274,7 @@ impl fmt::Display for IncrementalBuildError { pub fn incremental_build( build_state: &mut BuildState, default_timing: Option, - initial_build: bool, + _initial_build: bool, show_progress: bool, only_incremental: bool, create_sourcedirs: bool, @@ -349,17 +348,7 @@ pub fn incremental_build( ); } - // track the compile dirty state, we reset it when the compile fails - let mut tracked_dirty_modules = AHashSet::new(); - for (module_name, module) in build_state.modules.iter() { - if module.compile_dirty { - tracked_dirty_modules.insert(module_name.to_owned()); - } - } - if initial_build { - // repair broken state - mark_modules_with_expired_deps_dirty(build_state); - } + mark_modules_with_expired_deps_dirty(build_state); mark_modules_with_deleted_deps_dirty(build_state); current_step += 1; @@ -418,12 +407,6 @@ pub fn incremental_build( ); } log::error!("{}", &compile_errors); - // mark the original files as dirty again, because we didn't complete a full build - for (module_name, module) in build_state.modules.iter_mut() { - if tracked_dirty_modules.contains(module_name) { - module.compile_dirty = true; - } - } Err(IncrementalBuildError::CompileError(None)) } else { if show_progress {