[ 
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.

Reply via email to