Yes, you are absolutely right. Unless we don't "install" protocol.h at all and try to remove it from the hedwig public headers. I remember the protocol.h stuff was causing you problems recently so this could resolve that issue too.

Ivan Kelly
added a comment - 13/Sep/10 09:23 Yes, you are absolutely right. Unless we don't "install" protocol.h at all and try to remove it from the hedwig public headers. I remember the protocol.h stuff was causing you problems recently so this could resolve that issue too.

Ivan, the changes look good to me. One question though, do we need to include the auto-generated protocol.h header to the Makefile.am? I had to do that step manually a couple of weeks ago otherwise things didn't work when using the hedwig client libs generated.

Erwin Tam
added a comment - 09/Sep/10 22:06 Ivan, the changes look good to me. One question though, do we need to include the auto-generated protocol.h header to the Makefile.am? I had to do that step manually a couple of weeks ago otherwise things didn't work when using the hedwig client libs generated.

Michi Mutsuzaki
added a comment - 04/Sep/10 08:17 Here are my comments:
After applying the patch, I was able to compile cpp client.
I still see some compilation warnings. I'll attach the output.
Thanks!
--Michi