Page MenuHomePhabricator

LocalSettings.sample is misleading, obsolete
Closed, ResolvedPublic

Description

Much of LocalSettings.sample is incorrect, obsolete, misleading, or downright
dangerous. It leads people astray when they try to do manual installs instead of
using the installer, and desperately needs to be updated.


Version: 1.3.x
Severity: major

Details

Reference
bz704

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 6:58 PM
bzimport set Reference to bz704.
bzimport added a subscriber: Unknown Object (MLST).

jeluf wrote:

Should it be deleted instead?

  • Users don't need to generate LocalSettings.php from the sample any more, the config script generates a LocalSettings.php
  • Duplicating data (DefaultSettings.php and LocalSettings.sample) is error-prone.

Nothing should be duplicated between DefaultSettings.php and
LocalSettings.sample. There would, however, be duplication between the
generation code in config/index.php and LocalSettings.sample.

I've gone ahead and removed it.