Ubuntu One storage protocol

Merge lp:~dobey/ubuntuone-storage-protocol/no-cpp-tests into lp:ubuntuone-storage-protocol

  1. no-cpp-tests
  2. Merge into trunk
Proposed by dobey
Status: Merged
Approved by: dobey
Approved revision: 163
Merged at revision: 163
Proposed branch: lp:~dobey/ubuntuone-storage-protocol/no-cpp-tests
Merge into: lp:ubuntuone-storage-protocol
Diff against target: 21 lines (+5/-1)
1 file modified
run-tests (+5/-1)
To merge this branch: bzr merge lp:~dobey/ubuntuone-storage-protocol/no-cpp-tests
Reviewer Review Type Date Requested Status
Roberto Alsina (community) Approve
Review via email: mp+207996@code.launchpad.net

Commit message

Don't run tests with cpp protobuf module on newer protobuf versions.

To post a comment you must log in.
Revision history for this message
Roberto Alsina (ralsina) wrote :

Looks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'run-tests'
2--- run-tests 2012-12-03 19:45:43 +0000
3+++ run-tests 2014-02-24 18:46:25 +0000
4@@ -17,12 +17,16 @@
5
6 SYSNAME=`uname -s`
7
8+PROTOC_VERSION_AS_INT=`protoc --version | cut -d' ' -f2 | awk -F. '{ print $ 1 * 1000 + $2 * 100 + $ 3; }'`
9+
10 /usr/bin/env python setup.py build
11 # run the tests with pure python protobuf
12 PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python u1trial tests
13 if [ "$SYSNAME" != "Darwin" ]; then
14 # and with the cpp extension, for server (linux) only:
15- PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=cpp u1trial tests
16+ if [ $PROTOC_VERSION_AS_INT -lt 2500 ]; then
17+ PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=cpp u1trial tests
18+ fi
19 fi
20
21 USE_PYFLAKES=1 u1lint

Subscribers

People subscribed via source and target branches