649078 - Remove WinCE installer code from /mobile/installer/wince/* and related build configs

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect, P5)

Description

[Kevin Brosnan [Ex-Mozilla]]

The code at http://mxr.mozilla.org/mozilla-central/source/mobile/installer/wince/ should be unused.

Comment 1

[Ed Morley [:emorley]]

Can I rm http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/installer/wince/* at the same time too, given it fits with the bug title?

Comment 2

[Ed Morley [:emorley]]

Attachment: Patch (based on m-c rev 0cfe6840e0a4)

Ignore comment 1 - I've just stuck to just rm'ing /mobile/installer/wince/* and adjusting the mobile build configs that refer to the installer; to keep things simpler for now. 

I'll file /toolkit/installer/wince/* changes as a follow-up.

Comment 3

[Kevin Brosnan [Ex-Mozilla]]

I can't review this. Try :dougt he has a master plan for Windows mobile.

Comment 4

[Ed Morley [:emorley]]

Comment on attachment 525270 [details] [diff] [review]
Patch (based on m-c rev 0cfe6840e0a4)

Great - thanks! :-)

Comment 5

[Doug Turner (:dougt)]

Comment on attachment 525270 [details] [diff] [review]
Patch (based on m-c rev 0cfe6840e0a4)

thanks.

Comment 6

[Ed Morley [:emorley]]

Thanks for the speedy review :-)

Comment 7

[(no longer active)]

http://hg.mozilla.org/projects/cedar/rev/31eeb90b53da

Comment 8

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/31eeb90b53da

Comment 9

[Ed Morley [:emorley]]

http://mxr.mozilla.org/mozilla-central/source/mobile/installer/wince/
...directory does not exist.

http://mxr.mozilla.org/mozilla-central/search?string=wince&find=/mobile/
...no matching files found, so corresponding references successfully removed.

-> Verified.