Merge lp://staging/~thisfred/u1db/dont-install-cython-wrapper into lp://staging/u1db

Proposed by Eric Casteleijn
Status: Merged
Approved by: Eric Casteleijn
Approved revision: 397
Merged at revision: 396
Proposed branch: lp://staging/~thisfred/u1db/dont-install-cython-wrapper
Merge into: lp://staging/u1db
Diff against target: 120 lines (+41/-30)
2 files modified
CMakeLists.txt (+15/-6)
setup.py (+26/-24)
To merge this branch: bzr merge lp://staging/~thisfred/u1db/dont-install-cython-wrapper
Reviewer Review Type Date Requested Status
Samuele Pedroni Approve
Review via email: mp+123307@code.staging.launchpad.net

Commit message

Made setup.py not build cython wrapper by default.

Description of the change

Made setup.py not build cython wrapper by default.

To post a comment you must log in.
Revision history for this message
Samuele Pedroni (pedronis) wrote :

+1

review: Approve
Revision history for this message
Ubuntu One Auto Pilot (otto-pilot) wrote :
Download full text (3.8 KiB)

The attempt to merge lp:~thisfred/u1db/dont-install-cython-wrapper into lp:u1db failed. Below is the output from the failed tests.

-- The C compiler identification is GNU 4.7.1
-- The CXX compiler identification is GNU 4.7.1
-- Check for working C compiler: /usr/bin/gcc
-- Check for working C compiler: /usr/bin/gcc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Found Cython: /usr/bin/cython
-- Found CURL: /usr/lib/x86_64-linux-gnu/libcurl.so (found version "7.27.0")
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.26")
-- checking for module 'oauth'
-- found oauth, version 0.9.7
-- Found OAuth: /usr/lib/x86_64-linux-gnu/liboauth.so
-- checking for module 'json'
-- found json, version 0.10
-- Found JSON: /usr/lib/x86_64-linux-gnu/libjson.so
-- checking for module 'sqlite3'
-- found sqlite3, version 3.7.13
-- Found Sqlite3: /usr/lib/x86_64-linux-gnu/libsqlite3.so
-- Configuring done
-- Generating done
-- Build files have been written to: /mnt/tarmac/cache/u1db/trunk
Scanning dependencies of target ReplicatePythonSourceTree
[ 0%] Built target ReplicatePythonSourceTree
[ 11%] Generating u1db_schema.c
Scanning dependencies of target u1db
[ 22%] Building C object src/CMakeFiles/u1db.dir/mkstemp_compat.c.o
[ 33%] Building C object src/CMakeFiles/u1db.dir/u1db.c.o
[ 44%] Building C object src/CMakeFiles/u1db.dir/u1db_http_sync_target.c.o
[ 55%] Building C object src/CMakeFiles/u1db.dir/u1db_query.c.o
[ 66%] Building C object src/CMakeFiles/u1db.dir/u1db_sync_target.c.o
[ 77%] Building C object src/CMakeFiles/u1db.dir/u1db_uuid.c.o
[ 88%] Building C object src/CMakeFiles/u1db.dir/u1db_vectorclock.c.o
[100%] Building C object src/CMakeFiles/u1db.dir/u1db_schema.c.o
Linking C static library libu1db.a
[100%] Built target u1db
Scanning dependencies of target build-debug
running build_ext
[100%] Built target build-debug
Scanning dependencies of target check-valgrind
Tests running...
======================================================================
FAIL: u1db.tests.test_c_backend.TestCDatabaseExists.test_c_backend_compiled
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/mnt/tarmac/cache/u1db/trunk/u1db/tests/test_c_backend.py", line 38, in test_c_backend_compiled
    " Was it compiled properly?\n%s" % (c_backend_error,))
AssertionError: Could not import the c_backend_wrapper module. Was it compiled properly?
cannot import name c_backend_wrapper

Ran 1455 tests in 162.349s
FAILED (failures=1)

[59005 refs]
==2975== Memcheck, a memory error detector
==2975== Copyright (C) 2002-2011, and GNU GPL'd, by Julian Seward et al.
==2975== Using Valgrind-3.7.0 and LibVEX; rerun with -h for copyright info
==2975== Command: python-dbg -m testtools.run discover
==2975==
[168478 refs]
==2975==
==2975== HEAP SUMMARY:
==2975== in use at exit: 23,995,807 bytes in 9,372 blocks
==2975== total heap usage: 1,276,168 allocs, 1,266,796 frees, 632,798,209 bytes ...

Read more...

Revision history for this message
Samuele Pedroni (pedronis) wrote :

a more meditated +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