When installing a fresh HEAD from CVS, you get a fatal error because it attempts to initialize a database engine that doesn't exist (CVS). This is because it sees the CVS folder, and assumes it's a database engine.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RobLoach’s picture

Status: Needs review » Needs work
RobLoach’s picture

Status: Needs work » Needs review
FileSize
1.22 KB

Thoughts?

RobLoach’s picture

Title: Class 'DatabaseInstaller_CVS' not found in includes/install.inc » Database engine initialization during install failure
Assigned: Unassigned » RobLoach

Changing title to be a bit more descriptive.

moshe weitzman’s picture

Priority: Normal » Critical

bit me too.

xmacinfo’s picture

CVS update forces an install, too, giving out the same errors as a fresh install.

Crell’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
1.57 KB

Yeah, that one's my fault. The patch in #3 fixes it, but is not super approachable. The attached patch is identical except for the addition of some comments. No other changes. Dries, please just credit Rob.

lilou’s picture

Install work now for me.

catch’s picture

Title: Database engine initialization during install failure » HEAD broken: Database engine initialization during install failure

I posted a duplicate, this one's good.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks!

chx’s picture

Assigned: RobLoach » chx
Status: Fixed » Reviewed & tested by the community
FileSize
545 bytes

This is incorrect. I am surprised noone asked "huh? max on an array of strings? what that could mean?" max manual says:

the alphabetical highest string value will be returned.

so as soon as someone adds a db2 or firebird directory in there, you will end up with a driver called "includes". This is easy to avoid...

RobLoach’s picture

There we go! Thanks a lot, chx! You're my hero..... I knew max looked weird....

keith.smith’s picture

Oddly, I still had trouble, with the error that a text blob could not have a default value.

To get my Drupal 7 to install tonight, I had to use the following patch (which may have an entire host of consequences I don't understand, so use at your own risk).

Edit: Oh--and I don't really want to remove the RTBC on the earlier patch, though it did not work for me.

Heine’s picture

@keith.smith; your problem "a default on a text field" is distinct from the "wrong db driver selection during install". Can you please open a new issue?

keith.smith’s picture

My apologies for derailing the issue. I created another issue at http://drupal.org/node/299292.

Freso’s picture

Both with and without this patch, I'm getting Fatal error: Class 'PDO' not found in /srv/http/localhost/htdocs/drupal7/includes/database/database.inc on line 137

Freso’s picture

(And I'm not seeing the "CVS" error either way either.)

catch’s picture

Freso - do you definitely have PDO compiled into your PHP install? I've opened a new issue here for adding PDO to system requirements #299308: Installing Drupal by visiting index.php (rather than install.php) leads to a fatal error when PDO is not enabled. This error will only show up with a CVS checkout, not wit 7.x-dev tarball.

I

RobLoach’s picture

Both patches, #10 and #12, are good to go. Although, #12 is somewhat unrelated to this issue.

chx’s picture

#10 must go in.

Freso’s picture

#17: I can very much verify that I do not have PDO compiled in (I didn't know that was a requirement now either). Sorry for wasting everybody's time. :$

/me makes a mental note to add "pdo" to PHP's USE-flags when he gets home from work...

Dries’s picture

Status: Reviewed & tested by the community » Fixed

I've committed #10. Let's create a new issue for #12. Thanks all.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

nikemen’s picture

Well, doesn't work for me though I have PHP 5.2.6 with PDO installed. Help!

nikemen’s picture

Status: Closed (fixed) » Active
RobLoach’s picture

Status: Active » Fixed

If you're getting any errors with Drupal HEAD, open up a new issue and post in what warnings/errors your getting. This fix went in.

Crell’s picture

Status: Fixed » Closed (fixed)