Details
-
Epic
-
Resolution: Unresolved
-
Not Evaluated
-
None
-
None
-
Stable API for Runcontrol / Runworker
-
bd818c7f0 (11.0), 2cc496745 (master), 941c99c89 (master), e53bc1630 (master), a5ecc6207 (master), b196f5f03 (master), f9fb3f487 (master), 619c49958 (master), 382e003de (master), e75b81b0a (master), 9888e1982 (master), 4c8c793c0 (master), 5ee1f47d2 (16.0)
Description
RunControl, together with a number of dynamically created instances of RunWorker (45 different subclasses), is responsible for running user applications / profilers / debuggers, etc. The current implementation is error-prone with regards to the manual managing of the RunWorkers' lifetime, hidden dependencies between them, unexpected state transitions, and cumbersome data exchange between them.
The alternative design would be to use TaskTree as a replacement of RunControl, and operate on task tree recipes returned by relevant RunWorkerFactories instead on instances of RunWorker subclasses.
More details:
The request for "stable Qt Creator Plugin API" comes up regularly. We currently still need the freedom to change also core API, but for limited areas this seems in reach. This here is one of them.
The current API is sufficient for the current purposes and has not changed significantly since 2019, but it's not good enough to set in stone in its current shape.
We have (among others) two special kind of plugins, one set providing "platform" support (i.e. organize the actual of process on certain devices, possibly accessing the file system there - 'local', android, baremetal, ios, (remote)linux, docker, qnx, [webassembly, vxworks]) and "tool" support (input/output handling of certain 'tools' that can potentially run on different devices = 'normal application run', debugger (c++/qml/mixed - gdb/lldb/cdb), qmlprofiler, perf, profiler (memcheck, cachegrind),). The challenge here is to organize combination of those (i.e. "run a given tool on a certain plugin") without creating dependencies between the respective "platform plugin" and "tool plugins" (internally known as "The Matrix Problem" as the "obvious" solution to have specialized plugins for each combination does not scale).
With the current RunWorker system we have a partial solution to the Matrix problem, splitting the platform and tool parts into two workers, but setting up the right dependencies between them, especially when extra glue is needed (e.g. using PortGatherers to get free ports for comunication) it is pretty much ad hoc.
We should move the whole machinery to use uniformly TaskTree for dependency management and Aspects for data.
Attachments
Issue Links
- relates to
-
QTCREATORBUG-29337 New plugin/extension manager
-
- Open
-
-
QTCREATORBUG-31844 Redesign of RunControl / RunWorker
-
- Closed
-
Gerrit Reviews
For Gerrit Dashboard: QTCREATORBUG-29168 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
604950,22 | Boot2Qt: Dismantle QdbDevicePerfProfilerSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605420,11 | QmlPreview: Remove RefreshTranslationWorker | master | qt-creator/qt-creator | Status: MERGED | +2 | +1 |
605555,9 | Qnx: Inline QnxAttachDebugSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605558,11 | Qnx: Inline PDebugRunner | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605563,10 | Qnx: Inline QnxDebuggeeRunner | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605564,10 | Qnx: Inline QnxDebugSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605569,3 | PE: Promote protected methods to public in SimpleTargetRunner | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605599,4 | QmlPreview: Inline LocalQmlPreviewSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605602,2 | WebAssembly: Inline class EmrunRunWorker | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605604,2 | Boot2Qt: Inline QdbDeviceRunSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605609,7 | AppMan: Inline AppManagerRunner | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605621,4 | BareMetal: Inline GdbServerProviderRunner | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605622,4 | Qnx: Inline QnxQmlProfilerSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605624,15 | QmlProfiler: Inline LocalQmlProfilerSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605627,12 | RemoteLinux: Inline RemoteLinuxQmlToolingSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605629,4 | McuSupport: Inline FlashAndRunWorker | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605630,16 | ProjectExplorer: Make SimpleTargetRunner final | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605632,7 | ProjectExplorer: Remove unused SimpleTargetRunner::setExtraData() | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605633,6 | Android: Drop AndroidDebugSupport::stop() overload | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605634,15 | Boot2Qt: Drop QdbDeviceDebugSupport::stop() overload | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605635,7 | Android: Drop AndroidQmlToolingSupport::stop() | master | qt-creator/qt-creator | Status: ABANDONED | 0 | 0 |
605636,16 | Android: Inline AndroidQmlToolingSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605637,15 | AppMan: Inline AppManagerPerfProfilerSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605638,15 | AppMan: Inline AppManagerQmlToolingSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605639,15 | Boot2Qt: Remove unneeded friend declarations in QdbDeviceInferiorRunner | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605651,15 | Boot2Qt: Inline QdbDeviceQmlToolingSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605659,3 | Ios: Remove unused field | master | qt-creator/qt-creator | Status: MERGED | +2 | +1 |
605756,19 | Boot2Qt: Inline QdbDeviceInferiorRunner | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605767,11 | Boot2Qt: Simplify QdbDeviceDebugSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605769,15 | Boot2Qt: Inline QdbDeviceDebugSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605776,18 | Ios: Move part of start setup into IosDebugSupport's c'tor | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605808,3 | Android: Drop qml port parsing from application output | master | qt-creator/qt-creator | Status: MERGED | +2 | +1 |
605809,16 | Android: Move part of start setup into AndroidDebugSupport's c'tor | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
605834,14 | BareMetal: Inline UvscServerProviderRunner | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
613244,2 | VxWorks: Get rid of empty RunWorker subclass | master | qt-creator/tqtc-plugin-vxworks | Status: ABANDONED | 0 | 0 |
614532,16 | PerfProfiler: Get rid of LocalPerfRecordWorker | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
614757,17 | PerfProfiler: Remove private fields of PerfProfilerRunner | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
614761,17 | PerfProfiler: Dismantle PerfProfilerRunner | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616109,13 | Debugger: Transform DebuggerRunParameters::startMode | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616146,7 | Debugger: Transform DebuggerRunParameters::startMode | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616153,8 | Debugger: Transform DebuggerRunParameters::inferior | master | qt-creator/qt-creator | Status: MERGED | +2 | -1 |
616193,7 | Debugger: Transform DebuggerRunParameters::displayName | master | qt-creator/qt-creator | Status: MERGED | +2 | -1 |
616219,9 | Debugger: Transform DebuggerRunParameters::attachPid | master | qt-creator/qt-creator | Status: MERGED | +2 | +1 |
616223,11 | Debugger: Transform DebuggerRunParameters::solibSearchPath | master | qt-creator/qt-creator | Status: MERGED | +2 | +1 |
616226,11 | Debugger: Transform DebuggerRunParameters::qmlServer | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616230,11 | Debugger: Transform DebuggerRunParameters::remoteChannel | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616238,10 | Debugger: Transform DebuggerRunParameters::useExtendedRemote | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616259,5 | Debugger: Transform DebuggerRunParameters::symbolFile | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616266,4 | Debugger: Transform DebuggerRunParameters::sourcePathMap | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616269,4 | Debugger: Transform DebuggerRunParameters::commandsForReset | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616272,4 | Debugger: Transform DebuggerRunParameters::useContinueInsteadOfRun | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616274,5 | Debugger: Transform DebuggerRunParameters::commandsAfterConnect | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616276,5 | Debugger: Transform DebuggerRunParameters::expectedSignals | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616280,5 | Debugger: Transform DebuggerRunParameters::useCtrlCStub | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616282,5 | Debugger: Get rid of DebuggerRunParameters::skipExecutableValidation | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616283,4 | Debugger: Transform DebuggerRunParameters::useTargetAsync | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616286,4 | Debugger: Transform DebuggerRunParameters::additionalSearchDirectories | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616288,6 | Debugger: Transform DebuggerRunParameters::platform | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616289,6 | Debugger: Transform DebuggerRunParameters::deviceSymbolsRoot | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616291,6 | Debugger: Transform DebuggerRunParameters::continueAfterAttach | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616317,5 | Debugger: Transform DebuggerRunParameters::sysRoot | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616504,3 | Debugger: Transform DebuggerRunParameters::deviceUuid | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616506,3 | Debugger: Transform DebuggerRunParameters::coreFile | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616508,5 | Debugger: Transform DebuggerRunParameters::additionalStartupCommands | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616550,4 | Debugger: Transform DebuggerRunParameters::cppEngineType | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616553,2 | Debugger: Transform DebuggerRunParameters::version | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616566,3 | Debugger: Transform DebuggerRunParameters::isPythonDebugging | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616573,3 | Debugger: Transform DebuggerRunParameters::breakOnMain | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616592,2 | Debugger: Transform a few fields of DebuggerRunParameters | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616599,2 | Debugger: Transform DebuggerRunParameters::debugger | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616630,4 | Debugger: Transform a few fields of DebuggerRunParameters | master | qt-creator/qt-creator | Status: MERGED | +2 | +1 |
616658,4 | Debugger: Transform a few fields of DebuggerRunParameters | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616659,3 | Debugger: Transform DebuggerRunParameters::toolChainAbi | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616663,3 | Debugger: Transform a few fields of DebuggerRunParameters | master | qt-creator/qt-creator | Status: MERGED | +2 | +1 |
616664,3 | Debugger: Transform a few fields of DebuggerRunParameters | master | qt-creator/qt-creator | Status: MERGED | +2 | +1 |
616673,3 | Debugger: Transform a few fields of DebuggerRunParameters | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616681,3 | Debugger: Transform a few fields of DebuggerRunParameters | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616685,3 | Debugger: Transform a few fields of DebuggerRunParameters | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616686,3 | Debugger: Hide DebuggerRunParameters::m_validationErrors | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616687,3 | Debugger: Transform a few fields of DebuggerRunParameters | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616691,3 | Debugger: Transform remaining fields of DebuggerRunParameters | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
616947,7 | RunWorker: Drop device() indirection | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
617764,3 | RunWorker: Get rid of essential property | 16.0 | qt-creator/qt-creator | Status: MERGED | +2 | +1 |
626017,24 | ApplicationManager: Inline AppManagerDebugSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
626055,26 | Ios: Inline IosDebugSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
626060,23 | Android: Inline AndroidDebugSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
626067,26 | BareMetal: Inline BareMetalDebugSupport | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
626069,27 | Debugger: Make DebuggerRunTool final | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
626138,6 | ProjectExplorer: Introduce RecipeRunner | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
626143,4 | QNX: Dismantle Slog2InfoRunner, employ RecipeRunner instead | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
626368,6 | Debugger: Transform unpacking core file into recipe | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
626373,9 | Debugger: Introduce GlueInterface | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
626410,5 | Debugger: Transform terminal process into recipe | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
626443,4 | Debugger: Transform continueAfterTerminalStart() into recipe | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
626454,3 | Debugger: Transform debug server process into recipe | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
628095,7 | Android: Use ports gatherer for debugger | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
628838,4 | Debugger: Use postMessage instead of appendMessage | master | qt-creator/qt-creator | Status: MERGED | +2 | +1 |
628840,3 | ProcessRunner: Use postMessage instead of appendMessage | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
628844,7 | RunWorker: Use postMessage instead of appendMessage | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |
628845,2 | Android: Inline private slots | master | qt-creator/qt-creator | Status: MERGED | +2 | 0 |