Merge lp://staging/~unity-api-team/hud/reenable-valgrind into lp://staging/hud/15.04

Proposed by Ted Gould
Status: Superseded
Proposed branch: lp://staging/~unity-api-team/hud/reenable-valgrind
Merge into: lp://staging/hud/15.04
Prerequisite: lp://staging/~ted/hud/filter-underscores
Diff against target: 1391 lines (+511/-186)
21 files modified
CMakeLists.txt (+0/-1)
cmake/FindValgrind.cmake (+12/-6)
cmake/GtkDocScanGObjWrapper.cmake (+2/-0)
debian/changelog (+20/-0)
debian/control (+0/-1)
debian/rules (+4/-4)
libhud-client/CMakeLists.txt (+3/-0)
libhud-client/connection.c (+4/-0)
libhud-client/libhud-client.map (+15/-0)
libhud-client/param.c (+1/-0)
libhud/CMakeLists.txt (+3/-0)
libhud/libhud.map (+6/-0)
libqtgmenu/internal/QtGActionGroup.cpp (+1/-3)
libqtgmenu/internal/QtGMenuUtils.cpp (+1/-1)
tests/CMakeLists.txt (+2/-2)
tests/data/valgrind.suppression (+308/-33)
tests/testutils/MockHudService.cpp (+3/-2)
tests/unit/libhud-client/TestQuery.cpp (+43/-44)
tests/unit/libhud/TestActionPublisher.cpp (+52/-56)
tests/unit/qtgmenu/TestQtGMenu.cpp (+31/-30)
window-stack-bridge/WindowStack.h (+0/-3)
To merge this branch: bzr merge lp://staging/~unity-api-team/hud/reenable-valgrind
Reviewer Review Type Date Requested Status
Ted Gould (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Review via email: mp+240498@code.staging.launchpad.net

This proposal supersedes a proposal from 2014-11-03.

This proposal has been superseded by a proposal from 2015-08-19.

Commit message

Re-enable valgrind. Don't run valgrind on slow arches. Remove remaining traces of platform API.

Description of the change

Re-enable valgrind. Don't run valgrind on slow arches. Remove remaining traces of platform API.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
Ted Gould (ted) wrote : Posted in a previous version of this proposal

I disagree with Jenkins! ;-)

review: Approve
Revision history for this message
Ted Gould (ted) : Posted in a previous version of this proposal
review: Approve
Revision history for this message
Ted Gould (ted) wrote :

Resubmitting to try and see if we can get Jenkins happy with the MR.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ted Gould (ted) wrote :

Uhg, jenkins doesn't actually land dependent branches. Just doing it in CI Train.

review: Approve
Revision history for this message
Pete Woods (pete-woods) wrote :

Thanks for pushing this, Ted!

398. By Pete Woods

Sync with distro

415. By Pete Woods

Sync with distro

416. By Pete Woods

Give up on running tests under valgrind

Unmerged revisions

416. By Pete Woods

Give up on running tests under valgrind

415. By Pete Woods

Sync with distro

414. By Ted Gould

Some C++-isms

413. By Ted Gould

Lib hud client filtering

412. By Ted Gould

Filtering hud symbols

411. By Pete Woods

Remove gcov symbols again. Jenkins won't like it!

410. By Pete Woods

Another assert for list size

409. By Pete Woods

Fix mock HUD

408. By Pete Woods

Turn size comparisons into asserts

407. By Pete Woods

Assert about array lengths to avoid invalid memory access

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches