Open Bug 1177545 Opened 4 years ago Updated 2 years ago

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

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(firefox41 affected)

Tracking Status
firefox41 --- affected

People

(Reporter: parkouss, Unassigned)

References

(Blocks 2 open bugs)

Details

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.
Depends on: 1177552
Depends on: 1177780
Blocks: 1178158
bug 775756 was tracking this overarching work.
So, do you want to mark this bug as blocking bug 775756, or close this as WONTFIX or DUPLICATE ?
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.
This is being used as a tracking bug, which I think is arguably a prerequisite to bug 775756, so marking this as blocking.
Blocks: 775756
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.
Blocks: nomakefiles
Component: Mozbase → Build Config
Product: Testing → Core
Summary: clean automation.py.inc and hopefully remove it → Merge what's left of automation.py.in into remoteautomation.py, remove automation-build.mk
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.