Unity fake scopes

Merge lp://staging/~stolowski/unity-scope-fake/no-jsoncpp into lp://staging/unity-scope-fake

  1. no-jsoncpp
  2. Merge into trunk
Proposed by Paweł Stołowski
Status: Merged
Approved by: Marcus Tomlinson
Approved revision: 61
Merged at revision: 60
Proposed branch: lp://staging/~stolowski/unity-scope-fake/no-jsoncpp
Merge into: lp://staging/unity-scope-fake
Diff against target: 229 lines (+47/-87)
4 files modified
CMakeLists.txt (+0/-2)
debian/control (+0/-1)
src/CMakeLists.txt (+0/-1)
src/scope.cpp (+47/-83)
To merge this branch: bzr merge lp://staging/~stolowski/unity-scope-fake/no-jsoncpp
Reviewer Review Type Date Requested Status
Marcus Tomlinson (community) Approve
Review via email: mp+297297@code.staging.launchpad.net

Commit message

Drop jsoncpp, use Variant - JSON (de)serialization from scopes API.

Description of the change

Drop jsoncpp, use Variant - JSON (de)serialization from scopes API.

To post a comment you must log in.
Revision history for this message
Marcus Tomlinson (marcustomlinson) wrote :

While scanning through the project I noticed some very small issues here and there. When you have time: https://code.launchpad.net/~marcustomlinson/unity-scope-fake/small-fixes/+merge/297428

Otherwise, this MP looks good! +1

review: Approve

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

to all changes: