[
https://issues.apache.org/jira/browse/THRIFT-582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12921084#action_12921084
]
Roger Meier commented on THRIFT-582:
------------------------------------
great job!
I think there are some minor issues to be solved before commit:
* I would like to see language specific tests in their appropriate library
directory THRIFT-35 (few languages are done, its easy to do for new ones)
* all unit test should pass without error
* It seems, that at least the unit tests depend on cpp implementation, what
about adding a dependancy on cpp inside configure.ac to ensure that we have
with-cpp?
{code}
./configure --without-csharp --without-java --without-erlang --without-perl
--without-php --without-php_extension --without-ruby --without-haskell
--without-python --without-cpp --with-c_glib
.....
g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib/cpp/src -I./gen-cpp
-I../../lib/c_glib/src -I./gen-c -I/usr/include/glib-2.0
-I/usr/lib/glib-2.0/include -g -MT
testthrifttestclient-testthrifttestclient.o -MD -MP -MF
.deps/testthrifttestclient-testthrifttestclient.Tpo -c -o
testthrifttestclient-testthrifttestclient.o `test -f 'testthrifttestclient.cpp'
|| echo './'`testthrifttestclient.cpp
mv -f .deps/testthrifttestclient-testthrifttestclient.Tpo
.deps/testthrifttestclient-testthrifttestclient.Po
make[4]: *** No rule to make target `../../lib/cpp/.libs/libthrift.la', needed
by `testthrifttestclient'. Stop.
make[4]: Leaving directory
`/home/roger/software/thrift/thrift-trunk-c_glib/test/c_glib'
{code}
after configuring --with-cpp it nearly passes all unit tests.. but the
following 2 tests fail:
{code}
PASS: testdebugproto
/bin/sh: line 4: 21597 Segmentation fault ${dir}$tst
FAIL: testoptionalrequired
....
PASS: testwrapper-testdebugproto
./testwrapper-testoptionalrequired: line 7: 21888 Segmentation fault
"$stripcommand" "$@"
FAIL: testwrapper-testoptionalrequired
{code}
My Environment is: Debian GNU/Linux Lenny with
{code}
ii libglib2.0-0 2.16.6-3 The GLib
library of C routines
ii libglib2.0-dev 2.16.6-3 Development
files for the GLib library
ii gcc 4:4.3.2-2 The GNU C
compiler
ii gcc-4.3 4.3.2-1.1 The GNU C
compiler
ii gcc-4.3-base 4.3.2-1.1 The GNU
Compiler Collection (base package)
ii libtool 1.5.26-4+lenny1 Generic
library support script
{code}
> C implementation of Thrift
> --------------------------
>
> Key: THRIFT-582
> URL: https://issues.apache.org/jira/browse/THRIFT-582
> Project: Thrift
> Issue Type: New Feature
> Reporter: David Reiss
> Assignee: Michael Lum
> Priority: Minor
> Attachments: c_glib_bindings.patch, t_c_generator.cc,
> thrift_582_c_glib_bindings.patch, thrift_582_c_glib_bindings_20100927.patch,
> thrift_582_c_glib_bindings_20101012.patch
>
>
> Create a usable implementation of Thrift that uses only C at runtime, no C++.
> The code is at
> http://svn.apache.org/repos/asf/incubator/thrift/branches/c-bindings/.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.