aboutsummaryrefslogtreecommitdiffstats
path: root/src/plugins/projectexplorer/applicationlauncher.h
Commit message (Collapse)AuthorAgeFilesLines
* ProjectExplorer: Move ApplicationLauncher to runcontrol.cpphjk2022-05-231-76/+0
| | | | | | | | It's the only remaining use. Change-Id: Id631981945b4d1850ea880a583d99dbb1e3cf25c Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* ProjectExplorer: Remove ApplicationLauncher::setProcessChannelModehjk2022-05-091-1/+0
| | | | | | | Not used anymore. Change-Id: I2b5864cf321f0fed69ae6d8586e78a6017a42d1b Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* ProjectExplorer: Consolidate ApplicationLauncher error accesshjk2022-05-091-6/+6
| | | | | | Change-Id: I2f6dd54685c2c148f78f9853fd1d93a311eb16bb Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* ProjectExplorer: React to QtcProcess::done in ApplicationLauncherhjk2022-05-091-2/+2
| | | | | | | | The 'modern' way. Change-Id: I80c0bb0b9c7c08e6aef724d430a8dbbd4b0c2777 Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* ProjectExplorer: Remove now unused ApplicationLauncher membershjk2022-05-041-3/+0
| | | | | Change-Id: I54530571114b8dbc13e50432b8331427e845215b Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* ApplicationLauncher: CleanupJarek Kobus2022-04-251-2/+0
| | | | | | | | | | | Get rid of public msgWinCannotRetrieveDebuggingOutput() method, as it was used interally in Windows case only. Get rid of m_listeningPid field, as that's the same what applicationPID() returns. Change-Id: I8bfac790f0d363e6f7cc12012597c7b8aaad6d92 Reviewed-by: hjk <hjk@qt.io>
* IDevice: Make it possible to forward declare Ptr and ConstPtrJarek Kobus2022-04-141-2/+0
| | | | | | | | | | | | | | | | | | | | | | | Most inclusions of idevice.h inside other headers are required just because of IDevice::Ptr or IDevice::ConstPtr was used there. Unfortunately, since these "using" are defined inside a IDevice class, we just can't forward declare IDevice::Ptr nor IDevice::ConstPtr. Instead, create a separate header that defines IDevicePtr and IDeviceConstPtr while having IDevice forward declared. Redefine IDevice::Ptr/ConstPtr inside IDevice to use IDevicePtr/IDeviceConstPtr. Now, instead of forward declaring a IDevice::Ptr it's sufficient to include the idevicefwd.h. This drops the number of files being recompiled after touching idevice.h from ~770 to ~210. Change-Id: Ib6f2982aa0761fb4cd6593badb9c3c0c527ea535 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: hjk <hjk@qt.io>
* ApplicationLauncher: Unify APIJarek Kobus2022-03-021-3/+3
| | | | | | | | | | | | Make the API more similar to QtcProcess API. Rename: processStarted() -> started() error() -> errorOccurred() processError() -> error() Change-Id: I86d650d55a643f5999fc91e56b985f699e2d7245 Reviewed-by: hjk <hjk@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* ApplicationLauncher: Remove start(IDevice *) overloadJarek Kobus2022-02-231-1/+0
| | | | | | | | | | | | Take the device from runnable passed in setRunnable(). If the intention is to run locally, set the device to nullptr inside runnable. Otherwise we don't run locally. Follows 47957de2dcef70b644290c0d6b35f07837160e8a Change-Id: I5b381bb499cf76e469c844ac7474ce2f60761cef Reviewed-by: hjk <hjk@qt.io> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
* ProjectExplorer: Use simpler signature for ApplicationLauncher::processExited()hjk2022-02-151-1/+4
| | | | | | | | | And rename it to finished(). Maps better to what QtcProcess uses. Change-Id: Ibfa018549f436b27638a791c0b4937c4459c9452 Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* ProjectExplorer: Split ApplicationLauncher::start() parametershjk2022-02-151-2/+4
| | | | | | | | | There's traditionally an odd duplication of the runnable.device and the passed device here. Start disentangling things. Change-Id: I1cc1628c99cea04d761fc4d8dd0cb232127ce055 Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* ApplicationLauncher: remove unused bringToForeground()Jarek Kobus2021-11-111-1/+0
| | | | | | | | Surprisingly, it was also unimplemented. Change-Id: I7845d321fd11c34cf70c944000657d9f782f0a5e Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: hjk <hjk@qt.io>
* Uniform ApplicationLauncher API: get rid of finished()Jarek Kobus2021-11-101-3/+1
| | | | | | | Use existing processExited() signal instead. Change-Id: I1eecaa05fa8b3a97d8585fca4ca89e8e1acecec3 Reviewed-by: hjk <hjk@qt.io>
* Uniform ApplicationLauncher API: get rid of reportError()Jarek Kobus2021-11-101-1/+0
| | | | | | | | Use existing error() signal instead. Implement errorString() and processError() in non-local case. Change-Id: Ibdd6cec19ffa5efa0dad330515988da80e86e35b Reviewed-by: hjk <hjk@qt.io>
* Uniform ApplicationLauncher API: get rid of remoteProcessStarted()Jarek Kobus2021-11-101-2/+1
| | | | | | | | Use existing processStarted() signal instead. Replace isRemoteRunning() with isLocal(). Change-Id: I58d0204888116c00e793fa43f969707013db06eb Reviewed-by: hjk <hjk@qt.io>
* Uniform ApplicationLauncher API: get rid of remoteStderr/out()Jarek Kobus2021-11-091-2/+0
| | | | | | | | | | Use existing appendMessage() for this purpose either with StdErrFormat for remoteStderr() or with StdOutFormat for remoteStdout(). In case when device process is used in ApplicationLauncher no appendMessage() was emitted so far with StdErrFormat or StdOutFormat. Change-Id: I2f6603aaf28113fea2a8bb6bd1738320cc39be75 Reviewed-by: hjk <hjk@qt.io>
* Uniform ApplicationLauncher API: get rid of reportProgress()Jarek Kobus2021-11-091-1/+0
| | | | | | | | | Use existing appendMessage() for this purpose with NormalMessageFormat. In case when device process is used in ApplicationLauncher no appendMessage() was emitted so far. Change-Id: I96041ad19fe16066ea31d92f52253e0aa864f96d Reviewed-by: hjk <hjk@qt.io>
* ProjectExplorer/Debugger: Add a 'Run as root' option for Unix hostshjk2021-02-221-0/+1
| | | | | | | | | | For local run and GDB debug, with or without terminal. Task-number: QTCREATORBUG-2831 Task-number: QTCREATORBUG-25330 Change-Id: I9b5d2156bcffea4f358474ecdbcad580a4419917 Reviewed-by: Christian Stenger <christian.stenger@qt.io> Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* ProjectExplorer: Modernize even moreTobias Hunger2018-07-241-1/+3
| | | | | | | | Use unique_ptr for all *Private classes, except for those in singletons. Change-Id: Ib56c31ddedc6e9cf321f15de1f1e697a27ad4089 Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
* Remove ApplicationLauncher::Modehjk2018-06-051-5/+1
| | | | | | | | | | | | | | | | | | The actual remaining use was to pop up a terminal in some setups where Mode == Console, with a default of Gui meaning "no console". In some downstream uses it was used set to Console (probably to mean "this helper process does not need a gui") but then luckily ignored when actually starting the helper processes. All cases where the console is useful and requested are nowadays RunWorkers belonging to RunConfigurations with a TerminalAspect, so they can directly get the relevant bit from their RunConfiguration without having it part of all StandardRunnables. Change-Id: I1368d5968da5cf672656aebf200ccac8d45335d0 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* ProjectExplorer: Do not add too many newlinesTobias Hunger2017-11-241-1/+1
| | | | | | | | Do not add newlines when output gets flushed. Task-number: QTCREATORBUG-17403 Change-Id: I0dea3a15611fcde8aa8cbc5fbc6b7f22c9c40dd7 Reviewed-by: hjk <hjk@qt.io>
* Add isRemoteRunning() as isRunning() only apply to local processesFilipe Azevedo2017-11-161-0/+1
| | | | | Change-Id: I75d726cea160c557374051db3ba413d3dd1a41a2 Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
* Make ApplicationLauncher report QStrings, not QByteArrayhjk2017-07-141-2/+2
| | | | | | | | | | Realistically, we only have UTF-8/plain ASCII messages, and if not then the conversion should happen when the message enters the "creator eco system" as there's potentially information on the actual encoding present, not at some random consumer further down the chain. Change-Id: Ie21199b99255f5c4d28fce3da7db2fd6c3bcfb3a Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* ProjectExplorer: Return a ProcessHandle for AppLauncher::applicationPIDhjk2017-03-101-1/+2
| | | | | Change-Id: I00fc4dbd3a3c022006de95087b61a86bbb83faeb Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* ProjectExplorer: Merge ApplicationLauncher and DeviceApplicationRunnerhjk2017-03-101-0/+10
| | | | | | | | | | Treat ApplicationLauncher as a special case of a DeviceApplicationRunner with an implicit desktop device. As a first step, lump the two implementations together. Change-Id: Ifa3ea3f38d320023050378555e2d256e762b6683 Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* ProjectExplorer: Switch responsibilities for raising applicationshjk2017-03-081-1/+1
| | | | | | | | | | Instead of letting the launcher decide to raise something when started (and only then) leave it to the RunControl to trigger (and for now also to implement) as response to the application launcher start (or possibly now, in other circumstances). Change-Id: I0ac8f1e633981f7bf316c88e83c208765886d9a1 Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* ProjectExplorer: Hide ApplicationLauncher private functions in pimplhjk2017-03-031-16/+6
| | | | | | | In preparation of the merge with DeviceApplicationRunner. Change-Id: I455e7aba255fe760e3186225cc97ccd9214b1164 Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* ProjectExplorer: Make RunControl::bringApplicationToForeground virtualhjk2017-03-011-1/+1
| | | | | | | | | | | No practical change as the only case that is used so far are local setups on Mac. It's a conceptual change, though, giving concrete RunControl implementation the chance to act "sensibly", e.g. by raising a locally running simulator control in case of remote setups or such. Change-Id: I5679428934fe08b8c9756f3906ee3d9073278822 Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* Projectexplorer: Modernize codebaseTobias Hunger2016-04-151-1/+1
| | | | | | Change-Id: I4793b58aa77bcd46af99f8b843c0f7439cdc9269 Reviewed-by: Tim Jenssen <tim.jenssen@theqtcompany.com> Reviewed-by: Vikas Pachdha <vikas.pachdha@theqtcompany.com>
* Merge remote-tracking branch 'origin/3.6' into 4.0Eike Ziller2016-03-031-0/+1
|\ | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/plugins/projectexplorer/applicationlauncher.cpp src/plugins/projectexplorer/applicationlauncher.h src/plugins/qmlprofiler/qmlprofilerclientmanager.cpp src/plugins/qmlprofiler/qmlprofilertool.cpp src/plugins/qtsupport/uicodemodelsupport.cpp src/plugins/qtsupport/uicodemodelsupport.h Change-Id: I6f6ae77422d99f4f422b108ade6b64273df9dd57
| * ApplicationLaucher: Catch late WinDebug messagesTobias Hunger2016-02-291-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | Catch and display messages received from WinDebug interface, even for a short while after the process has already finished. This can apparently happen, e.g. when doing a qDebug right before the application ends (and QT_LOGGING_TO_CONSOLE is not 1, the application is being run in GUI mode, on windows). Task-number: QTCREATORBUG-15546 Change-Id: I37a015da89fc409234adeb597171fa328f50c14d Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
* | ProjectExplorer: ModernizeTobias Hunger2016-02-031-5/+2
| | | | | | | | | | | | | | | | | | * Use override where appropriate * Use pragma once * Make more constructors explicit Change-Id: I2865fe10f288e3de570826058e43b70a0cb4ee37 Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* | ProjectExplorer: Use Qt5-style connectsOrgad Shaneh2016-02-011-2/+1
| | | | | | | | | | | | | | The heavy lifting was done by clazy. Change-Id: I619db09a79760186b72e7662490ed1205155c1a7 Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>
* | ProjectExplorer: Use StandardRunnable in ApplicationLauncherhjk2016-01-281-9/+3
| | | | | | | | | | Change-Id: I7092d748207762d3dbd6c69c01cc06c88cbf63d8 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Update License according to agreement with Free Qt FoundationTobias Hunger2016-01-191-17/+12
|/ | | | | | | * Update files in src/plugins Change-Id: Ia5d77fad7d19d4bb3498e78661982f68729adb22 Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>
* Merge commit '3c85058694ee2e41658d17f524fb48f0b187d2fe'Eike Ziller2015-02-121-6/+6
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/libs/utils/tooltip/tipcontents.cpp src/libs/utils/tooltip/tipcontents.h src/plugins/android/androiddeployqtstep.cpp src/plugins/baremetal/baremetalconstants.h src/plugins/baremetal/baremetaldevice.cpp src/plugins/baremetal/baremetaldevice.h src/plugins/baremetal/baremetaldeviceconfigurationwidget.cpp src/plugins/baremetal/baremetaldeviceconfigurationwidget.h src/plugins/baremetal/baremetaldeviceconfigurationwizard.cpp src/plugins/baremetal/baremetaldeviceconfigurationwizardpages.cpp src/plugins/baremetal/baremetaldeviceconfigurationwizardpages.h src/plugins/baremetal/baremetalplugin.cpp src/plugins/baremetal/baremetalplugin.h src/plugins/baremetal/baremetalruncontrolfactory.cpp src/plugins/baremetal/baremetalruncontrolfactory.h src/plugins/cppeditor/cppcodemodelinspectordialog.cpp src/plugins/cppeditor/cppdoxygen_test.cpp src/plugins/cppeditor/cppdoxygen_test.h src/plugins/debugger/breakpointmarker.cpp src/plugins/debugger/debuggeritemmodel.cpp src/plugins/debugger/debuggeritemmodel.h src/plugins/debugger/loadcoredialog.cpp src/plugins/genericprojectmanager/cppmodelmanagerhelper.cpp src/plugins/projectexplorer/addnewmodel.cpp src/plugins/projectexplorer/addnewmodel.h src/plugins/projectexplorer/jsonwizard/jsonfieldpage.cpp src/plugins/qmlprofiler/abstracttimelinemodel.cpp src/plugins/qmlprofiler/abstracttimelinemodel.h src/plugins/qmlprofiler/notesmodel.cpp src/plugins/qmlprofiler/qml/CategoryLabel.qml src/plugins/qmlprofiler/qml/MainView.qml src/plugins/qmlprofiler/qml/Overview.js src/plugins/qmlprofiler/qml/Overview.qml src/plugins/qmlprofiler/qml/TimeDisplay.qml src/plugins/qmlprofiler/qml/TimeMarks.qml src/plugins/qmlprofiler/qmlprofilertimelinemodelproxy.cpp src/plugins/qmlprofiler/sortedtimelinemodel.cpp src/plugins/qmlprofiler/sortedtimelinemodel.h src/plugins/qmlprofiler/timelinemodelaggregator.cpp src/plugins/qmlprofiler/timelinemodelaggregator.h src/plugins/qmlprofiler/timelinerenderer.cpp src/plugins/qmlprofiler/timelinerenderer.h src/plugins/qmlprojectmanager/QmlProjectManager.json.in src/plugins/texteditor/findinfiles.cpp src/plugins/vcsbase/vcsconfigurationpage.cpp src/shared/qbs src/shared/scriptwrapper/interface_wrap_helpers.h src/shared/scriptwrapper/wrap_helpers.h tests/auto/qmlprofiler/abstracttimelinemodel/tst_abstracttimelinemodel.cpp tests/system/suite_debugger/tst_debug_empty_main/test.py tests/system/suite_debugger/tst_qml_js_console/test.py tests/system/suite_debugger/tst_qml_locals/test.py Change-Id: I67540b648f8b162496f4aa606b04d50c7c9125c6
| * Update LicenseEike Ziller2015-01-161-6/+6
| | | | | | | | | | Change-Id: I711d5fb475ef814a1dc9d2822740e827f3f67125 Reviewed-by: Alessandro Portale <alessandro.portale@digia.com>
* | ProjectExplorer/Valgrind: Disambiguate signal nameOrgad Shaneh2015-02-091-1/+1
| | | | | | | | | | | | | | | | | | error => processError To make it easier to use Qt5-style connect Change-Id: Idfa987512ebb7d1c1c02d3cd01b2d05088cbf38a Reviewed-by: hjk <hjk@theqtcompany.com>
* | Retrieve output from journaldTobias Hunger2015-01-281-1/+1
|/ | | | | | | | | Qt nowadays logs into journald in some setups, so retrieve the output from there and put it into the application output pane. Change-Id: Ia2199a1420e1d80541f2f455e242d0b473922125 Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com> Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>
* License updateEike Ziller2014-10-091-7/+8
| | | | | Change-Id: I3c22ef2685d7aa589f5d0ab74d693653a4c32082 Reviewed-by: Alessandro Portale <alessandro.portale@digia.com>
* ProjectExplorer: Extend ApplicationLauncherOrgad Shaneh2014-05-071-0/+7
| | | | | | | Implement some more QProcess functions Change-Id: I1afe3fa5f5b353508cf300178cb50507b3dd4b3e Reviewed-by: Daniel Teske <daniel.teske@digia.com>
* Clean up single namespace forward-declarationsOrgad Shaneh2014-02-121-3/+1
| | | | | | | | | | | | | | | | | | Done using the following ruby script: Dir.glob('**/*.h').each { |file| if File.file?(file) s = File.read(file) t = s.gsub(/^namespace .+ \{\n\s*class .*;\n\s*\}.*$/) { |m| m.gsub(/\n\s*/, ' ').gsub(/\s*\/\/.*$/, '') } if t != s puts file File.open(file, 'w').write(t) end end } Change-Id: Iffcb966e90eb8e1a625eccd5dd0b94f000ae368e Reviewed-by: hjk <hjk121@nokiamail.com>
* Incremented year in copyright infoRobert Loehning2014-01-081-1/+1
| | | | | | Change-Id: Ib5423fdd064e4546f848c0b640b0ed0514c26d3a Reviewed-by: Leena Miettinen <riitta-leena.miettinen@digia.com> Reviewed-by: Kai Koehne <kai.koehne@digia.com>
* ApplicationLauncher: Also emit the exit statusDaniel Teske2013-08-021-1/+1
| | | | | | | | | And adjust the message in the appliation output to take the exit status into account. Change-Id: I1b7507fdc8ff6fa7ec3db48dba72ad723f124fc3 Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com> Reviewed-by: Daniel Teske <daniel.teske@digia.com>
* ConsoleProcess: Show correct exit codeDaniel Teske2013-08-011-1/+0
| | | | | | | | Task-number: QTCREATORBUG-9740 Change-Id: I41721356b3612b6c98e774168f520ff49426271b Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com> Reviewed-by: Daniel Teske <daniel.teske@digia.com>
* Incremented year in copyright infov2.6.2Robert Loehning2013-01-291-1/+1
| | | | | Change-Id: Ic6a9ff0359625021ebc061d22db6811814534205 Reviewed-by: Kai Koehne <kai.koehne@digia.com>
* Adjust license headershjk2012-10-051-21/+20
| | | | | Change-Id: Ice592c6de9951ee3b2c4cb52ed0bb3b6770e0825 Reviewed-by: Eike Ziller <eike.ziller@digia.com>
* Contact -> qt-project.orgEike Ziller2012-07-191-3/+1
| | | | | Change-Id: I7134d7de30bcf9f9dcfad42520dd45ee083a852d Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
* Removed module names from #include directives.Erik Verbruggen2012-02-151-1/+1
| | | | | | | | Getting the #include directives ready for Qt5. This includes the new-project wizards. Change-Id: Ia9261f1e8faec06b9285b694d2b7e9a095978d2b Reviewed-by: Eike Ziller <eike.ziller@nokia.com>
* Long live the king!hjk2012-01-261-1/+1
| | | | | Change-Id: I2b72b34c0cfeafc8bdbaf49b83ff723544f2b6e2 Reviewed-by: Daniel Teske <daniel.teske@nokia.com>