Re: [OE-core] [PATCH v3] qemu: Add qemu-common package

["Yu, Mingli" <mingli.yu@windriver.com>]

Hi Richard,

On 7/19/23 18:08, Richard Purdie wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
> 
> On Wed, 2023-07-19 at 17:39 +0800, Yu, Mingli wrote:
>> On 7/19/23 17:24, Richard Purdie wrote:
>>>>
>>>> It is relevant, this is because of the dependency that this gets built
>>>> and fails. I've seen v4 but didn't have the time to test it yet.
>>>
>>> I've still not seen an answer to Ross' question or mine about why we
>>> can't just add a couple of dependencies and resolve things that way.
>>
>> Sorry for delayed respond!
>>
>> Have added qemu-common to make qemu as meta package to keep backward
>> compatibility. For user who install qemu rpm such as
>> qemu-7.2.0-r0.corei7_64.rpm, it still pull in all of things as before.
>> For user who cares the rpm size, can just choose to install the needed
>> qemu arch rpms such as qemu-system-aarch64-7.2.0-r0.corei7_64.rpm.
> 
> So why not just add:
> 
> RRECOMMENDS:${PN} += "${PN}-system-all ${PN}-user-all"
> 
> as I think we've then covered all the options we need?

If just add RRECOMMENDS:${PN} += "${PN}-system-all ${PN}-user-all" by 
default, then how about the user who want only install 
qemu-system-aarch64-7.2.0-r0.corei7_64.rpm still install all qemu 
binaries as qemu-system-aarch64 rdepends on qemu which RRECOMMENDS 
qemu-system-all and 
qemu-user-all(https://git.openembedded.org/openembedded-core/commit/?id=893846ead7ee54d53e9076150cd655e0c8bca5db).

So it's better make qemu as meta package to keep backward compatibility 
for user who install qemu can install all qemu binaries as before and 
also can meet the need for user who want just install qemu arm64 
emulation rpm such as 
qemu-system-aarch64-7.2.0-r0.corei7_64.rpm(https://patchwork.yoctoproject.org/project/oe-core/patch/20230717071114.2734859-1-mingli.yu@eng.windriver.com/) 
via adding IMAGE_INSTALL:append = " qemu-system-aarch64" into 
conf/local.conf.

Thanks,

> 
> Cheers,
> 
> Richard