As seen here it's not so obvious what to fill into the $dburl line in include/conf.php.

My suggestion to clarify this a bit:

Replace
# $db_url = "mysql://user:password@hostname/database";
with
# $db_url = "mysql://dbuser:dbpassword@dbhostname/database";

This should make it a little clearer that the database user, password and host is meant.

The same should be done with the PostgreSQL line...

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Chris Johnson’s picture

Version: 4.5.0-rc »
Assigned: Unassigned » Chris Johnson
FileSize
579 bytes

Here's a simple patch against 4.5.2.

Chris Johnson’s picture

FileSize
551 bytes

And here is the equivalent patch for HEAD, against settings.php.

tangent’s picture

Makes sense to me. +1

Bèr Kessels’s picture

FileSize
550 bytes

+1 from me. This very small change makes it very clear that we do not want the hostname of the server, but of the database.
One small question: in drupal we use underscores to mark words everywhere (functions, variables, modules, themes etc.) so can you use db_hostname?

Attached patch does that.

Dries’s picture

Err, your patch looks inconsistent: it only uses underscore on some strings. I'll fix it up and commit it to HEAD.

Dries’s picture

Committed to HEAD. Thanks.

Anonymous’s picture