From: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: Re: Cooking of the ab/i18n series
Date: Sat, 7 Aug 2010 01:13:47 +0000 [thread overview]
Message-ID: <AANLkTimvNgZ4UU595==Tx5i0LYF56ecNzvYWki0FM3u6@mail.gmail.com> (raw)
In-Reply-To: <7viq3n73c6.fsf@alter.siamese.dyndns.org>
On Fri, Aug 6, 2010 at 23:01, Junio C Hamano <gitster@pobox.com> wrote:
> Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
>
>> My mental plan for this series has basically been as follows:
>>
>> 1. Get it to a state where it can cook in pu [You Are Here]
>>
>> 2. After it's been there for a while get it to master
>>
>> 3. Once it's there for a while and we're sure the new dependency /
>> code doesn't harm some more obscure systems..
>>
>> 4. Start submitting patches to the main porcelain $(grep
>> 'mainporcelain common' command-list.txt) to make the most common
>> user-visible messages translatable.
>>
>> 5. Recruit translators to translate the strings in #4. Send
>> translations in as patches adding/altering the *.po files.
>
> Matches my expectations modulo s/master/next/.
Yes, I meant to say "on track to master" (via next).
> The stuff parked in 'pu' was primarily because most of the work to
> get to this point was done during the pre-release freeze for 1.7.2
> and I didn't want to get distracted, while I obviously did not lose
> patches.
I didn't mean to rush you, the delay is fine. It just sounded like the
series might be stuck in pu in perpetuity from the initial "When
people see the benefit of doing so" comment.
But if not, nevermind.
>> Now, your main concern is that this doesn't break plumbing output....
>
> That's not 'main'.
>
> 'Breaking plumbing' is merely an example of a larger 'main concern' which
> is 'unintended consequences'.
Sure, all code's going to have problems, and the ab/i18n series is
around 1300 lines, so I'd be quite surprised if it didn't have some
bug.
> And please don't ask me to enumerate them exhaustively. By definition
> 'unintended consequences' cannot be enumerated. They are discovered over
> time either by code inspection, by people guinea-pigging a version outside
> 'master', and by careful thinking.
I was just wondering if there was anything particular that you were
worried about that could be addressed.
But as you've clarified there's nothing about it that I should be
explicitly addressing before it can move forward (if I've understood
you correctly).
Thanks for clarifying that and for your time.
prev parent reply other threads:[~2010-08-07 1:14 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-05 21:09 Cooking of the ab/i18n series Ævar Arnfjörð Bjarmason
2010-08-05 22:18 ` Junio C Hamano
2010-08-06 12:33 ` Ævar Arnfjörð Bjarmason
2010-08-06 13:30 ` Junio C Hamano
2010-08-06 14:03 ` Ævar Arnfjörð Bjarmason
2010-08-06 15:48 ` Junio C Hamano
2010-08-06 17:01 ` Ævar Arnfjörð Bjarmason
2010-08-06 19:28 ` Ævar Arnfjörð Bjarmason
2010-08-06 23:01 ` Junio C Hamano
2010-08-07 1:13 ` Ævar Arnfjörð Bjarmason [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='AANLkTimvNgZ4UU595==Tx5i0LYF56ecNzvYWki0FM3u6@mail.gmail.com' \
--to=avarab@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).