1197874 - Remove MatchstickApp.jsm from the build

Status: RESOLVED FIXED

(Firefox Build System :: Android Studio and Gradle Integration, defect)

Description

[Richard Newman [:rnewman]]

Matchstick isn't going to ship, right? If we don't have any other consumers for this code, we should save the kilobytes.

Comment 1

[Anirudh S [:anirudh24seven]]

Attachment: Remove MatchstickApp.jsm from the build

Comment 2

[Anirudh S [:anirudh24seven]]

I have removed all references to Matchstick. This should be enough, right?

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 8659283 [details] [diff] [review]
Remove MatchstickApp.jsm from the build

Yes. Looks good.

Comment 4

[Richard Newman [:rnewman]]

Anirudh: could you push this to Try before setting checkin-needed?

https://wiki.mozilla.org/ReleaseEngineering/TryServer

Comment 5

[Anirudh S [:anirudh24seven]]

Richard: Sure. Will do so.

I would require someone to vouch for my Level 1 Commit Access though. 
https://bugzilla.mozilla.org/show_bug.cgi?id=1204188

Comment 6

[Richard Newman [:rnewman]]

Vouched.

Comment 7

[Anirudh S [:anirudh24seven]]

I pushed it to Try and it completed successfully.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=29fe2bcd7ff6

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/d9824a8871c8

Comment 9

[Carsten Book [:Tomcat]]

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