Merge lp:~abreu-alexandre/qtcreator-plugin-ubuntu/add-inspector-option-to-html5-run into lp:qtcreator-plugin-ubuntu

Proposed by Alexandre Abreu
Status: Merged
Approved by: David Barth
Approved revision: 93
Merged at revision: 95
Proposed branch: lp:~abreu-alexandre/qtcreator-plugin-ubuntu/add-inspector-option-to-html5-run
Merge into: lp:qtcreator-plugin-ubuntu
Diff against target: 52 lines (+6/-4)
3 files modified
share/qtcreator/ubuntu/menu.json (+1/-1)
share/qtcreator/ubuntu/scripts/qtc_device_run_app (+4/-2)
src/ubuntu/ubunturuncontrol.cpp (+1/-1)
To merge this branch: bzr merge lp:~abreu-alexandre/qtcreator-plugin-ubuntu/add-inspector-option-to-html5-run
Reviewer Review Type Date Requested Status
Alexandre Abreu (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+206955@code.launchpad.net

Commit message

Add inspector option for remote debugging by default when launching html5 app from qtc

Description of the change

Add inspector option for remote debugging by default when launching html5 app from qtc

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alexandre Abreu (abreu-alexandre) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'share/qtcreator/ubuntu/menu.json'
2--- share/qtcreator/ubuntu/menu.json 2014-02-12 19:22:39 +0000
3+++ share/qtcreator/ubuntu/menu.json 2014-02-18 15:12:57 +0000
4@@ -83,7 +83,7 @@
5 "ubuntuHtmlProjectRequired": true,
6 "saveRequired": true,
7 "actions": [
8- "%SCRIPTDIRECTORY%/qtc_device_run_app %SERIALNUMBER% %APPRUNNEREXECNAME% %FOLDERNAME% %DISPLAYNAME%.desktop ../%FOLDERNAME% /home/%USERNAME%/dev_tmp /home/%USERNAME%/.local/share/applications %USERNAME%@%IP% %PORT% --www="
9+ "%SCRIPTDIRECTORY%/qtc_device_run_app %SERIALNUMBER% %APPRUNNEREXECNAME% %FOLDERNAME% %DISPLAYNAME%.desktop ../%FOLDERNAME% /home/%USERNAME%/dev_tmp /home/%USERNAME%/.local/share/applications %USERNAME%@%IP% %PORT% --www= --inspector"
10 ]
11 },
12 {
13
14=== modified file 'share/qtcreator/ubuntu/scripts/qtc_device_run_app'
15--- share/qtcreator/ubuntu/scripts/qtc_device_run_app 2014-02-10 14:15:34 +0000
16+++ share/qtcreator/ubuntu/scripts/qtc_device_run_app 2014-02-18 15:12:57 +0000
17@@ -28,10 +28,12 @@
18 # potential prefix that should be prepended to the target app path
19 shift
20 APP_LAUNCH_PREFIX=$9
21+shift
22+EXTRA_COMMAND_LINE_ARGS=$9
23
24 APP_ID=$(echo ${DESKTOP_FILENAME}|sed "s/\.desktop//")
25
26-USAGE="$0 [serialnumber] [app_launcher] [foldername] [desktop_filename] [main_qml] [target_device_home] [target_device_desktop_path] [target_device] [target_device_port] [app_launch_prefix]"
27+USAGE="$0 [serialnumber] [app_launcher] [foldername] [desktop_filename] [main_qml] [target_device_home] [target_device_desktop_path] [target_device] [target_device_port] [optional_app_launch_prefix] [optional_extra_command_line_args]"
28
29 if [[ -z ${DESKTOP_FILENAME} || -z ${FOLDERNAME} ]]; then
30 echo ${USAGE}
31@@ -91,7 +93,7 @@
32 done
33 echo "IMPORT: ${ARGS_FROM_DESKTOP}"
34
35-COMMAND="export ${DBUS_SESSION_BUS_ADDRESS=}; APP_ID=${APP_ID} ${APP_LAUNCHER} ${ARGS_FROM_DESKTOP} ${APP_LAUNCH_PREFIX}${TARGET_DEVICE_HOME}/${APP_ID}/${FOLDERNAME}/${MAIN_QML} --desktop_file_hint=${TARGET_DEVICE_DESKTOP_PATH}/${DESKTOP_FILENAME}"
36+COMMAND="export ${DBUS_SESSION_BUS_ADDRESS=}; APP_ID=${APP_ID} ${APP_LAUNCHER} ${ARGS_FROM_DESKTOP} ${APP_LAUNCH_PREFIX}${TARGET_DEVICE_HOME}/${APP_ID}/${FOLDERNAME}/${MAIN_QML} --desktop_file_hint=${TARGET_DEVICE_DESKTOP_PATH}/${DESKTOP_FILENAME} ${EXTRA_COMMAND_LINE_ARGS}"
37 echo "Going to execute: ${COMMAND}"
38
39 # make a tar package
40
41=== modified file 'src/ubuntu/ubunturuncontrol.cpp'
42--- src/ubuntu/ubunturuncontrol.cpp 2013-12-11 16:28:52 +0000
43+++ src/ubuntu/ubunturuncontrol.cpp 2014-02-18 15:12:57 +0000
44@@ -35,7 +35,7 @@
45 if (ubuntuProject->mainFile().compare(QString::fromLatin1("index.html"), Qt::CaseInsensitive) == 0) {
46 //TODO move into abstracted location
47 m_executable = QString::fromLatin1(Ubuntu::Constants::UBUNTUHTML_PROJECT_LAUNCHER_EXE);
48- m_commandLineArguments = QString(QLatin1String("--www=%0")).arg(ubuntuProject->projectDirectory());
49+ m_commandLineArguments = QString(QLatin1String("--www=%0 --inspector")).arg(ubuntuProject->projectDirectory());
50 }
51 else {
52 m_executable = QtSupport::QtKitInformation::qtVersion(runConfiguration->target()->kit())->qmlsceneCommand();

Subscribers

People subscribed via source and target branches