tree ecc24e08b13914645df7868a611aa9bc877f44a9
parent 4d869a142aa866b26bbd29c4a8fbc6ba2457bfe8
author Oliver Newman <olivernewman@google.com> 1638891695 +0000
committer Commit Bot <commit-bot@chromium.org> 1638891695 +0000

Reland "[run_script] Stop using api.python()"

This reverts commit 3733f83f5a9ecdeb73de5f8ce924bd68d6eb4982.

Reason for revert: relanding after fixing the firmware repo to always
use the correct vpython spec for all scripts.

Original change's description:
> Revert "[run_script] Stop using api.python()"
>
> This reverts commit e0c129d2aee1e23f2e8a8767e99fc8b2820bc435.
>
> Reason for revert: Broke the firmware builders :(
> https://ci.chromium.org/b/8828825951656567969
> Seems like the Python script invokes a tree of python subprocesses
> and they all implicitly rely on vpython. I'll look into fixing that 
> next week.
>
> Original change's description:
> > [run_script] Stop using api.python()
> >
> > If a script relies on vpython then it should explicitly use a vpython
> > shebang, which will keep this recipe simple and also help with locally
> > reproducing builds since infra won't hold the script differently than a
> > local developer would.
> >
> > The only builders that rely on this feature are the firmware builders,
> > so before I land this I'll send changes to update those builders'
> > scripts to use vpython shebangs.
> >
> > Bug: 89307
> > Change-Id: I71f48db5b84f52ead4c3713d61abeff54fb44040
> > Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/610781
> > Reviewed-by: Anthony Fandrianto <atyfto@google.com>
> > Commit-Queue: Oliver Newman <olivernewman@google.com>
>
> Bug: 89307
> Change-Id: I97b7dfd45ceda94dbc0b346dd542f4993dd99455
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/613969
> Reviewed-by: RubberStamper 🤖 <android-build-ayeaye@system.gserviceaccount.com>
> Commit-Queue: Oliver Newman <olivernewman@google.com>

Bug: 89307
Change-Id: I21dc65ac5f688e624566bb5f7e042db9e84500b0
Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/614861
Reviewed-by: RubberStamper 🤖 <android-build-ayeaye@system.gserviceaccount.com>
Reviewed-by: Anthony Fandrianto <atyfto@google.com>
Commit-Queue: Oliver Newman <olivernewman@google.com>