Merge pull request #48 from Neko-Box-Coder/StaleFileFix

Fixing stale files not being detected properly

Author: Neko-Box-Coder

Include/runcpp2/runcpp2.hpp

@@ -25,13 +25,17 @@ namespace runcpp2
 
     void SetLogLevel(const std::string& logLevel);
 
-    PipelineResult CheckSourcesNeedUpdate(  const std::string& scriptPath,
-                                            const std::vector<Data::Profile>& profiles,
-                                            const std::string& configPreferredProfile,
-                                            const Data::ScriptInfo& scriptInfo,
-                                            const std::unordered_map<   CmdOptions, 
-                                                                        std::string>& currentOptions,
-                                            bool& outNeedsUpdate);
+    //NOTE: This should be run after running StartPipeline first
+    PipelineResult 
+    CheckSourcesNeedUpdate( const std::string& scriptPath,
+                            const std::vector<Data::Profile>& profiles,
+                            const std::string& configPreferredProfile,
+                            const Data::ScriptInfo& scriptInfo,
+                            const std::unordered_map<   CmdOptions, 
+                                                        std::string>& currentOptions,
+                            const ghc::filesystem::file_time_type& prevFinalSourceWriteTime,
+                            const ghc::filesystem::file_time_type& prevFinalIncludeWriteTime,
+                            bool& outNeedsUpdate);
 
     PipelineResult StartPipeline(   const std::string& scriptPath, 
                                     const std::vector<Data::Profile>& profiles,
@@ -41,6 +45,8 @@ namespace runcpp2
                                     const Data::ScriptInfo* lastScriptInfo,
                                     const std::string& buildOutputDir,
                                     Data::ScriptInfo& outScriptInfo,
+                                    ghc::filesystem::file_time_type& outFinalSourceWriteTime,
+                                    ghc::filesystem::file_time_type& outFinalIncludeWriteTime,
                                     int& returnStatus);
 
     std::string PipelineResultToString(PipelineResult result);

Src/runcpp2/main.cpp

@@ -424,6 +424,8 @@ int main(int argc, char* argv[])
         }
 
         runcpp2::Data::ScriptInfo* lastParsedScriptInfo = nullptr;
+        ghc::filesystem::file_time_type lastFinalSourceWriteTime;
+        ghc::filesystem::file_time_type lastFinalIncludeWriteTime;
         bool needsRunning = true;  //First run always needs running
 
         while(true)
@@ -439,6 +441,8 @@ int main(int argc, char* argv[])
                                                         *lastParsedScriptInfo : 
                                                         parsedScriptInfo,
                                                     currentOptions,
+                                                    lastFinalSourceWriteTime,
+                                                    lastFinalIncludeWriteTime,
                                                     needsUpdate) != runcpp2::PipelineResult::SUCCESS)
                 {
                     ssLOG_ERROR("Failed to check if sources need update");
@@ -477,6 +481,8 @@ int main(int argc, char* argv[])
                                             lastParsedScriptInfo,
                                             "",
                                             parsedScriptInfo,
+                                            lastFinalSourceWriteTime,
+                                            lastFinalIncludeWriteTime,
                                             result);
 
                 static_assert(static_cast<int>(runcpp2::PipelineResult::COUNT) == 13, "Update this");
@@ -518,6 +524,9 @@ int main(int argc, char* argv[])
     if(currentOptions.count(runcpp2::CmdOptions::BUILD) > 0)
         outputDir = ghc::filesystem::current_path().string();
 
+    ghc::filesystem::file_time_type finalSourceWriteTime;
+    ghc::filesystem::file_time_type finalIncludeWriteTime;
+    
     if(runcpp2::StartPipeline(  script, 
                                 profiles, 
                                 preferredProfile, 
@@ -526,6 +535,8 @@ int main(int argc, char* argv[])
                                 nullptr,
                                 outputDir,
                                 parsedScriptInfo,
+                                finalSourceWriteTime,
+                                finalIncludeWriteTime,
                                 result) != runcpp2::PipelineResult::SUCCESS)
     {
         return -1;