]> xenbits.xensource.com Git - libvirt.git/commit
ci: build.sh: Join MESON_ARGS and MESON_OPTS
authorErik Skultety <eskultet@redhat.com>
Wed, 1 Feb 2023 14:22:59 +0000 (15:22 +0100)
committerErik Skultety <eskultet@redhat.com>
Fri, 11 Aug 2023 13:11:50 +0000 (15:11 +0200)
commit9c9848f955fd2b42e7f258b945cd90aacd2c114e
treefa11ee15ad56070ba3c7f95f161e43a583189427
parent6788b2d3cde7b3738dfff666facee2735218a273
ci: build.sh: Join MESON_ARGS and MESON_OPTS

It is quite confusing seeing these two in a call like this one:
    $ meson build $MESON_OPTS $MESON_ARGS

One has to ask 'how are they different' and 'shouldn't these be
merged'. In fact, these variables hold very different things and we
should make it more obvious. The problem is that renaming MESON_OPTS to
something more meaningful, like 'MESON_CROSS_OPTS' which is what
MESON_OPTS really does would require changes to lcitool and would
impact Dockerfile generation which in turn might have an impact on
other projects which rely on this lcitool functionality which is risky.

Instead, provide a docstring for the former to supplement the latter
and join the two variables in a single one MESON_ARGS which is then
passed to meson's command line so it's a little less confusing.

Signed-off-by: Erik Skultety <eskultet@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
ci/build.sh