674453 - Update distribution.ini in the NTT Resonant (greengoo) Partner Repack to remove "goo RSS Reader"

Status: VERIFIED FIXED

(Release Engineering :: Release Requests, defect)

Description

[Kohei Yoshino [:kohei]]

Attachment: distribution.ini diff

NTT Resonant has discontinued their own RSS reader service. It has already been removed from the vanilla Firefox builds. (Bug 605653)

The patch for distribution.ini is attached. The browser.contentHandlers.* prefs, which has been set to make the "goo RSS Reader" first, are no longer required.

Please use this for Firefox 6+. Thanks!

Comment 1

[Kohei Yoshino [:kohei]]

(In reply to comment #0)
> NTT Resonant has discontinued their own RSS reader service. It has already
> been removed from the vanilla Firefox builds. (Bug 605653)

Nope, Bug 649888.

Comment 2

[Kohei Yoshino [:kohei]]

ping?

Comment 3

[Kohei Yoshino [:kohei]]

Hi Kev, the Firefox 8 release is approaching. Could you take this?

Comment 4

[Kohei Yoshino [:kohei]]

Kev, ping...?

Comment 5

[Kohei Yoshino [:kohei]]

Seems like Kev is busy, so I'll manually edit the ini file and repack the exe/dmg as before, for the meantime.

Comment 6

[Kev Needham [:kev]]

Not so much busy as overwhelmed sometimes by bugmail. All readers are removed by the patch, is this intended behaviour?

Comment 7

[Kohei Yoshino [:kohei]]

Yes, by removing the designation, the order and listing of the feed readers will be the same as the vanilla Firefox.

Comment 8

[Kohei Yoshino [:kohei]]

So I've manually repacked and signed the Firefox 8.0 and 8.0.1 builds.
Please apply my patch to the upcoming Firefox 9 release. Thanks!

Comment 9

[Kohei Yoshino [:kohei]]

Kev: could you review and land my patch when you get a chance?
We are manually repacking and signing the greengoo builds still.

Comment 10

[Kev Needham [:kev]]

Tomcat: can you take this, please?

Comment 11

[Kev Needham [:kev]]

(I'm good with the changes, and Tomcat should be able to add this back into the release workflow so you don't have to do it every time).

Comment 12

[Carsten Book [:Tomcat]]

yeah will do the changes

Comment 13

[Carsten Book [:Tomcat]]

Comment on attachment 548686 [details] [diff] [review]
distribution.ini diff

looks good

Comment 14

[Carsten Book [:Tomcat]]

can we get this into the hg repro ? (don't think i have checkin rights or so - and never did before).

Also Kohei, during the review tests i found also Bug 726948, could you take a look? thanks!

Comment 15

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 548686 [details] [diff] [review]
distribution.ini diff

Landed:
http://hg.mozilla.org/build/partner-repacks/rev/b707cc0195dc

Comment 16

[Kohei Yoshino [:kohei]]

Confirmed: Firefox 10.0.2 was built with the updated distribution.ini. Thank you!

Comment 17

[Carsten Book [:Tomcat]]

thanks nick and kohei! -> verified