aboutsummaryrefslogtreecommitdiff
path: root/libbuild2/cc/compile-rule.cxx
diff options
context:
space:
mode:
Diffstat (limited to 'libbuild2/cc/compile-rule.cxx')
-rw-r--r--libbuild2/cc/compile-rule.cxx35
1 files changed, 28 insertions, 7 deletions
diff --git a/libbuild2/cc/compile-rule.cxx b/libbuild2/cc/compile-rule.cxx
index 2af5012..8fe5e98 100644
--- a/libbuild2/cc/compile-rule.cxx
+++ b/libbuild2/cc/compile-rule.cxx
@@ -1568,10 +1568,26 @@ namespace build2
//
// c1xx: fatal error C1083: Cannot open source file: 's.cpp': No such
// file or directory
+ //
+ // And it turns out C1083 is also used when we are unable to open a type
+ // library specified with #import. In this case the error looks like this
+ // (at least in VC 14, 15, and 16):
+ //
+ // ...\comdef.h: fatal error C1083: Cannot open type library file:
+ // 'l.tlb': Error loading type library/DLL.
+ //
- size_t
+ pair<size_t, size_t>
msvc_sense_diag (const string&, char); // msvc.cxx
+ static inline bool
+ msvc_header_c1083 (const string& l, const pair<size_t, size_t>& pr)
+ {
+ return
+ l.compare (pr.second, 5, "c1xx:") != 0 && /* Not source file. */
+ l.compare (pr.second, 9, "comdef.h:") != 0; /* Not type library. */
+ }
+
// Extract the include path from the VC /showIncludes output line. Return
// empty string if the line is not an include note or include error. Set
// the good_error flag if it is an include error (which means the process
@@ -1584,7 +1600,9 @@ namespace build2
//
assert (!good_error);
- size_t p (msvc_sense_diag (l, 'C'));
+ pair<size_t, size_t> pr (msvc_sense_diag (l, 'C'));
+ size_t p (pr.first);
+
if (p == string::npos)
{
// Include note.
@@ -1623,8 +1641,7 @@ namespace build2
return string (l, p);
}
- else if (l.compare (p, 4, "1083") == 0 &&
- l.compare (0, 5, "c1xx:") != 0 /* Not the main source file. */ )
+ else if (l.compare (p, 4, "1083") == 0 && msvc_header_c1083 (l, pr))
{
// Include error.
//
@@ -3816,7 +3833,7 @@ namespace build2
{
// D8XXX are errors while D9XXX are warnings.
//
- size_t p (msvc_sense_diag (l, 'D'));
+ size_t p (msvc_sense_diag (l, 'D').first);
if (p != string::npos && l[p] == '9')
continue;
@@ -4015,9 +4032,13 @@ namespace build2
//
for (; !eof (getline (is, l)); )
{
- size_t p (msvc_sense_diag (l, 'C'));
- if (p != string::npos && l.compare (p, 4, "1083") != 0)
+ pair<size_t, size_t> p (msvc_sense_diag (l, 'C'));
+ if (p.first != string::npos &&
+ l.compare (p.first, 4, "1083") != 0 &&
+ msvc_header_c1083 (l, p))
+ {
diag_stream_lock () << l << endl;
+ }
}
}