Merge lp:~zsombi/ubuntu-ui-toolkit/fixGcc5LinkError into lp:ubuntu-ui-toolkit/staging

Proposed by Zsombor Egri
Status: Merged
Approved by: Cris Dywan
Approved revision: 2075
Merged at revision: 2075
Proposed branch: lp:~zsombi/ubuntu-ui-toolkit/fixGcc5LinkError
Merge into: lp:ubuntu-ui-toolkit/staging
Diff against target: 29 lines (+5/-3)
2 files modified
app-launch-profiler/app-launch-tracepoints.c (+2/-2)
src/Ubuntu/UbuntuMetrics/applicationmonitor.cpp (+3/-1)
To merge this branch: bzr merge lp:~zsombi/ubuntu-ui-toolkit/fixGcc5LinkError
Reviewer Review Type Date Requested Status
ubuntu-sdk-build-bot continuous-integration Approve
Zoltan Balogh Approve
Review via email: mp+303514@code.launchpad.net

Commit message

Build failures on staging.

To post a comment you must log in.
Revision history for this message
Zoltan Balogh (bzoltan) wrote :

OK

review: Approve
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'app-launch-profiler/app-launch-tracepoints.c'
2--- app-launch-profiler/app-launch-tracepoints.c 2016-08-16 08:43:28 +0000
3+++ app-launch-profiler/app-launch-tracepoints.c 2016-08-22 06:21:13 +0000
4@@ -28,8 +28,8 @@
5
6 int main (int argc, char* argv[])
7 {
8- #pragma unused(argc)
9- #pragma unused(argv)
10+ (void)argc;
11+ (void)argv;
12 tracepoint(app,invokeApplauncher);
13 return 0;
14 }
15
16=== modified file 'src/Ubuntu/UbuntuMetrics/applicationmonitor.cpp'
17--- src/Ubuntu/UbuntuMetrics/applicationmonitor.cpp 2016-08-06 00:54:30 +0000
18+++ src/Ubuntu/UbuntuMetrics/applicationmonitor.cpp 2016-08-22 06:21:13 +0000
19@@ -567,7 +567,9 @@
20 event.generic.id = id;
21 // We don't bother fixing up non null-terminated string, just potential
22 // overflows.
23- event.generic.stringSize = qMin(size, UMGenericEvent::maxStringSize);
24+ // Note: GCC5 comoiler/linker cannot find UMGenericEvent::maxStringSize
25+ // if used in qMin(); force type to satisfy it
26+ event.generic.stringSize = qMin(size, quint32(UMGenericEvent::maxStringSize));
27 memcpy(event.generic.string, string, event.generic.stringSize);
28 d->m_loggingThread->push(&event);
29 return true;

Subscribers

People subscribed via source and target branches