Merge topic 'always_prefer_last_source_dir'

b44ad7992a cmake: Always prefer the last source directory provided

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !7084

2 files changed, 7 insertions, 23 deletions

diff --git a/Source/cmake.cxx b/Source/cmake.cxx
index 81d225d..f211314 100644
--- a/Source/cmake.cxx
+++ b/Source/cmake.cxx
@@ -819,7 +819,7 @@ void cmake::SetArgs(const std::vector<std::string>& args)
     std::string path = cmSystemTools::CollapseFullPath(value);
     cmSystemTools::ConvertToUnixSlashes(path);
 
-    state->SetHomeDirectoryViaCommandLine(path, HomeDirArgStyle::Dash_S);
+    state->SetHomeDirectoryViaCommandLine(path);
     return true;
   };
 
@@ -1555,7 +1555,7 @@ bool cmake::SetDirectoriesFromFile(const std::string& arg)
     // When invoked with a path that points to an existing CMakeCache
     // This function is called multiple times with the same path
     if (is_source_dir) {
-      this->SetHomeDirectoryViaCommandLine(listPath, HomeDirArgStyle::Plain);
+      this->SetHomeDirectoryViaCommandLine(listPath);
       if (no_build_tree) {
         std::string cwd = cmSystemTools::GetCurrentWorkingDirectory();
         this->SetHomeOutputDirectory(cwd);
@@ -1780,28 +1780,19 @@ void cmake::PrintPresetList(const cmCMakePresetsGraph& graph) const
 }
 #endif
 
-void cmake::SetHomeDirectoryViaCommandLine(std::string const& path,
-                                           HomeDirArgStyle argStyle)
+void cmake::SetHomeDirectoryViaCommandLine(std::string const& path)
 {
-  bool fromDashS = argStyle == HomeDirArgStyle::Dash_S;
-  static bool homeDirectorySetExplicitly = false;
   if (path.empty()) {
     return;
   }
 
   auto prev_path = this->GetHomeDirectory();
   if (prev_path != path && !prev_path.empty()) {
-    const bool ignore_prev_path =
-      (fromDashS || (!fromDashS && !homeDirectorySetExplicitly));
-    const std::string& ignored_path = (ignore_prev_path) ? prev_path : path;
     this->IssueMessage(MessageType::WARNING,
                        cmStrCat("Ignoring extra path from command line:\n \"",
-                                ignored_path, "\""));
+                                prev_path, "\""));
   }
-  if (fromDashS || !homeDirectorySetExplicitly) {
-    this->SetHomeDirectory(path);
-  }
-  homeDirectorySetExplicitly = fromDashS;
+  this->SetHomeDirectory(path);
 }
 
 void cmake::SetHomeDirectory(const std::string& dir)
diff --git a/Source/cmake.h b/Source/cmake.h
index 9c795c5..3c2a36c 100644
--- a/Source/cmake.h
+++ b/Source/cmake.h
@@ -183,12 +183,6 @@ public:
 #endif
   std::string ReportCapabilities() const;
 
-  enum class HomeDirArgStyle
-  {
-    Plain,
-    Dash_S,
-  };
-
   /**
    * Set the home directory from `-S` or from a known location
    * that contains a CMakeLists.txt. Will generate warnings
@@ -199,12 +193,11 @@ public:
    *  | `dirA dirA`       | dirA   | N/A            |
    *  | `-S dirA -S dirA` | dirA   | N/A            |
    *  | `-S dirA -S dirB` | dirB   | Ignoring dirA  |
-   *  | `-S dirA dirB`    | dirA   | Ignoring dirB  |
+   *  | `-S dirA dirB`    | dirB   | Ignoring dirA  |
    *  | `dirA -S dirB`    | dirB   | Ignoring dirA  |
    *  | `dirA dirB`       | dirB   | Ignoring dirA  |
    */
-  void SetHomeDirectoryViaCommandLine(std::string const& path,
-                                      HomeDirArgStyle argStyle);
+  void SetHomeDirectoryViaCommandLine(std::string const& path);
 
   //@{
   /**