INSTALL.txt has not been updated to match multisite functionality change

According to my read of
https://drupal.org/node/1792924
the scanning stuff still works as it used to, but is only invoked if the sites.php file exists. The documentation in sites.php agrees with that interpretation.

So this patch isn't correct. All it needs is an addition to say that the multi-site scanning and the possibility of using multiple different settings.php files doesn't take place unless there is a sites.php file present.

If the interpretation in #3 is incorrect, then we need to update:
- that change notice
- the INSTALL.txt file
- the documentation in sites.php

I do not think we want to remove these lines from INSTALL.php:

 Additional site configurations are created in subdirectories within the 'sites'
 directory. Each subdirectory must have a 'settings.php' file, which specifies
-the configuration settings. The easiest way to create additional sites is to
-copy the 'default' directory and modify the 'settings.php' file as appropriate.
-The new directory name is constructed from the site's URL. The configuration for
-www.example.com could be in 'sites/example.com/settings.php' (note that 'www.'
-should be omitted if users can access your site at http://example.com/).
+the configuration settings. Make a copy of sites/default/default.settings.php
+to your new site directory.

The changes to sites.php look good!

True, we need to say to start with default.settings.php and not settings.php. But we still need the information about the name of the directory mapping to the site URL, which was removed.

Thanks for the reroll!

I gave this patch a review, and found a typo:

+copy file 'default.settings.php' from the 'sites/default' directory into the new
+site directory into the a 'settings.php' file and modify as appropriate.

Other than that, I think it is clear. However, I think we also need a patch for the top portion of default.settings.php, which doesn't make it clear that you need sites.php in order fo this to work.

That looks fine, thanks!

Another typo I missed in the last review, in INSTALL.txt:

+For this to work you need the file sites/sites.php to exists. Make a copy of

exists -> exist

And we still need some changes to be made to default.settings.php.

Can you upload the patch again? It seems to be empty.

Thanks!

One error:

+ * sites/sites.php must be present. It's optional settings will be loaded, and

It's should be Its.

That small fix and I think we'll be done!

I think this latest patch is good to go. Thanks!

Thanks again! Committed to 8.x.