Closed Bug 339783 Opened 16 years ago Closed 16 years ago

stop shipping sherlock plugins


(Firefox Build System :: General, defect)

2.0 Branch
Not set


(Not tracked)

mozilla2 beta1


(Reporter: Pike, Assigned: Pike)




(1 file)

This is bug is basically to revert the fuzzy code from bug 331118 back to
strictly checking the list of shipped plugins.

I'd like to be able to pick up plugins from the en-US directory, too, so that
we only have to maintain one google plugin.
I want to look at creating the yahoo search plugins by preprocessing, too.
Flags: blocking-firefox2?
Whiteboard: [swag:1d]
As one google plugin suffices, I made the logic pick up search plugins from the
en-US directory before the locale dir. That way, we only have to maintain that 
If other plugins are heavily shared, we may want to do the same for them.
Yahoo is not one of them for now, though.
Assignee: nobody → l10n
Attachment #224112 - Flags: review?(benjamin)
Whiteboard: [swag:1d] → [patch-r?]
Flags: blocking-firefox2? → blocking-firefox2+
There is one problem with using the en-US Google plugin for all locales - the <SearchForm> tag is set to , while e.g. I'd like to have in Polish Firefox. 

These /firefox pages don't use the whole IP sniffing voodoo that is used by the main page.

(The SearchForm tag specifies the page opened when user uses Tools > Web search, but has the search bar hidden).
I don't think that's necessary. As we currently don't do any of the official
flag stuff for the search plugins (bug 335460), that may very well change.
The german plugin in for example gets 
client=firefox-a&rls=org.mozilla:de:official added, which actually triggers the
language to be ok. And I would expect those params to be added to the searchform
query, too. Gavin?
Comment on attachment 224112 [details] [diff] [review]
use vpath to pick only xml files for search plugins, from en-US or locale dir

That's pretty cool, I didn't know about the vpath directive.
Attachment #224112 - Flags: review?(benjamin) → review+
Whiteboard: [patch-r?] → checked in on trunk
Attachment #224112 - Flags: approval-branch-1.8.1?(benjamin)
Attachment #224112 - Flags: approval-branch-1.8.1?(benjamin) → approval-branch-1.8.1+
Checked in on trunk and 1.8 branch.
Closed: 16 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Whiteboard: checked in on trunk
Component: Build Config → General
Product: Firefox → Firefox Build System
Keywords: fixed1.8.1
Target Milestone: Firefox 2 beta1 → mozilla2 beta1
You need to log in before you can comment on or make changes to this bug.