From: Phillip Wood Date: Wed, 23 Nov 2022 15:07:41 GMT Subject: Re: [PATCH v2] config: introduce an Operating System-specific `includeIf` condition Message-ID: In-Reply-To: Hi Junio Welcome back, I hope you enjoyed your time away from the list. On 23/11/2022 00:16, Junio C Hamano wrote: > Phillip Wood 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