1282866 - Remove widget/qt from mozilla-central for lack of strong ownership

Status: RESOLVED FIXED

(Core Graveyard :: Widget: Qt, defect)

Description

[Benjamin Smedberg]

See https://groups.google.com/forum/#!topic/mozilla.dev.platform/ZlGQ__zZy28 for prior discussion.

We need not just active code peers, but also active bug triagers and some commitment to continuous integration. Because nobody is committing to that, we're going to remove this code from the main tree and interested parties can maintain it separately if desired.

Comment 1

[Benjamin Smedberg]

Attachment: Bug 1282866 - remove widget/qt and other supporting QT code, This patch does not remove all of the checks for MOZ_WIDGET_QT (which are dead code), but that will be a followup mentored bug.

Review commit: https://reviewboard.mozilla.org/r/62860/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/62860/

Comment 2

[Doug Turner (:dougt)]

Comment on attachment 8768845 [details]
Bug 1282866 - remove widget/qt and other supporting QT code,  This patch does not remove all of the checks for MOZ_WIDGET_QT (which are dead code), but that will be a followup mentored bug.

https://reviewboard.mozilla.org/r/62860/#review59766

lgtm.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/cdbdbb4df09a
remove widget/qt and other supporting QT code, r=dougt.  This patch does not remove all of the checks for MOZ_WIDGET_QT (which are dead code), but that will be a followup mentored bug.

Comment 4

[Benjamin Smedberg]

In bug 1282866, I have removed the QT code from the Firefox tree with the approval of the active peer. That code is not currently maintained by any team.

I have looked through the bugs in the Widget: Qt bugzilla component and I believe all of these are no longer relevant because the Qt code has been removed. If you believe that this bug is still valid, please move it to another more appropriate bugzilla component as you reopen it.

Comment 5

[Benjamin Smedberg]

Except for this one, duh.

Comment 6

[Carsten Book [:Tomcat]]

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