# [GIT PULL] hotfix on fr.po for the maint branch

4 messages from 2013-08-30 to 2013-08-30. Participants: Jiang Xin, Junio C Hamano, Jean-Noël AVILA.
Thread: https://gitlist.dev/t/34807

## Jiang Xin, 2013-08-30 09:07

Subject: [GIT PULL] hotfix on fr.po for the maint branch
Message-ID: <CANYiYbG6MweCwLaUw5Uk3S9a0hUY1vuS+ORCF1BipkZkJeHjrw@mail.gmail.com>
URL: https://gitlist.dev/e/CANYiYbG6MweCwLaUw5Uk3S9a0hUY1vuS%2BORCF1BipkZkJeHjrw%40mail.gmail.com

```
Hi, Junio

Please merge this commit to the maint branch.

The following changes since commit e230c568c4b9a991e3175e5f65171a566fd8e39c:

  Git 1.8.4 (2013-08-23 11:49:46 -0700)

are available in the git repository at:

  git://github.com/git-l10n/git-po master

for you to fetch changes up to 21860882c8782771e99aa68fab6e365c628ff39d:

  l10n: fr.po: hotfix for commit 6b388fc (2013-08-30 16:59:29 +0800)

----------------------------------------------------------------
Sebastien Helleu (1):
      l10n: fr.po: hotfix for commit 6b388fc

 po/TEAMS |    1 +
 po/fr.po | 1591 ++++++++++++++++++++++++++++++++++----------------------------
 2 files changed, 874 insertions(+), 718 deletions(-)

--
Jiang Xin

```

## Junio C Hamano, 2013-08-30 16:54

Subject: Re: [GIT PULL] hotfix on fr.po for the maint branch
Message-ID: <xmqq61un15lo.fsf@gitster.dls.corp.google.com>
URL: https://gitlist.dev/e/xmqq61un15lo.fsf%40gitster.dls.corp.google.com
In-Reply-To: <CANYiYbG6MweCwLaUw5Uk3S9a0hUY1vuS+ORCF1BipkZkJeHjrw@mail.gmail.com>

```
Jiang Xin <worldhello.net@gmail.com> writes:

> Please merge this commit to the maint branch.
>
> The following changes since commit e230c568c4b9a991e3175e5f65171a566fd8e39c:
>
>   Git 1.8.4 (2013-08-23 11:49:46 -0700)
>
> are available in the git repository at:
>
>   git://github.com/git-l10n/git-po master
>
> for you to fetch changes up to 21860882c8782771e99aa68fab6e365c628ff39d:
>
>   l10n: fr.po: hotfix for commit 6b388fc (2013-08-30 16:59:29 +0800)
>
> ----------------------------------------------------------------
> Sebastien Helleu (1):
>       l10n: fr.po: hotfix for commit 6b388fc

Hmph, what happened that requires a "hotfix" to message strings?

>
>  po/TEAMS |    1 +
>  po/fr.po | 1591 ++++++++++++++++++++++++++++++++++----------------------------
>  2 files changed, 874 insertions(+), 718 deletions(-)
>
> --
> Jiang Xin

```

## Jean-Noël AVILA, 2013-08-30 17:23

Subject: Re: [GIT PULL] hotfix on fr.po for the maint branch
Message-ID: <21533323.C8UgJI1nMs@cayenne>
URL: https://gitlist.dev/e/21533323.C8UgJI1nMs%40cayenne
In-Reply-To: <xmqq61un15lo.fsf@gitster.dls.corp.google.com>

```
Le vendredi 30 août 2013 09:54:59 Junio C Hamano a écrit :
> Jiang Xin <worldhello.net@gmail.com> writes:
> > Please merge this commit to the maint branch.
> > 
> > The following changes since commit 
e230c568c4b9a991e3175e5f65171a566fd8e39c:
> >   Git 1.8.4 (2013-08-23 11:49:46 -0700)
> > 
> > are available in the git repository at:
> >   git://github.com/git-l10n/git-po master
> > 
> > for you to fetch changes up to 21860882c8782771e99aa68fab6e365c628ff39d:
> >   l10n: fr.po: hotfix for commit 6b388fc (2013-08-30 16:59:29 +0800)
> > 
> > ----------------------------------------------------------------
> > 
> > Sebastien Helleu (1):
> >       l10n: fr.po: hotfix for commit 6b388fc
> 
> Hmph, what happened that requires a "hotfix" to message strings?

Well, as the author of the first commit,, I must say that the quality was not 
good. But it became visible and hooked up a reviewer only when it was promoted 
to stable version...

There are some strings that do not make sense in the usage pages, a missing %s 
in one.  	Agreed that applying this patch does not solve a critical issue, but 
it is low risk.

> 
> >  po/TEAMS |    1 +
> >  po/fr.po | 1591
> >  ++++++++++++++++++++++++++++++++++---------------------------- 2 files
> >  changed, 874 insertions(+), 718 deletions(-)
> > 
> > --
> > Jiang Xin

```

## Junio C Hamano, 2013-08-30 18:07

Subject: Re: [GIT PULL] hotfix on fr.po for the maint branch
Message-ID: <xmqqli3jyrw5.fsf@gitster.dls.corp.google.com>
URL: https://gitlist.dev/e/xmqqli3jyrw5.fsf%40gitster.dls.corp.google.com
In-Reply-To: <21533323.C8UgJI1nMs@cayenne>

```
Jean-Noël AVILA <jn.avila@free.fr> writes:

> Le vendredi 30 août 2013 09:54:59 Junio C Hamano a écrit :
>> Jiang Xin <worldhello.net@gmail.com> writes:
>> > Please merge this commit to the maint branch.
>> > 
>> > The following changes since commit 
> e230c568c4b9a991e3175e5f65171a566fd8e39c:
>> >   Git 1.8.4 (2013-08-23 11:49:46 -0700)
>> > 
>> > are available in the git repository at:
>> >   git://github.com/git-l10n/git-po master
>> > 
>> > for you to fetch changes up to 21860882c8782771e99aa68fab6e365c628ff39d:
>> >   l10n: fr.po: hotfix for commit 6b388fc (2013-08-30 16:59:29 +0800)
>> > 
>> > ----------------------------------------------------------------
>> > 
>> > Sebastien Helleu (1):
>> >       l10n: fr.po: hotfix for commit 6b388fc
>> 
>> Hmph, what happened that requires a "hotfix" to message strings?
>
> Well, as the author of the first commit,, I must say that the quality was not 
> good. But it became visible and hooked up a reviewer only when it was promoted 
> to stable version...
>
> There are some strings that do not make sense in the usage pages, a missing %s 
> in one.  	Agreed that applying this patch does not solve a critical issue, but 
> it is low risk.

OK, I was just reacting to the "hotfix" wording and wondering the
level of urgency.  Will pull to maint later before cutting 1.8.4.1.

Thanks.

>
>> 
>> >  po/TEAMS |    1 +
>> >  po/fr.po | 1591
>> >  ++++++++++++++++++++++++++++++++++---------------------------- 2 files
>> >  changed, 874 insertions(+), 718 deletions(-)
>> > 
>> > --
>> > Jiang Xin

```
