1177545 - Merge what's left of automation.py.in into remoteautomation.py, remove automation-build.mk

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Julien Pagès (:parkouss)]

The build/automation.py.inc file is quite a mess, and we should remove the need for it, if possible. :) This is blocking for mochitest for example as we would like to package it properly.

So, quite some work to do - these are the dependencies:

https://dxr.mozilla.org/mozilla-central/search?q=%22from+automation%22&case=true

I propose to make this bug a tracking bug, so we can do the job step by step.

Comment 1

[(not currently active) Ted Mielczarek]

bug 775756 was tracking this overarching work.

Comment 2

[Julien Pagès (:parkouss)]

So, do you want to mark this bug as blocking bug 775756, or close this as WONTFIX or DUPLICATE ?

Comment 3

[(not currently active) Ted Mielczarek]

It's up to you. Removing automation.py.in is definitely something we want, but I'd also rather have bugs on file to fix the specific issues with it rather than just a generic "remove it" bug.

Comment 4

[Jonathan Griffin (:jgriffin)]

This is being used as a tracking bug, which I think is arguably a prerequisite to bug 775756, so marking this as blocking.

Comment 5

[(not currently active) Ted Mielczarek]

We removed all uses of automation.py from desktop automation. It's still used in mobile automation, but fixing that is a fair amount of work. We can simplify some build system gunk by just moving the contents of automation.py.in into remoteautomation.py  since we no longer need the preprocessor interpolation.

Comment 6

[Nicholas Nethercote [:njn] (On Long Service Leave)]

As far as I can tell, some or all of automation.py.in is dead code. Automation.runApp is definitely dead, and if you remove that a lot of other code can be removed too.

Comment 7

[Nicholas Nethercote [:njn] (On Long Service Leave)]

Here's a try push where I added a raise Exception("blah") to the start of runApp, and it appears to have run just fine: https://treeherder.mozilla.org/#/jobs?repo=try&revision=17c835c2a31e9a6519c9ed9b102a4edf63df8e74

Comment 8

[Geoff Brown [:gbrown]]

Thanks! I'll see if I can move this forward.

Comment 9

[Geoff Brown [:gbrown]]

Attachment: Bug 1177545 - Cleanup automation.py.in; r=

Remove unused code from automation.py.in, move some android-specific code from
automation.py.in to remoteautomation.py, and eliminate some other easily-replaced
code. In the long term we want to eliminate automation.py.in completely; I may
attempt that once these changes have landed.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a0758efe4b39
Cleanup automation.py.in; r=egao

Comment 11

[Andrei Ciure[:aciure]]

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

Comment 12

[Geoff Brown [:gbrown]]

Attachment: Bug 1177545 - Remove automation.py.in entirely; r=

Remove the remainder of automation.py.in and the build support for generating
automation.py.
Some of this functionality was in use, especially for android tests.
Some code was moved or re-implemented in remoteautomation.py or in the affected
harness(es). Some features were removed: There are some minor changes in behavior.
For instance, instead of using a different server startup timeout for debug builds,
one value is used for all builds (due to performance improvements over time,
the longer timeout is no longer needed).

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/76d9fb8a0d3d
Remove automation.py.in entirely; r=egao,bc

Comment 14

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/76d9fb8a0d3d