Re: [PATCH v2] config: introduce an Operating System-specific `includeIf` condition

From: Phillip Wood <phillip.wood123@gmail.com>
Date: Nov 23, 2022, 15:07 UTC
Message-ID: <a07b5cd7-7ada-7cdf-78cc-375ef21687b0@dunelm.org.uk>
In-Reply-To: <xmqqilj6plta.fsf@gitster.g>

Hi Junio

Welcome back, I hope you enjoyed your time away from the list.

On 23/11/2022 00:16, Junio C Hamano wrote:

Show 10 quoted lines

> Phillip Wood <phillip.wood123@gmail.com> writes:
> 
>> All the other items listed in your table such as branch names are case
>> sensitive. The os name is not so it is of no benefit at all to the
> 
> You keep saying that you consider the OS name is case insensitive,
> but I doubt that is the case, not in the sense that MacOS and macOS
> are two different operating systems, but in the sense that OS
> publishers have a single preferred way to spell their ware (which is
> shown in "uname -s" output), and we should respect that.

I can see that we would want to respect the preferred spelling in our documentation but it seems a bit mean to penalize users who write

	[IncludeIf "os:windows"]

rather than

	[IncludeIf "os:Windows"]

Best Wishes

Phillip