• lukasza's avatar
    Split run_tool.py into run_tool.py, extract_edits.py and apply_edits.py · f9b89e7b
    lukasza authored
    The split will allow generation of edits on multiple configs (e.g. linux
    vs windows OR rel vs dbg) and merging the edits before applying them once:
        $ tools/clang/scripts/run_tool.py rewrite_to_chrome_style \
            --generate-compdb --all out/rel >run_tool.linux.rel.out
        $ ...
        $ cat run_tool.*.out \
            | tools/clang/scripts/extract_edits.py \
            | tools/clang/scripts/apply_edits.py
            --generate-compdb --all out/rel >run_tool.linux.rel.out
    
    Test steps:
    - tools/clang/translation_unit/test_translation_unit.py
    - tools/clang/scripts/test_tool.py rewrite_to_chrome_style
    - manually running run_tool | extract_edits | apply_edits pipeline
      on WTF and verifying that it still builds after the rename
    
    BUG=598138
    TEST=See "Test steps" above.
    
    Review-Url: https://codereview.chromium.org/2599193002
    Cr-Commit-Position: refs/heads/master@{#440881}
    f9b89e7b
apply_edits.py 6.75 KB