aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2022-03-17 13:43:24 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2022-03-17 13:43:24 +0200
commit2bb461f433da1a121dafc96a95363618795bd7fb (patch)
tree8002077195f3c10e5752a7a0234daa49d9e1b0c5
parent7b56e3f90be0efece268e85832dd9828a7490b13 (diff)
Handle absolute POSIX paths in MinGW GCC search paths
-rw-r--r--libbuild2/cc/gcc.cxx36
1 files changed, 34 insertions, 2 deletions
diff --git a/libbuild2/cc/gcc.cxx b/libbuild2/cc/gcc.cxx
index 3376c16..2c844b6 100644
--- a/libbuild2/cc/gcc.cxx
+++ b/libbuild2/cc/gcc.cxx
@@ -59,6 +59,25 @@ namespace build2
}
}
+#ifdef _WIN32
+ // Some misconfigured MinGW GCC builds add absolute POSIX directories to
+ // their built-in search paths (e.g., /mingw/{include,lib}) which GCC then
+ // interprets as absolute paths relative to the current drive (so the set
+ // of built-in search paths starts depending on where we run things from).
+ //
+ // While that's definitely misguided, life is short and we don't want to
+ // waste it explaining this in long mailing list threads and telling
+ // people to complain to whomever built their GCC. So we will just
+ // recreate the behavior in a way that's consistent with GCC and let
+ // people discover this on their own.
+ //
+ static inline void
+ add_current_drive (string& s)
+ {
+ s.insert (0, work.string (), 0, 2); // Add e.g., `c:`.
+ }
+#endif
+
// Extract system header search paths from GCC (gcc/g++) or compatible
// (Clang, Intel) using the `-v -E </dev/null` method.
//
@@ -169,7 +188,13 @@ namespace build2
dir_path d;
try
{
- d = dir_path (s, 1, s.size () - 1);
+ string ds (s, 1, s.size () - 1);
+
+#ifdef _WIN32
+ if (path_traits::is_separator (ds[0]))
+ add_current_drive (ds);
+#endif
+ d = dir_path (move (ds));
if (d.relative () || !exists (d, true))
continue;
@@ -343,7 +368,14 @@ namespace build2
dir_path d;
try
{
- d = dir_path (l, b, (e != string::npos ? e - b : e));
+ string ds (l, b, (e != string::npos ? e - b : e));
+
+#ifdef _WIN32
+ if (path_traits::is_separator (ds[0]))
+ add_current_drive (ds);
+#endif
+
+ d = dir_path (move (ds));
if (d.relative ())
throw invalid_path (move (d).string ());