Merge lp://staging/~clint-fewbar/ubuntu/precise/zookeeper/run-unit-tests into lp://staging/ubuntu/precise/zookeeper
Proposed by
Clint Byrum
| Status: | Merged |
|---|---|
| Merge reported by: | James Page |
| Merged at revision: | not available |
| Proposed branch: | lp://staging/~clint-fewbar/ubuntu/precise/zookeeper/run-unit-tests |
| Merge into: | lp://staging/ubuntu/precise/zookeeper |
| Diff against target: |
400 lines (+340/-1) 7 files modified
.pc/applied-patches (+1/-0) .pc/fix-broken-c-client-unittest.patch/src/c/tests/TestZookeeperInit.cc (+301/-0) debian/changelog (+9/-0) debian/patches/fix-broken-c-client-unittest.patch (+20/-0) debian/patches/series (+1/-0) debian/rules (+7/-0) src/c/tests/TestZookeeperInit.cc (+1/-1) |
| To merge this branch: | bzr merge lp://staging/~clint-fewbar/ubuntu/precise/zookeeper/run-unit-tests |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| James Page | Approve | ||
|
Review via email:
|
|||
Description of the change
Runs the unit tests that make sense (zkpython's is a bit broken for automated running) on package build. This is to support the ZK MIR.
To post a comment you must log in.
Revision history for this message
| Clint Byrum (clint-fewbar) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Heh, and now I just saw where you've already added more elaborate code to do this into debian/rules, but commented out.
The C patch is necessary, but it looks like maybe your code has some other things that I hadn't thought of.