Closed
Bug 731078
Opened 13 years ago
Closed 13 years ago
[firefox] Merge Google search plugin changes with google-ku.xml
Categories
(Mozilla Localizations :: ku / Kurdish, defect)
Mozilla Localizations
ku / Kurdish
Tracking
(firefox12+ fixed)
RESOLVED
FIXED
People
(Reporter: MattN, Assigned: MattN)
References
()
Details
(Whiteboard: [qa-])
Attachments
(1 file)
4.08 KB,
patch
|
rail
:
review+
Pike
:
review+
|
Details | Diff | Splinter Review |
Bug 722352 and bug 724116 made changes to the Google search plugin that need to propagate to all locales.
google-ku.xml needs to be updated to reflect the changes. See https://hg.mozilla.org/mozilla-central/file/tip/browser/locales/en-US/searchplugins/google.xml for the latest en-US version.
Updated•13 years ago
|
Assignee | ||
Comment 1•13 years ago
|
||
I preserved the 4 ways in which google-ku.xml differed from google.xml (en-US):
* <Description>
* <Param name="hl" value="en"/>
* Search suggestions: hl=en
* Search form with hl=en
Assignee: nobody → mnoorenberghe+bmo
Status: NEW → ASSIGNED
Attachment #601853 -
Flags: review?(rail)
Attachment #601853 -
Flags: review?(l10n)
Comment 2•13 years ago
|
||
Comment on attachment 601853 [details] [diff] [review]
v.1 Include channel and client changes while preserving 4 variances from en-US (m-a)
Looks good to me.
Updated•13 years ago
|
Attachment #601853 -
Flags: review?(rail) → review+
Comment 3•13 years ago
|
||
Comment on attachment 601853 [details] [diff] [review]
v.1 Include channel and client changes while preserving 4 variances from en-US (m-a)
r=me, too. Rail, do you want to land that?
Attachment #601853 -
Flags: review?(l10n) → review+
Comment 4•13 years ago
|
||
(In reply to Axel Hecht [:Pike] from comment #3)
> Comment on attachment 601853 [details] [diff] [review]
> v.1 Include channel and client changes while preserving 4 variances from
> en-US (m-a)
>
> r=me, too. Rail, do you want to land that?
Pushed to l10n-central
http://hg.mozilla.org/l10n-central/ku/rev/afdf0c1c73d8
Axle, Matthew, should I push to Aurora as well?
Comment 5•13 years ago
|
||
Yes, please.
Comment 6•13 years ago
|
||
http://hg.mozilla.org/releases/l10n/mozilla-aurora/ku/rev/b0b03cda3d8d
Matthew, feel free to close the bug.
Assignee | ||
Comment 7•13 years ago
|
||
Looks good to me. Is there some sign-off needed for this to get into l10n builds or do they use the tip of the repo?
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•