Re: [PATCH v3] t: use pre-defined utf-8 locale for testing svn
- From
Jeff King <peff@peff.net>
- Date
- Jun 8, 2021, 06:35 UTC
- Message-ID
- <YL8PsmLaU+MpWv58@coredump.intra.peff.net>
- In-Reply-To
- <xmqqk0n6wkrr.fsf@gitster.g>
On Mon, Jun 07, 2021 at 10:01:12AM +0900, Junio C Hamano wrote:
Show 14 quoted lines
> >> I kind of wonder if trying "C.UTF-8" would be a reasonable fallback so > >> that people don't even have to set this extra Makefile knob. But I'm not > >> sure if we have a good way of testing if that locale works (if we can't > >> find the "locale" binary). > > > > I also think we should fallback to "C.UTF-8" instead of not testing those > > tests. However, I don't know if there're any systems that not have "C.UTF-8" > > locale. > > I do share the feeling, but have a hunch that systems lacking > "locale -a" may be either superset of, or has at least large overlap > with, those lacking "C.UTF-8", and the new mechanism introduced here > will cover both of them, so I think it is OK to stop here, at least > for now.
Yeah, I think it is OK to stop here, too. I'd worry that we'd make life unnecessarily complicated for people testing on platforms that lack "locale" and C.UTF-8, for little gain.
The v4 patch looks good to me.
-Peff