[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits |
Date: |
Tue, 15 Sep 2020 18:58:18 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 15/09/20 16:27, Richard Henderson wrote:
> On 9/14/20 11:27 PM, Paolo Bonzini wrote:
>> Looks good. Can you just add a "# Submodules" heading above the test?
>>
>> I would also like to remove the "yes" value (that is, the default fails if
>> the
>> internal copy is not there) but it can be done later for all submodules.
>
> Unless you simply plan to rename {no, yes} to {disabled, enabled}, as for the
> Feature objects, why?
>
> That seems to be the only sensible value for --enable-foo, without the =system
> or =git specifiers. We *should* fail if no system library nor internal copy
> is
> present.
Yes, that was a bit concise. I would like "auto" to take the meaning
that "yes" currently as. Right now we have
no -> Easy :)
system -> System capstone if found, fail otherwise
internal/git -> Compile capstone if found, fail otherwise
auto -> System capstone, then internal, then disable
yes -> System capstone, then internal, then fail
I'm not sure of the usefulness of disabling a dependency because we
don't have it checked out, since we have the machinery to do the
checkout automatically. So that would become:
no -> Easy :)
system -> System capstone if found, fail otherwise
internal/git -> Compile capstone if found, fail otherwise
auto -> System capstone, then internal, then fail
The disadvantage is that it would be different from other "auto"
symbols, which never fail. But then those other "auto" symbols do not
have a built-in fallback, so the question is whether the combination of
1) building from a fresh git checkout
2) --disable-git-update
3) not having a system capstone/libfdt/slirp
4) not having --disable-{capstone,libfdt,slirp} on the command line
is more likely to be intentional or operator error.
Paolo
- [PATCH v2 00/10] capstone + disassembler patches, Richard Henderson, 2020/09/14
- [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits, Richard Henderson, 2020/09/14
- Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits, Richard Henderson, 2020/09/14
- Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits, Paolo Bonzini, 2020/09/15
- Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits, Richard Henderson, 2020/09/15
- Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits, Yonggang Luo, 2020/09/15
- Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits, Paolo Bonzini, 2020/09/15
- Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits, Yonggang Luo, 2020/09/15
- Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits, Paolo Bonzini, 2020/09/15
- Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits, Yonggang Luo, 2020/09/15
- Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits,
Paolo Bonzini <=
Re: [PATCH v2 01/10] capstone: Convert Makefile bits to meson bits, Thomas Huth, 2020/09/15
[PATCH v2 02/10] capstone: Update to upstream "next" branch, Richard Henderson, 2020/09/14
[PATCH v2 03/10] disas: Move host asm annotations to tb_gen_code, Richard Henderson, 2020/09/14
[PATCH v2 04/10] disas: Clean up CPUDebug initialization, Richard Henderson, 2020/09/14
[PATCH v2 05/10] disas: Use qemu/bswap.h for bfd endian loads, Richard Henderson, 2020/09/14