[PCBC-155] Compile error at git:master 2f51ee9573cc93fdf343570ac5422bb99f6ce7f6 Created: 04/Dec/12  Updated: 05/Dec/12  Resolved: 04/Dec/12

Status: Resolved
Project: Couchbase PHP client library
Component/s: None
Affects Version/s: None
Fix Version/s: None
Security Level: Public

Type: Bug Priority: Major
Reporter: Jeff Minard Assignee: Trond Norbye
Resolution: Won't Fix Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   
Probably a temporary error, but I updated from git this morning to recompile and I'm getting this:

/root/sources/php-ext-couchbase/remove.c: In function 'php_couchbase_remove_callback':
/root/sources/php-ext-couchbase/remove.c:16: error: 'const struct <anonymous>' has no member named 'cas'
make: *** [remove.lo] Error 1

And the end of the make process. PHP 5.3.19, CentOS5.6. Let me know if you need any more specific system information.

 Comments   
Comment by Jeff Minard [ 04/Dec/12 ]
This commit https://github.com/couchbase/php-ext-couchbase/commit/ad692a4f28f031ff02385f1985e45f0b17a39d87 introduced the error.
Comment by Matt Ingenthron [ 04/Dec/12 ]
Thanks Jeff! We'll let you know if we need more info. If you can, please "git clean -dfx" (brutal clean) as I've had things stick around and cause problems before.

Trond: can you check this out?
Comment by Trond Norbye [ 04/Dec/12 ]
You have to upgrade libcouchbase to the latest 2.0 release
Comment by Matt Ingenthron [ 04/Dec/12 ]
Thanks Trond. Do you think we should add some checks for a 2.0 lcb and print out a sane error message?
Comment by Trond Norbye [ 04/Dec/12 ]
That's easier said than done.. this changed with some of the betas... it is going to be pretty much work to always keep that up to date (and backport fixes etc)
Comment by Jeff Minard [ 04/Dec/12 ]
> You have to upgrade libcouchbase to the latest 2.0 release

How does one go about that? I had installed libcouchbase from the rhel5x64 repo (specifically http://packages.couchbase.com/preview/rpm/5.5/x86_64) -- is the repo not going to be up-to-date enough?
Comment by Matt Ingenthron [ 04/Dec/12 ]
If it's the preview repo, then it should be up to date. Have you done a "yum update" (I think it is)? Which lcb does RPM/yum say you have installed right now?

That said, libcouchbase is actually released, so you can actually move to the release repo. When I had to do this recently, I had to clean my yum cache with "yum clean all" IIRC. in order to get the latest bits.
Comment by Jeff Minard [ 04/Dec/12 ]
Yup, updating the repo did the trick. I'll give switching over the to the release repo a try too. That would be http://packages.couchbase.com/rpm/couchbase-centos55-x86_64.repo correct?

Thanks guys!
Comment by Jeff Minard [ 04/Dec/12 ]
Updating to the real repo is failing, but not for other reasons I'll address over here: http://www.couchbase.com/issues/browse/PCBC-155
Comment by Matt Ingenthron [ 04/Dec/12 ]
Well, now I've run into this myself. It looks like something changed between beta3 and release on libcouchbase :( Same building problem Jeff had.
Comment by Trond Norbye [ 05/Dec/12 ]
That is a risk dealing with previews or betas.
Comment by Matt Ingenthron [ 05/Dec/12 ]
This is true, but we shouldn't have changed API/ABI between beta3 and GA. Apparently something changed.
Comment by Trond Norbye [ 05/Dec/12 ]
The change was necessary in order to be able to support observe on deleted items. Personally I don't think this is a big deal since it was all sorted out on for the GA. If we can't fix up missing pieces between betas and GA we'll end up with a million different APIs for things just because we can't clean up stuff..
Comment by Matt Ingenthron [ 05/Dec/12 ]
We can, but it should be exceptional and something that's communicated. It also means that we need to ship updates so we don't leave people broken. It's unfortunately not always trivial to get the older one, even if you know you need it.

We'll work through it-- we just need to be more disciplined between Beta and GA, otherwise Beta is no different than Dev Preview.
Generated at Thu Oct 02 03:11:07 CDT 2014 using JIRA 5.2.4#845-sha1:c9f4cc41abe72fb236945343a1f485c2c844dac9.