Hi Joel, thanks for the mail. I just ran a quick test and compilation works for me (g++ 4.6.3). Which compiler version are you using? But i'm going to change the VERSION symbol to avoid the conflict.
I'll also update the documentation on the webpage. An up-to-date file is here: https://github.com/cruppstahl/hypertable/tree/v0.9.6/src/cc/ThriftBroker/tests Bye Christoph 2012/9/19 Joel Nelson <[email protected]> > Hi, > > There seems to be a naming conflict between the Thrift and Hypertable > headers. > > If cwd is /opt/hypertable/0.9.6.3, there here are the conflicting > definitions: > > ./include/thrift/config.h:324:#define VERSION "0.8.0" > ./include/ThriftBroker/SerializedCellsFlag.h:39: VERSION = 0x01 > > The result of building something > (e.g. http://hypertable.com/documentation/code_examples/cpp/) is: > > /opt/hypertable/0.9.6.3/include/ThriftBroker/SerializedCellsFlag.h:39:7: > error: expected identifier before string constant > /opt/hypertable/0.9.6.3/include/ThriftBroker/SerializedCellsFlag.h:39:7: > error: expected ‘}’ before string constant > /opt/hypertable/0.9.6.3/include/ThriftBroker/SerializedCellsFlag.h:39:7: > error: expected unqualified-id before string constant > /opt/hypertable/0.9.6.3/include/ThriftBroker/SerializedCellsFlag.h:44:1: > error: expected declaration before ‘}’ token > make: *** [client_test.o] Error 1 > > I am using the latest release 0.9.6.3, but I think this has existed > previously. > > In my code I just changed SerializedCellsVersion::VERSION to > SerializedCellsVersion::SVERSION and seemed to work ok. > > Separate from that, in the C++ example > http://hypertable.com/documentation/code_examples/cpp > it seems there are several typos that make it unable to compile as is. > > E.g. line 182: > std::vector<hypertable::thriftgen::cell> cells; > needs to be > std::vector<Hypertable::ThriftGen::Cell> cells; > > There are several other lines like that. If you want I'll send through my > cleaned up > for compile version, but I used things like the C++11 range-based for > instead of > your foreach macro so not sure you want it. > > Thanks > > Joel > > > > -- > You received this message because you are subscribed to the Google Groups > "Hypertable Development" group. > To view this discussion on the web visit > https://groups.google.com/d/msg/hypertable-dev/-/kHFzRwjojFcJ. > To post to this group, send email to [email protected]. > To unsubscribe from this group, send email to > [email protected]. > For more options, visit this group at > http://groups.google.com/group/hypertable-dev?hl=en. > -- You received this message because you are subscribed to the Google Groups "Hypertable Development" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/hypertable-dev?hl=en.
