881818 - [ar] Remove en-US searchplugins from browser/searchplugins

Status: RESOLVED FIXED

(Mozilla Localizations :: ar / Arabic, defect)

Description

[Francesco Lodolo [:flod]]

amazondotcom.xml
eBay.xml
yahoo.xml

These 3 files are inside the browser/searchplugins folder of your locale but are unused, since the corresponding en-US searchplugins are picked at build time.

Therefore these files should be removed.

Comment 1

[Axel Hecht [:Pike]]

CCing Khaled, I think he can take this.

Talking of which, Khaled, should you get bugmail for Arabic bugs? If so, head over to https://bugzilla.mozilla.org/userprefs.cgi?tab=component_watch&product=Mozilla%20Localizations and select the arabic component?

Comment 2

[Khaled Hosny]

http://hg.mozilla.org/releases/l10n/mozilla-aurora/ar/rev/56978e271b26
http://hg.mozilla.org/releases/l10n/mozilla-beta/ar/rev/168a8fccfe13

Thanks Axel, I’m waching Arabic bugs now.

Comment 3

[Khaled Hosny]

Attachment: Remove en-US search plugins

Oops, I shouldn’t have commited the change directly. Reverted both commits, patch attached.

Comment 4

[Axel Hecht [:Pike]]

Comment on attachment 761511 [details] [diff] [review]
Remove en-US search plugins

Review of attachment 761511 [details] [diff] [review]:
-----------------------------------------------------------------

r=me, go ahead and land again. The commit message is fine, you may just want to ammend it with an 'r=Pike'.

Comment 5

[Khaled Hosny]

http://hg.mozilla.org/releases/l10n/mozilla-aurora/ar/rev/4d24377bf3ee

Comment 6

[Francesco Lodolo [:flod]]

THank you very much Khaled.