Closed Bug 339783 Opened 16 years ago Closed 16 years ago

stop shipping sherlock plugins

Categories

(Firefox Build System :: General, defect)

2.0 Branch
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla2 beta1

People

(Reporter: Pike, Assigned: Pike)

References

Details

Attachments

(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 
once.
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
Status: NEW → ASSIGNED
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 http://www.google.com/firefox , while e.g. I'd like to have http://www.google.pl/firefox in Polish Firefox. 

These /firefox pages don't use the whole IP sniffing voodoo that is used by the main http://www.google.com/ 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 1.5.0.4 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.
Status: ASSIGNED → RESOLVED
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.