Created attachment 498207 [details] [diff] [review] patch to bring device manager up to speed with mozilla central (1.0) currently we need to disable crashreporter and this requires us adding environment variables into the command that we send to android. instead of: 'exec org.mozilla.fennec -profile /tmp/profile http://my/test.html' it needs to be: 'exec "MOZ_CRASHREPORTER_NO_REPORT=1" org.mozilla.fennec -profile /tmp/profile http://my/test.html'
Created attachment 498208 [details] [diff] [review] support android style environment variables from the .config file (1.0) secondary patch which fixes the talos code to support the environment variables on the command line which android needs. These will be running on talos staging
Assignee: nobody → jmaher
Attachment #498208 - Flags: review?(anodelman)
Created attachment 498336 [details] [diff] [review] patch to bring device manager up to speed with mozilla central (1.5) updated patch to fix a few devicemanager specific things.
Created attachment 498337 [details] [diff] [review] support android style environment variables from the .config file (1.5) moved stuff out of here into the devicemanager patch to keep it unique. Also added in a new ENV variable that mobile supports as well as a cleanup on the run_tests.py application.ini discovery.
updated both patches. The original patches were green on try! I am running these new patches on try and we will see the results. I will have to update the results in this bug this weekend as I will be AFK the majority of today.
Please update when you get a green run through so that I can complete my review.
Attachment #498336 - Flags: review?(anodelman) → review+
Attachment #498337 - Flags: review?(anodelman) → review+
Just a general comment that we may need to clean up in future the general case setting of env vars from the specific remote setting - instead of just having one thing called 'env' might be smarter to split into local_env and remote_env and then handle those things separately. Could just become confusing when pretty much the same config file has different results depending on if you are running remote or not.
as a note, these passed talos staging all green.
landed: http://hg.mozilla.org/build/talos/pushloghtml?changeset=3efbac1f685a http://hg.mozilla.org/build/talos/pushloghtml?changeset=aa376673fab5 I filed bug 620104 to track the remote vs local environment variable issue.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.