Re: [PATCH] gitweb/INSTALL: GITWEB_CONFIG_SYSTEM is for backward compatibility

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Drew Northup <n1xim.email@xxxxxxxxx> writes:

>> That makes it sound like the "per instance overrides common overrides
>> built-in" cascading is what is unusual and what we need to apologize
>> for.
>
> I don't think were apologizing for anything. It is helpful to say "we
> do some things differently here and don't plan on changing for a very
> important reason.

My biggest resistance against this whole thing is that word
"differently".

It really depends on who reads this document.  For some, it may be
different.  For others, it may not be unexpected at all.  "We do
things differently" may help the former but I do not want the latter
to try to find "difference" that does not exist elsewhere and get
confused.

>>
>> +BUGS
>> +----
>> +Debugging would be easier if the fallback configuration file
>> +(`/etc/gitweb.conf`) and environment variable to override its location
>> +('GITWEB_CONFIG_SYSTEM') had names reflecting their "fallback" role.
>> +The current names are kept to avoid breaking working setups.
>> +
>>  ENVIRONMENT
>>  -----------
>>  The location of per-instance and system-wide configuration files can be
>
> I don't disagree with this, as some would consider the naming to be a
> bug, but after having been given a good schooling on the git list a
> while back as to why it is the way it is I'm hesitant to label "has
> history" as a bug.

BUGS section being traditionally where you would throw this kind of
thing, either a bug, non-bug, or wai-but-some-may-not-agree, I think
this is a good description.

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]