DescriptionSupport spaces in Mac GN build output names.
We can't use object paths with spaces in them due to a mismatch between Ninja and the Mac LD's filelist parameter (explained in more detail in a comment added in the mac toolchain GN file). In order to support output_names with spaces, the object file directory must be based on something else.
Adds a new substitution "{{target_label}}" to the tool that expands to the target name after the colon, not overridden by the output_name. This makes more sense from a directory structure perspective, and label names won't have spaces (they can but don't, and with this change won't work on Mac).
Also adds quoting for Mac tool commands so spaces will work in names.
I made the change in the Mac toolchain definition for testing, but it is currently commented out pending the binary roll. After the binary roll, we should uncomment this and update the other toolchains to match.
BUG=546894
Committed: https://crrev.com/782b1bc8562e9adbb1ea7d98929d6709ff7290b9
Cr-Commit-Position: refs/heads/master@{#358792}
Patch Set 1 #
Messages
Total messages: 13 (4 generated)
|