Merge lp:~zeller-benjamin/qtcreator-plugin-ubuntu/more-image-info into lp:qtcreator-plugin-ubuntu

Proposed by Benjamin Zeller
Status: Merged
Approved by: Zoltan Balogh
Approved revision: 465
Merged at revision: 461
Proposed branch: lp:~zeller-benjamin/qtcreator-plugin-ubuntu/more-image-info
Merge into: lp:qtcreator-plugin-ubuntu
Diff against target: 54 lines (+9/-4)
3 files modified
src/ubuntu/ubuntuclicktool.cpp (+5/-1)
src/ubuntu/ubuntuclicktool.h (+1/-1)
src/ubuntu/wizards/createtargetwizard.cpp (+3/-2)
To merge this branch: bzr merge lp:~zeller-benjamin/qtcreator-plugin-ubuntu/more-image-info
Reviewer Review Type Date Requested Status
Zoltan Balogh Approve
ubuntu-sdk-build-bot continuous-integration Needs Fixing
Review via email: mp+299780@code.launchpad.net

Commit message

Show more informations about the images

Description of the change

Show more informations about the images

To post a comment you must log in.
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Zoltan Balogh (bzoltan) wrote :

OK

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/ubuntu/ubuntuclicktool.cpp'
--- src/ubuntu/ubuntuclicktool.cpp 2016-07-06 09:18:16 +0000
+++ src/ubuntu/ubuntuclicktool.cpp 2016-07-12 08:50:01 +0000
@@ -267,7 +267,7 @@
267 return in.readAll().trimmed();267 return in.readAll().trimmed();
268}268}
269269
270bool UbuntuClickTool::parseContainerName(const QString &name, UbuntuClickTool::Target *target)270bool UbuntuClickTool::parseContainerName(const QString &name, UbuntuClickTool::Target *target, QStringList *allExt)
271{271{
272 QStringList ext;272 QStringList ext;
273 target->framework = UbuntuClickFrameworkProvider::getBaseFramework(name, &ext);273 target->framework = UbuntuClickFrameworkProvider::getBaseFramework(name, &ext);
@@ -279,6 +279,10 @@
279 if (ext.isEmpty() || ext.size() != 3)279 if (ext.isEmpty() || ext.size() != 3)
280 return false;280 return false;
281 target->architecture = ext[1];281 target->architecture = ext[1];
282
283 if (allExt)
284 *allExt = ext;
285
282 return true;286 return true;
283}287}
284288
285289
=== modified file 'src/ubuntu/ubuntuclicktool.h'
--- src/ubuntu/ubuntuclicktool.h 2016-07-06 09:18:16 +0000
+++ src/ubuntu/ubuntuclicktool.h 2016-07-12 08:50:01 +0000
@@ -77,7 +77,7 @@
77 static void openChrootTerminal (const Target& target);77 static void openChrootTerminal (const Target& target);
7878
79 static QString targetBasePath (const Target& target);79 static QString targetBasePath (const Target& target);
80 static bool parseContainerName (const QString &name, Target *target);80 static bool parseContainerName (const QString &name, Target *target, QStringList *allExt = 0);
81 static bool getTargetFromUser (Target* target, const QString &framework=QString());81 static bool getTargetFromUser (Target* target, const QString &framework=QString());
82 static QStringList getSupportedFrameworks (const Target *target);82 static QStringList getSupportedFrameworks (const Target *target);
83 static QString getMostRecentFramework ( const QString &subFramework, const Target *target );83 static QString getMostRecentFramework ( const QString &subFramework, const Target *target );
8484
=== modified file 'src/ubuntu/wizards/createtargetwizard.cpp'
--- src/ubuntu/wizards/createtargetwizard.cpp 2016-06-14 15:23:46 +0000
+++ src/ubuntu/wizards/createtargetwizard.cpp 2016-07-12 08:50:01 +0000
@@ -323,11 +323,12 @@
323 continue;323 continue;
324324
325 UbuntuClickTool::Target target;325 UbuntuClickTool::Target target;
326 if (!UbuntuClickTool::parseContainerName(alias, &target))326 QStringList extensions;
327 if (!UbuntuClickTool::parseContainerName(alias, &target, &extensions))
327 continue;328 continue;
328329
329 QTreeWidgetItem *item = new QTreeWidgetItem;330 QTreeWidgetItem *item = new QTreeWidgetItem;
330 item->setText(0,target.framework);331 item->setText(0,target.framework+QStringLiteral("-")+extensions.last());
331 item->setData(0, Qt::UserRole, m.value(QStringLiteral("fingerprint"), QStringLiteral("error")));332 item->setData(0, Qt::UserRole, m.value(QStringLiteral("fingerprint"), QStringLiteral("error")));
332 item->setData(0, Qt::UserRole+1, alias);333 item->setData(0, Qt::UserRole+1, alias);
333334

Subscribers

People subscribed via source and target branches