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...

CommentFileSizeAuthor
#4 db_hostname.patch550 bytesBèr Kessels
#2 settings.php.patch551 bytesChris Johnson
#1 conf.php.patch579 bytesChris Johnson

Comments

Chris Johnson’s picture

Chris Johnson commented
Version: 4.5.0-rc »
Assigned: Unassigned » Chris Johnson
StatusFileSize
new conf.php.patch579 bytes

Here's a simple patch against 4.5.2.

Chris Johnson’s picture

Chris Johnson commented
StatusFileSize
new settings.php.patch551 bytes

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

tangent’s picture

tangent commented

Makes sense to me. +1

Bèr Kessels’s picture

Bèr Kessels commented
StatusFileSize
new db_hostname.patch550 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

dries commented

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

dries commented

Committed to HEAD. Thanks.

Anonymous’s picture

(not verified) commented