diff options
author | Eli Schwartz <eschwartz@archlinux.org> | 2021-02-22 20:03:07 -0500 |
---|---|---|
committer | Eli Schwartz <eschwartz@archlinux.org> | 2021-02-23 19:37:03 -0500 |
commit | 691eb0250a58fa8e63a2cf6b256043a5479c2722 (patch) | |
tree | afafa5bda0ce230f4d6d6dcfc23067c71ee2b8f3 /mesonbuild/cmake/common.py | |
parent | 671647188ce0caf6f734a898ff8ae6784f556c25 (diff) | |
download | meson-691eb0250a58fa8e63a2cf6b256043a5479c2722.zip meson-691eb0250a58fa8e63a2cf6b256043a5479c2722.tar.gz meson-691eb0250a58fa8e63a2cf6b256043a5479c2722.tar.bz2 |
mcompile: make sure arguments are passed in the correct order
meson compile itself doesn't permit GNU-style argument permutation, i.e.
TARGET to precede options, so why should we expect ninja to?
And indeed, ninja doesn't document support for this -- but it does
accept it anyway, which is confusing and results in people thinking it's
"supposed to" work.
However, if NINJA=samu then this is in fact enforced. samu does not
permit GNU-style argument permutation. As a result, the arguments passed
to mcompile are actively re-ordered before being passed to the
subprocess, and samu dies with a fatal error.
Fix ordering in mcompile.py and add a comment to warn future readers
that the order does, in fact, matter.
Diffstat (limited to 'mesonbuild/cmake/common.py')
0 files changed, 0 insertions, 0 deletions