Commit d1a7176f authored by Daniel Teske's avatar Daniel Teske
Browse files

QmakeProjectManager: Don't add mkspec include dir explicitly

It will be added via INCLUDE anyway.

This also removes all code using BaseQtVersion::systemHeaderPaths(),
which is thus also removed. This function was introduced to support

Change-Id: Id372e1cfc20bd22de948ed9a7d28dfa9135cbfa9
Task-number: QTCREATORBUG-13964
Reviewed-by: default avatarNikolai Kosjar <>
parent f5e2e6d4
......@@ -507,22 +507,9 @@ void QmakeProject::updateCppCodeModel()
// part->defines
templatePart->projectDefines += pro->cxxDefines();
// part->headerPaths
if (qtVersion) {
foreach (const HeaderPath &header, qtVersion->systemHeaderPathes(k)) {
ProjectPart::HeaderPath::Type type = ProjectPart::HeaderPath::IncludePath;
if (header.kind() == HeaderPath::FrameworkHeaderPath)
type = ProjectPart::HeaderPath::FrameworkPath;
templatePart->headerPaths += ProjectPart::HeaderPath(header.path(), type);
foreach (const QString &inc, pro->variableValue(IncludePathVar)) {
const auto headerPath
= ProjectPart::HeaderPath(inc, ProjectPart::HeaderPath::IncludePath);
// We've added header paths from qtVersion->systemHeaderPathes() above,
// which also contains the mkspecs dir. However, it's also part of
// the pro->variableValue(IncludePathVar), so check for duplicates.
if (!templatePart->headerPaths.contains(headerPath))
templatePart->headerPaths += headerPath;
......@@ -1183,14 +1183,6 @@ MacroExpander *BaseQtVersion::macroExpander() const
return &m_expander;
QList<HeaderPath> BaseQtVersion::systemHeaderPathes(const Kit *k) const
QList<HeaderPath> result;
result.append(HeaderPath(mkspecPath().toString(), HeaderPath::GlobalHeaderPath));
return result;
void BaseQtVersion::addToEnvironment(const Kit *k, Environment &env) const
......@@ -146,7 +146,6 @@ public:
bool hasDemos() const;
QString demosPath() const;
virtual QList<ProjectExplorer::HeaderPath> systemHeaderPathes(const ProjectExplorer::Kit *k) const;
virtual QString frameworkInstallPath() const;
// former local functions
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment