Closed Bug 1314285 Opened 5 years ago Closed 5 years ago

Switch search scripts to use global versions

Categories

(Thunderbird :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 52.0

People

(Reporter: mkaply, Assigned: mkaply)

Details

Attachments

(1 file)

When I originally did my search changes for Thunderbird, I used the same PY file in TB that I used in Firefox (so they were duplicated).

I've moved the files to a central location so we don't have to duplicate.
Attachment #8806357 - Flags: review?(jorgk)
Comment on attachment 8806357 [details] [diff] [review]
Use python actions for search

Thanks. That looks like a build issue of some sort ;-) I'd rather pass this in to Aleth. He's our build peer:
https://wiki.mozilla.org/Modules/MailNews_Core#Build_Config:_comm-central

Sorry, I'm not competent here.
Attachment #8806357 - Flags: review?(jorgk) → review?(aleth)
Comment on attachment 8806357 [details] [diff] [review]
Use python actions for search

The removal of the mail/ copy of searchplugins.py doesn't seem to be in this patch yet? Please add this if it's no longer needed.

For future reference, you can also use MOZILLA_DIR to refer to the m-c root in a m-c/c-c independent way.
Attachment #8806357 - Flags: review?(aleth) → review+
Assignee: nobody → mozilla
I have the patch updated with removal and ready to go when the tree opens.
(In reply to Mike Kaply [:mkaply] from comment #3)
> I have the patch updated with removal and ready to go when the tree opens.
The C-C tree opened again, so will you attach an updated patch?
Thanks. I guess we're done here then.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 52.0
You need to log in before you can comment on or make changes to this bug.