• dpranke's avatar
    Revert of Update MB to use `gn analyze`. (patchset #9 id:160001 of... · 10118bff
    dpranke authored
    Revert of Update MB to use `gn analyze`. (patchset #9 id:160001 of https://codereview.chromium.org/2298403002/ )
    
    Reason for revert:
    Need to add "aura_builder", probably others ...
    
    Original issue's description:
    > Update MB to use `gn analyze`.
    >
    > Now that GN supports a form of `analyze` directly, this patch
    > updates //tools/mb to use it rather than working awkwardly through
    > `gn refs`. The MB implementation now is mostly responsible for
    > mapping ninja targets to labels, calling GN, and then mapping returned
    > labels back to ninja targets. Eventually we should update the
    > recipes to just pass labels around instead.
    >
    > One significant change is that now we need entries in gn_isolate_map.pyl
    > for every target listed in 'additional_compile_targets' and
    > 'isolated_scripts' as well as in 'gtest_tests', because we have to remap
    > everything to the GN labels, not just tests for swarming.
    >
    > We should switch everything to using GN labels directly, but that'll require
    > follow-on CLs.
    >
    > R=brettw@chromium.org
    > BUG=555273
    >
    > Committed: https://crrev.com/9aba8b21177df3ed3e897d5c500d845fbf5523e6
    > Cr-Commit-Position: refs/heads/master@{#419315}
    
    TBR=phajdan.jr@chromium.org,brettw@chromium.org,kbr@chromium.org
    # Skipping CQ checks because original CL landed less than 1 days ago.
    NOPRESUBMIT=true
    NOTREECHECKS=true
    NOTRY=true
    BUG=555273
    
    Review-Url: https://codereview.chromium.org/2341403004
    Cr-Commit-Position: refs/heads/master@{#419321}
    10118bff
mb.py 53 KB