[SM-Commit] PERFORCE change 76944 by Eric Sandall for review
Arwed von Merkatz
v.merkatz at gmx.net
Wed Mar 29 03:17:53 EST 2006
On Tue, Mar 28, 2006 at 11:52:46PM -0600, Jeremy Blosser wrote:
> On Mar 29, Perforce Review Daemon [p4review at smee.org] wrote:
> > Change 76944 by eric_sandall at sandalle-moby on 2006/03/29 06:36:20
> > * INSTALL: Added to install default global mpd.conf
>
> We need to establish some kind of policy around this. We just had a long
> bug discussion (http://bugs.sourcemage.org/show_bug.cgi?id=10486) that
> ended up with "if upstream doesn't install it during 'make install', we
> should ask before installing it and default to 'n'".
>
> However, there are some obvious differences between a sample config that
> _changes_ default behavior in a user app and a config file for a daemon
> that needs some kind of config to operate. Is that where we want to draw
> the line? We should establish what it is so spells can adhere to it and
> users can get a consistent experience from spells.
Default config files where the app requires them to run should usually
be installed by the apps installation process. If that doesn't happen
I'd consider it an upstream bug and our installing the config a bug
workaround, so that's acceptable.
Another case that would be fine in my opinion is a config file that
doesn't change the defaults, just makes them explicit. This gives the
user a starting point for his own customization.
--
Arwed v. Merkatz Source Mage GNU/Linux developer
http://www.sourcemage.org
More information about the SM-Commit
mailing list