Travis: --enable-dba --with-gdbm --tcadb #164

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
6 participants

This PR enables travis tests of ext/dba with the handlers inifile (buildin), flatfile (buildin), cdb (buildin), gdbm and tcadb.

I also would like to add --with-db4 but can't compile :( - see http://travis-ci.org/#!/marc-mabe/php-src/builds/2138877

This pull request fails (merged 99ca7f8 into 018395e).

Contributor

reeze commented Aug 16, 2012

PHP's test didn't passed in Travis as travisbot reported. currently, it seems useless :(

Owner

nikic commented Aug 16, 2012

It would be nice to get a clean build before adding any more stuff. Right now Travis isn't particularly helpful as it always fails ^^

Sure, we need a clean build but it should be clean for all build-in functions. Else we could disable other failed extensions to get a clean build but this doesn't help.

Travis should run all possible test and the bugs/tests should be fixed instead of ignoring tests.

Contributor

lstrojny commented Jan 14, 2013

@reeze I agree with @nikic here. We need to fix the travis build first (btw: any takers welcome) and than enable more and more extension.

Comment on behalf of lstrojny at php.net:

Closing for now

@php-pulls php-pulls closed this Jan 14, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment