451199 - Add gmail to the localizations that want that

Status: RESOLVED FIXED

(Mozilla Localizations :: Other, defect)

Description

[Axel Hecht [:Pike]]

We want to add Gmail to a bunch of locales, I'd create a patch for that. En-US was bug 444395.

Mic collected a list of locales that want that.

Things to watch out for are at least Gmail vs Googlemail (just de, en-GB?), and I saw some locales asking to drop yahoo mail. Do we want to do that? If so, I'll need to treat those differently.

Mic, can you follow up here with a list of locales? Just language code per procedure would do.

Comment 1

[Mic Berman]

hi
here are the list of locales who have opted in via google group notice or bugmail. There are a few that are still to be confirmed (pt-PT, gu-IN, el). I will verify those asap

af
ar
as
be
bg
bn
ca
cs
da
de
en-gb
es-ar
es-ES
eu
fi
fr
fy-nl
ga-ie
gl
gl
he
hi
hu
id
is
it
ja
ko
kn
lt
mk
nb-no
nn-no
pa
pl
pt-br
ro
ru
si
sk
sl
sq
sv-SE
te
th
th
uk
zh-cn
zh-tw

Comment 2

[João Neves]

pt-PT opted in (10-07-2008 15:37 (GMT+1) according to my sent folder)

Comment 3

[Axel Hecht [:Pike]]

Attachment: add Gmail/Googlemail to locales

This patch for reference, and inspection.

This is the list I used for copy'n'paste:

Gmail:
af ar as be bg bn-IN ca cs da es-AR es-ES eu fi fr fy-NL ga-IE gl gl he hi-IN hu id is it ja ja-JP-mac ko kn lt mk nb-NO nn-NO pa-IN pl pt-BR pt-PT ro ru si sk sl sq sv-SE te th th uk zh-CN zh-TW

Googlemail:
de en-GB

As per irc discussion with Mic, we didn't do any yahoo removals.

Comment 4

[Axel Hecht [:Pike]]

Attachment: add Gmail/Googlemail to locales, v2

I adjusted the spacing around the '=' to be more like en-US region.properties, and removed one gl from the list, it was on there twice, leading to a double gmail handler.

Updated list for Gmail:
af ar as be bg bn-IN ca cs da es-AR es-ES eu fi fr fy-NL ga-IE gl he hi-IN hu id is it ja ja-JP-mac ko kn lt mk nb-NO nn-NO pa-IN pl pt-BR pt-PT ro ru si sk sl sq sv-SE te th th uk zh-CN zh-TW

Comment 5

[Axel Hecht [:Pike]]

Comment on attachment 334693 [details] [diff] [review]
add Gmail/Googlemail to locales, v2

Thai is doubled, too.

Not mentioned so far, but in all-locales are et, hy-AM, ka, ku, ml, mn, mr, nl, sr, tr

mr is bug 446613 comment 8, adding that, too.

New patch coming up.

Comment 6

[Axel Hecht [:Pike]]

Attachment: add Gmail/Googlemail to locales, v3

Yet another updated locale list for Gmail:

af ar as be bg bn-IN ca cs da es-AR es-ES eu fi fr fy-NL ga-IE gl he hi-IN hu id is it ja ja-JP-mac ko kn lt mk mr nb-NO nn-NO pa-IN pl pt-BR pt-PT ro ru si sk sl sq sv-SE te th uk zh-CN zh-TW

Googlemail:

de en-GB

Comment 7

[Axel Hecht [:Pike]]

Comment on attachment 334695 [details] [diff] [review]
add Gmail/Googlemail to locales, v3

Phu, finally, a=me for me landing this on cvs trunk for 3.0.2.

I'll probably abuse this patch to generate a series of patches to land on hg to finalize the l10n buildbot work.

Comment 8

[Axel Hecht [:Pike]]

Tim, can QA help out with spot tests to verify this?

Comment 9

[Axel Hecht [:Pike]]

... which means to say that I landed this on CVS trunk.

Checking in af/browser/chrome/browser-region/region.properties;
/l10n/l10n/af/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.4; previous revision: 1.3
done
Checking in ar/browser/chrome/browser-region/region.properties;
/l10n/l10n/ar/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.6; previous revision: 1.5
done
Checking in as/browser/chrome/browser-region/region.properties;
/l10n/l10n/as/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.2; previous revision: 1.1
done
Checking in be/browser/chrome/browser-region/region.properties;
/l10n/l10n/be/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.7; previous revision: 1.6
done
Checking in bg/browser/chrome/browser-region/region.properties;
/l10n/l10n/bg/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.7; previous revision: 1.6
done
Checking in bn-IN/browser/chrome/browser-region/region.properties;
/l10n/l10n/bn-IN/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.5; previous revision: 1.4
done
Checking in ca/browser/chrome/browser-region/region.properties;
/l10n/l10n/ca/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.15; previous revision: 1.14
done
Checking in cs/browser/chrome/browser-region/region.properties;
/l10n/l10n/cs/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in da/browser/chrome/browser-region/region.properties;
/l10n/l10n/da/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.6; previous revision: 1.5
done
Checking in de/browser/chrome/browser-region/region.properties;
/l10n/l10n/de/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in en-GB/browser/chrome/browser-region/region.properties;
/l10n/l10n/en-GB/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.8; previous revision: 1.7
done
Checking in es-AR/browser/chrome/browser-region/region.properties;
/l10n/l10n/es-AR/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.11; previous revision: 1.10
done
Checking in es-ES/browser/chrome/browser-region/region.properties;
/l10n/l10n/es-ES/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.8; previous revision: 1.7
done
Checking in eu/browser/chrome/browser-region/region.properties;
/l10n/l10n/eu/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.13; previous revision: 1.12
done
Checking in fi/browser/chrome/browser-region/region.properties;
/l10n/l10n/fi/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.19; previous revision: 1.18
done
Checking in fr/browser/chrome/browser-region/region.properties;
/l10n/l10n/fr/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.22; previous revision: 1.21
done
Checking in fy-NL/browser/chrome/browser-region/region.properties;
/l10n/l10n/fy-NL/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.10; previous revision: 1.9
done
Checking in ga-IE/browser/chrome/browser-region/region.properties;
/l10n/l10n/ga-IE/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.14; previous revision: 1.13
done
Checking in gl/browser/chrome/browser-region/region.properties;
/l10n/l10n/gl/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in he/browser/chrome/browser-region/region.properties;
/l10n/l10n/he/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.11; previous revision: 1.10
done
Checking in hi-IN/browser/chrome/browser-region/region.properties;
/l10n/l10n/hi-IN/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.6; previous revision: 1.5
done
Checking in hu/browser/chrome/browser-region/region.properties;
/l10n/l10n/hu/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.10; previous revision: 1.9
done
Checking in id/browser/chrome/browser-region/region.properties;
/l10n/l10n/id/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in is/browser/chrome/browser-region/region.properties;
/l10n/l10n/is/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in it/browser/chrome/browser-region/region.properties;
/l10n/l10n/it/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.21; previous revision: 1.20
done
Checking in ja-JP-mac/browser/chrome/browser-region/region.properties;
/l10n/l10n/ja-JP-mac/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.11; previous revision: 1.10
done
Checking in ja/browser/chrome/browser-region/region.properties;
/l10n/l10n/ja/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.11; previous revision: 1.10
done
Checking in kn/browser/chrome/browser-region/region.properties;
/l10n/l10n/kn/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in ko/browser/chrome/browser-region/region.properties;
/l10n/l10n/ko/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.11; previous revision: 1.10
done
Checking in lt/browser/chrome/browser-region/region.properties;
/l10n/l10n/lt/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in mk/browser/chrome/browser-region/region.properties;
/l10n/l10n/mk/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in mr/browser/chrome/browser-region/region.properties;
/l10n/l10n/mr/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.4; previous revision: 1.3
done
Checking in nb-NO/browser/chrome/browser-region/region.properties;
/l10n/l10n/nb-NO/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.10; previous revision: 1.9
done
Checking in nn-NO/browser/chrome/browser-region/region.properties;
/l10n/l10n/nn-NO/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.7; previous revision: 1.6
done
Checking in pa-IN/browser/chrome/browser-region/region.properties;
/l10n/l10n/pa-IN/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in pl/browser/chrome/browser-region/region.properties;
/l10n/l10n/pl/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.25; previous revision: 1.24
done
Checking in pt-BR/browser/chrome/browser-region/region.properties;
/l10n/l10n/pt-BR/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.13; previous revision: 1.12
done
Checking in pt-PT/browser/chrome/browser-region/region.properties;
/l10n/l10n/pt-PT/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in ro/browser/chrome/browser-region/region.properties;
/l10n/l10n/ro/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.10; previous revision: 1.9
done
Checking in ru/browser/chrome/browser-region/region.properties;
/l10n/l10n/ru/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.24; previous revision: 1.23
done
Checking in si/browser/chrome/browser-region/region.properties;
/l10n/l10n/si/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in sk/browser/chrome/browser-region/region.properties;
/l10n/l10n/sk/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.8; previous revision: 1.7
done
Checking in sl/browser/chrome/browser-region/region.properties;
/l10n/l10n/sl/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.10; previous revision: 1.9
done
Checking in sq/browser/chrome/browser-region/region.properties;
/l10n/l10n/sq/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.7; previous revision: 1.6
done
Checking in sv-SE/browser/chrome/browser-region/region.properties;
/l10n/l10n/sv-SE/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.22; previous revision: 1.21
done
Checking in te/browser/chrome/browser-region/region.properties;
/l10n/l10n/te/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in th/browser/chrome/browser-region/region.properties;
/l10n/l10n/th/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.2; previous revision: 1.1
done
Checking in uk/browser/chrome/browser-region/region.properties;
/l10n/l10n/uk/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in zh-CN/browser/chrome/browser-region/region.properties;
/l10n/l10n/zh-CN/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.20; previous revision: 1.19
done
Checking in zh-TW/browser/chrome/browser-region/region.properties;
/l10n/l10n/zh-TW/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.17; previous revision: 1.16
done

Comment 10

[Ankit Patel]

Hi,

gu-IN also wants to add gmail to the list of mailto protocol handlers.
Let me knowk, if you want me to make this change myself OR you want to do it yourself.

Comment 11

[Axel Hecht [:Pike]]

Hey Ankit, mind filing a new bug for gu-IN? Thanks.

Comment 12

[Axel Hecht [:Pike]]

Oh, forgot to mention, landings on hg done, too.

Comment 13

[Pi Wei Cheng [:piwei] (inactive)]

Out of scope. Adding keyword to exclude.