feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity
Files
32018623a444a53038239e911df59b6f879ff89c
qt-creator/src/plugins/cmakeprojectmanager/builddirreader.h

87 lines
2.9 KiB
C
Raw Normal View History

CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
/****************************************************************************
**
** Copyright (C) 2016 The Qt Company Ltd.
** Contact: https://www.qt.io/licensing/
**
** This file is part of Qt Creator.
**
** Commercial License Usage
** Licensees holding valid commercial Qt licenses may use this file in
** accordance with the commercial license agreement provided with the
** Software or, alternatively, in accordance with the terms contained in
** a written agreement between you and The Qt Company. For licensing terms
** and conditions see https://www.qt.io/terms-conditions. For further
** information use the contact form at https://www.qt.io/contact-us.
**
** GNU General Public License Usage
** Alternatively, this file may be used under the terms of the GNU
** General Public License version 3 as published by the Free Software
** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
** included in the packaging of this file. Please review the following
** information to ensure the GNU General Public License requirements will
** be met: https://www.gnu.org/licenses/gpl-3.0.html.
**
****************************************************************************/
#pragma once
CMake: Fix project parsing notification This builds on top of 08677c0b014cc44d944e32d462f502a67c948404 and fixes one more code path to go through a common entry/exit point. Change-Id: I1d00fa9242f247028e5d3b0ef3b5fe1d3f4cb03d Reviewed-by: hjk <hjk@qt.io> Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
2017-09-28 11:32:39 +02:00
#include "builddirparameters.h"
#include "cmakebuildtarget.h"
CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
#include "cmakeconfigitem.h"
#include "cmaketool.h"
CppTools: Move RawProjectPart to ProjectExplorer Doesn't have any dependencies into CppTools anymore, therefore moving it reduces the dependencies of the project managers to CppTools as well. Change-Id: Ibe728abe59eb88a8877943dca1f48a85163e27ac Reviewed-by: hjk <hjk@qt.io>
2019-08-28 18:22:45 +02:00
#include <projectexplorer/rawprojectpart.h>
CppTools/ProjectManagers: Reduce ui blocking when loading projects ${AnyProject}::updateCppCodeModel() did two potentially not that cheap operations in the ui thread: (1) Querying the MimeDatabase for the mime type for the source files of the project. In 99.9% of the cases no files need to be read for this as the file extension will resolve the type. The expensiveness comes from the sheer number of files that can occur. (2) Calling compilers with the "(sub)project's compiler command line" to determine the macros. While the caches avoid redundant calls, the number of the unique compiler calls makes this still a ui-freezing experience. These two operations are moved into a worker thread. For this, the expensive compiler calls are encapsulated in thread safe lambdas ("runners") in order to keep the "mutexed" data minimal. The original API calls of the toolchains are implemented in terms of the runners. While adapting the project managers, remove also the calls to setProjectLanguage(). These are redundant because all of the project managers already set a proper value in the constructor. Also, currently there is no need (client) to report back detection of C sources in project parts. This also keeps CppProjectUpdater simple. There is still room for improvement: * Run the compiler calls in parallel instead of sequence. * Ensure that the mime type for a file is determined exactly once. Change-Id: I2efc4e132ee88e3c8f264012ec8fafe3d86c404f Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
2017-02-06 16:59:53 +01:00
CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
#include <utils/environment.h>
#include <utils/fileutils.h>
#include <utils/macroexpander.h>
#include <QFutureInterface>
#include <QObject>
CMake: Simplify based on Project::projectFileIsDirty signal Delegate all the necessary file watching to Project and connect to the relevant signal. Server-mode insists on watching files itself, so that may not report extra project files. Change-Id: If821c54a7b0f8b72beed53dd1c83f255973faf3e Reviewed-by: hjk <hjk@qt.io>
2019-08-15 12:22:09 +02:00
#include <QVector>
CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
CMake: Fix project parsing notification This builds on top of 08677c0b014cc44d944e32d462f502a67c948404 and fixes one more code path to go through a common entry/exit point. Change-Id: I1d00fa9242f247028e5d3b0ef3b5fe1d3f4cb03d Reviewed-by: hjk <hjk@qt.io> Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
2017-09-28 11:32:39 +02:00
namespace ProjectExplorer { class FileNode; }
CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
namespace CMakeProjectManager {
namespace Internal {
class CMakeBuildConfiguration;
CMake: Use a CMakeProjectNode as top level cmake node Change-Id: I6e68b971cf490b9b064a2866bd99865a7f4a7bd9 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
2017-03-14 15:35:02 +01:00
class CMakeProjectNode;
CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
class BuildDirReader : public QObject
{
Q_OBJECT
public:
CMake: Use unique_ptr to manage builddirreader Change-Id: I1acac29bdfa4dfaaf79835636c3e3a4484e8183c Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>
2019-06-05 15:42:46 +02:00
static std::unique_ptr<BuildDirReader> createReader(const BuildDirParameters &p);
CMake: Make all readers use isReadyNow() signal The tealeafreader used to not ever use this signal and used the isReady() method to short-circuit the whole mechanism. Change-Id: I2f7caedeedf977c70a5d7807e3a92775757a6fef Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>
2019-06-05 15:56:37 +02:00
virtual void setParameters(const BuildDirParameters &p) = 0;
CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
CMake: Fix project parsing notification This builds on top of 08677c0b014cc44d944e32d462f502a67c948404 and fixes one more code path to go through a common entry/exit point. Change-Id: I1d00fa9242f247028e5d3b0ef3b5fe1d3f4cb03d Reviewed-by: hjk <hjk@qt.io> Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
2017-09-28 11:32:39 +02:00
virtual bool isCompatible(const BuildDirParameters &p) = 0;
CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
virtual void resetData() = 0;
CMake: Make sure to run cmake when that is requested Pass on the request to run cmake on to the builddirreaders. Task-number: QTCREATORBUG-19704 Change-Id: Iafeba61f46d6bb6783fc62d0504bb07dde2c4612 Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
2019-06-06 15:07:17 +02:00
virtual void parse(bool forceCMakeRun, bool forceConfiguration) = 0;
CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
virtual void stop() = 0;
virtual bool isParsing() const = 0;
CMake: Simplify based on Project::projectFileIsDirty signal Delegate all the necessary file watching to Project and connect to the relevant signal. Server-mode insists on watching files itself, so that may not report extra project files. Change-Id: If821c54a7b0f8b72beed53dd1c83f255973faf3e Reviewed-by: hjk <hjk@qt.io>
2019-08-15 12:22:09 +02:00
virtual QVector<Utils::FilePath> takeProjectFilesToWatch() = 0;
CMake: Unify error reporting for builddirmanager's information retrieval Use a dedicated errrorMessage out parameter for error reporting in the builddirmanager methods related to information retrieval. Those are called after the parsing has finished. This frees the errrorOccured signal of the builddirmanager to be used only when the parsing itself has failed. Change-Id: Ieefc32c0386769479177a6bd4bc4a0e77df5db7b Reviewed-by: Cristian Adam <cristian.adam@qt.io> Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
2019-06-07 17:13:49 +02:00
virtual QList<CMakeBuildTarget> takeBuildTargets(QString &errorMessage) = 0;
virtual CMakeConfig takeParsedConfiguration(QString &errorMessage) = 0;
CMake: Add initial fileapireader class Change-Id: I620cba7cc1c2a5ac56789fa9770dce573c6b19cd Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2019-06-13 14:24:04 +02:00
virtual std::unique_ptr<CMakeProjectNode> generateProjectTree(
const QList<const ProjectExplorer::FileNode *> &allFiles, QString &errorMessage)
= 0;
CppTools: Move RawProjectPart to ProjectExplorer Doesn't have any dependencies into CppTools anymore, therefore moving it reduces the dependencies of the project managers to CppTools as well. Change-Id: Ibe728abe59eb88a8877943dca1f48a85163e27ac Reviewed-by: hjk <hjk@qt.io>
2019-08-28 18:22:45 +02:00
virtual ProjectExplorer::RawProjectParts createRawProjectParts(QString &errorMessage) = 0;
CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
signals:
CMake: Implement a reader that interacts with CMake server mode Implement a reader that interacts with CMake server mode to retrieve its data. Change-Id: I4146a648475c2c5811b8a08126f10ff3b99e79f5 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
2016-10-17 14:35:48 +02:00
void isReadyNow() const;
CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
void configurationStarted() const;
void dataAvailable() const;
void dirty() const;
void errorOccured(const QString &message) const;
protected:
CMake: Fix project parsing notification This builds on top of 08677c0b014cc44d944e32d462f502a67c948404 and fixes one more code path to go through a common entry/exit point. Change-Id: I1d00fa9242f247028e5d3b0ef3b5fe1d3f4cb03d Reviewed-by: hjk <hjk@qt.io> Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
2017-09-28 11:32:39 +02:00
BuildDirParameters m_parameters;
CMake: Implement different backends to run cmake Only the original one is implemented so far, but at least in theory backends for retrieving data from cmake can now be switched at runtime. Change-Id: Id73a81c7d40f078be95defd30a38511dca3a3720 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
2016-10-17 13:55:54 +02:00
};
} // namespace Internal
} // namespace CMakeProjectManager
Copy Permalink