Re: [PATCH v2] fetch: align new ref summary printout in UTF-8 locales
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 12, 2012, 18:02 UTC
- Message-ID
- <7vpq5rce6y.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <CACsJy8APtTU-7ZfTdcjg0nKoBSboNpP+7Va=akXcvn95Sd166A@mail.gmail.com>
Nguyen Thai Ngoc Duy <pclouds@gmail.com> writes:
> Ping.. what happens to this patch? Do you want to support other > encodings as well via iconv()? I can't test that though.
I thought I refuted a potential blocker, which was an implied objection from Torsten, in $gmane/204912 already. As long as we make it clear that your patch helps only "ASCII/UTF-8 only" audience but we still "try to be nicer to 'others'" by doing two things I said in the message, I think we should proceed without iconv() to keep things simple.