Closed Bug 1906540 Opened 2 years ago Closed 1 year ago

Remove mozrunner devices and mozdevice from SeaMonkey

Categories

(SeaMonkey :: General, task)

SeaMonkey 2.53

Tracking

(seamonkey2.53+ fixed)

RESOLVED FIXED
Tracking Status
seamonkey2.53 + fixed

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

Details

(Whiteboard: SM2.53.20)

Attachments

(2 files, 3 obsolete files)

We don't need mozrunner devices or mozdevice in the SeaMonkey tree so remove that code.

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: none
Testing completed (on m-c, etc.): 2.53.20
Risk to taking this patch (and alternatives if risky): low
String changes made by this patch: none

Attachment #9411432 - Flags: review?(frgrahl)
Attachment #9411432 - Flags: approval-comm-release?

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: none
Testing completed (on m-c, etc.): 2.53.20
Risk to taking this patch (and alternatives if risky): low
String changes made by this patch: none

Attachment #9411433 - Flags: review?(frgrahl)
Attachment #9411433 - Flags: approval-comm-release?

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: none
Testing completed (on m-c, etc.): 2.53.20
Risk to taking this patch (and alternatives if risky): low
String changes made by this patch: none

Attachment #9411434 - Flags: review?(frgrahl)
Attachment #9411434 - Flags: approval-comm-release?
Attached patch check.patch (obsolete) — — Splinter Review

I found a few more references and als removed part of Bug 787203. Suspect it will not not work without additinalremovals. Could you check and add to the other patches what you think can be used.

Flags: needinfo?(iannbugzilla)

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: none
Testing completed (on m-c, etc.): 2.53.20
Risk to taking this patch (and alternatives if risky): low
String changes made by this patch: none
Note: def check_output is used by other parts of jittests.py so kept it

Attachment #9411432 - Attachment is obsolete: true
Attachment #9411433 - Attachment is obsolete: true
Attachment #9411439 - Attachment is obsolete: true
Attachment #9411432 - Flags: review?(frgrahl)
Attachment #9411432 - Flags: approval-comm-release?
Attachment #9411433 - Flags: review?(frgrahl)
Attachment #9411433 - Flags: approval-comm-release?
Flags: needinfo?(iannbugzilla)
Attachment #9411445 - Flags: review?(frgrahl)
Attachment #9411445 - Flags: approval-comm-release?
Whiteboard: SM2.53.20

Comment on attachment 9411434 [details] [diff] [review]
TOP-1906540-mozdevice-removal-comm-25320.patch

lgtm r/a+ for our 2.53 branch

Attachment #9411434 - Flags: review?(frgrahl)
Attachment #9411434 - Flags: review+
Attachment #9411434 - Flags: approval-comm-release?
Attachment #9411434 - Flags: approval-comm-release+

Comment on attachment 9411445 [details] [diff] [review]
TOP-1906540-mozdevice-removal-mozilla-25320.patch

lgtm r/a+ for our 2.53 branch

Attachment #9411445 - Flags: review?(frgrahl)
Attachment #9411445 - Flags: review+
Attachment #9411445 - Flags: approval-comm-release?
Attachment #9411445 - Flags: approval-comm-release+

Fixed in our 2.53 branches only.

https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/a49337c37218cc46fb2e8f49a1cea1b5f4a10e3a
Remove mozrunner devices and mozdevice from SeaMonkey - mozdevice part in comm. r=frg a=frg

https://gitlab.com/seamonkey-project/seamonkey-2.53-mozilla/-/commit/c8e5b204f280543aa0b34944d44c9102bc14d79d
Remove mozrunner devices and mozdevice from SeaMonkey. r=frg a=frg

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Version: SeaMonkey 2.53 Branch → SeaMonkey 2.53
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: