696014 - [hu] Remove Bluu search engine from Hungarian builds

Status: VERIFIED FIXED

(Firefox :: General, defect)

Description

[Andras Timar]

Attachment: patch

Remove Bluu search engine from Hungarian Firefox, because their site has been down for several weeks and they do not reposond to e-mails.

Comment 1

[Staś Małolepszy :stas]

Thanks for filing the bug, Andras.  I was out of the office for 2 weeks which is why I didn't comment earlier.

Milos, would you like to take that bug and help Andras?

Andras's patch looks good but it will only affect new installs.  To remove the plugin retroactively from the existing installs, we will need to add the xml file name to browser/installer/removed-files.in in mozilla-central (and possibly aurora and beta too).

CC'ing Axel, too.

Comment 2

[Milos Dinic [:Milos]]

(In reply to Staś Małolepszy :stas from comment #1)
> Milos, would you like to take that bug and help Andras?

Sure!

Hey Andras,

Now that we're removing a search engine, would you like to use this bug and discuss possible replacements for this search engine, or you're good with removing it only, for now?

Comment 3

[Andras Timar]

(In reply to Milos Dinic [:Milos] from comment #2)
Hi Milos,

Just remove it for the time being. I cannot think of a good replacement.

Comment 4

[Milos Dinic [:Milos]]

Andras,

Ok. Can you please implement changes from Stas' comment 1, and attach a new patch?

Comment 5

[Milos Dinic [:Milos]]

Andras, ping. Do you still want this change?

Comment 6

[Andras Timar]

Attachment: remove bluu.xml from all versions during update

Milos, yes, I still want this change. Please apply this patch to central, aurora, and beta. Also, please approve the first patch, I can commit it myself. And please, don't ask for more patches. You know what I want: remove bluu. You know how to do that. Why did I need to checkout the whole repo for a one liner patch? Thanks!

Comment 7

[Milos Dinic [:Milos]]

Comment on attachment 568346 [details] [diff] [review]
patch

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

Looks good, as per previous comments. Please land with appropriate comment, mentioning this bug number, description and my approval(r=milos)

Comment 8

[Milos Dinic [:Milos]]

Comment on attachment 575120 [details] [diff] [review]
remove bluu.xml from all versions during update

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

Thanks for this one, r+. Please land with the same descritpion(possibly in the same commit) as previous patch.

Now, as far as landing and patches are concerned, we prefer localizers have the full control over their repositories, so that's why we mainly don't commit. Of course, if you're really busy, or have problems with your computer, or anything else comes in your way that makes it hard for you to land your patches, I can give you a hand.

Comment 9

[Andras Timar]

I pushed the fix to all Hungarian l10n repos. The second patch is for browser/installer/removed-files.in in mozilla-central (and aurora, and beta), and I don't have write access, so can you please do it for me. Thanks,

Comment 10

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

Comment on attachment 575120 [details] [diff] [review]
remove bluu.xml from all versions during update

I'd put it in between these two
 searchplugins/baidu.src
 searchplugins/bok-NO.gif
to maintain the alphabetical sort. Do you need someone to land this Milos ?

Comment 11

[Staś Małolepszy :stas]

Nick, Axel -- since the change to the Hungarian repo will happen on l10n/mozilla-aurora, should the removed-files.in patch land on both central and aurora?

Comment 12

[Axel Hecht [:Pike]]

Yes, and landing help would be appreciated.

Comment 13

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

Attachment: remove bluu.xml from all versions during update, v2

As landed:
 http://hg.mozilla.org/integration/mozilla-inbound/rev/a4492c6d02b0

Comment 14

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

Hmm, there are no flags here to request approval for aurora or beta. How do you normally handle that Axel ?

Comment 15

[Mounir Lamouri (:mounir)]

https://hg.mozilla.org/mozilla-central/rev/a4492c6d02b0

Comment 16

[Axel Hecht [:Pike]]

(In reply to Nick Thomas [:nthomas] from comment #14)
> Hmm, there are no flags here to request approval for aurora or beta. How do
> you normally handle that Axel ?

Let's just move it to Firefox and use the flags there. There's really no "usually".

Comment 17

[Axel Hecht [:Pike]]

Comment on attachment 576841 [details] [diff] [review]
remove bluu.xml from all versions during update, v2

Requesting approval to land the removed-files.in patch in aurora and beta, to correspond with http://hg.mozilla.org/releases/l10n/mozilla-beta/hu/rev/f96dc0a754fd and http://hg.mozilla.org/releases/l10n/mozilla-aurora/hu/rev/6f67195ee19d

Comment 18

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

FWIW, I have searched mxr's l10n indexes for aurora and beta and found no instances of bluu.xml, so no other locales are using this search engine.

Comment 19

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

Comment on attachment 576841 [details] [diff] [review]
remove bluu.xml from all versions during update, v2

http://hg.mozilla.org/releases/mozilla-aurora/rev/187e2ae243b5
http://hg.mozilla.org/releases/mozilla-beta/rev/b100a8d12b9b

Comment 20

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

All done.

Comment 21

[Mihaela Velimiroviciu (:mihaelav)]

Mozilla/5.0 (Windows NT 6.1; rv:11.0a1) Gecko/20111206 Firefox/11.0a1
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:11.0a1) Gecko/20111206 Firefox/11.0a1
Mozilla/5.0 (X11; Linux x86_64; rv:11.0a1) Gecko/20111206 Firefox/11.0a1
Mozilla/5.0 (Windows NT 5.1; rv:11.0a1) Gecko/20111206 Firefox/11.0a1

Mozilla/5.0 (Windows NT 6.1; rv:10.0a2) Gecko/20111206 Firefox/10.0a2
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:10.0a2) Gecko/20111206 Firefox/10.0a2
Mozilla/5.0 (X11; Linux i686 on x86_64; rv:10.0a2) Gecko/20111206 Firefox/10.0a2
Mozilla/5.0 (Windows NT 5.1; rv:10.0a2) Gecko/20111206 Firefox/10.0a2

Bluu search engine was removed from Nightly and Aurora HU builds (above), but latest Betas (9 beta 4) still have it:
Mozilla/5.0 (Windows NT 6.1; rv:9.0) Gecko/20100101 Firefox/9.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:9.0) Gecko/20100101 Firefox/9.0
Mozilla/5.0 (X11; Linux i686 on x86_64; rv:9.0) Gecko/20100101 Firefox/9.0
Mozilla/5.0 (Windows NT 5.1; rv:9.0) Gecko/20100101 Firefox/9.0

Comment 22

[u279076]

Nick, can you comment on the claim that 9.0 Beta builds do not reflect this change?

Comment 23

[Axel Hecht [:Pike]]

That's consistent with http://hg.mozilla.org/releases/l10n/mozilla-beta/hu/graph/, b4 doesn't have this. B5 should, though.

Comment 24

[u279076]

Mihaela, please retest this once we get 9.0b5 builds later this week.

Comment 25

[Mihaela Velimiroviciu (:mihaelav)]

Mozilla/5.0 (Windows NT 6.1; rv:9.0) Gecko/20100101 Firefox/9.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:9.0) Gecko/20100101 Firefox/9.0
Mozilla/5.0 (Windows NT 5.1; rv:9.0) Gecko/20100101 Firefox/9.0
Mozilla/5.0 (X11; Linux x86_64; rv:9.0) Gecko/20100101 Firefox/9.0

Verified on latest beta(9beta5) and the Bluu search engine was removed from HU builds.
Markiing as VERIFIED.