Closed Bug 1177545 Opened 6 years ago Closed 1 year ago
Merge what's left of automation
.py .in into remoteautomation .py, remove automation-build .mk
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.
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.
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.
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
Assignee: nobody → gbrown
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/a0758efe4b39 Cleanup automation.py.in; r=egao
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/76d9fb8a0d3d Remove automation.py.in entirely; r=egao,bc
You need to log in before you can comment on or make changes to this bug.