• Daniel Bratell's avatar
    jumbo + gn analyze workaround: Add sources to |input| · 97ec3202
    Daniel Bratell authored
    Jumbo, as implemented as a gn template, will naturally hide
    the original files behind new files target_jumbo_1.cc,
    target_jumbo_2.cc, .... This breaks the gn analyze step which is
    used by cq to figure out if it needs to test compile or not.
    
    To give gn analyze a chance to see what is going on, put the original
    sources in the merge action's |input| list. The action won't use
    the files and it will serialize actions that don't need serializing
    but it might be a small price to pay.
    
    There might be unexpected and unwanted side effects from this.
    The cleanest solution would be to give gn native jumbo support
    which would also allow other changes like better IDE support
    and better error messages, but this is what we got.
    
    Bug: 893071
    Change-Id: I36aa498795871a63cfcd4e2d27c29e2f59d5ccbb
    Reviewed-on: https://chromium-review.googlesource.com/c/1455943
    Commit-Queue: Daniel Bratell <bratell@opera.com>
    Reviewed-by: default avatarDirk Pranke <dpranke@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#629971}
    97ec3202
jumbo.gni 10.9 KB