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

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

Hi Richard and Ross,

On 7/19/23 17:24, 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 11:20 +0200, Alexandre Belloni via
> lists.openembedded.org wrote:
>> On 19/07/2023 17:10:37+0800, Yu, Mingli wrote:
>>> Hi Alex,
>>>
>>> On 7/17/23 20:46, Alexandre Belloni 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 17/07/2023 15:10:35+0800, Yu, Mingli wrote:
>>>>> Hi Alex,
>>>>>
>>>>> On 7/16/23 19:47, Alexandre Belloni 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.
>>>>>>
>>>>>> Hello,
>>>>>>
>>>>>> This causes the following meta-mingw error on the AB:
>>>>>>
>>>>>> https://autobuilder.yoctoproject.org/typhoon/#/builders/89/builds/7501/steps/12/logs/stdio
>>>>>
>>>>> I didn't find the core-image-mingw-sdktest recipe which I noticed in the
>>>>> above log, so I cannot reproduce the issue as you mentioned.
>>>>
>>>> As stated above, this is part of meta-mingw:
>>>>
>>>> https://git.yoctoproject.org/meta-mingw/tree/recipes-core/images?h=master-next
>>>
>>> Thanks for your pointer!
>>>
>>> BTW, the below error which in https://autobuilder.yoctoproject.org/typhoon/#/builders/89/builds/7501/steps/12/logs/stdio
>>> seems irrelevant to the added nativesdk dependency though I have removed the
>>> nativesdk dependency in https://patchwork.yoctoproject.org/project/oe-core/patch/20230717071114.2734859-1-mingli.yu@eng.windriver.com/.
>>>
>>>> /home/pokybuild/yocto-worker/meta-mingw/build/build/tmp/work/i686-nativesdk-mingw32-w64-mingw32/nativesdk-ncurses/6.4-r0/recipe-sysroot-native/usr/bin/i686-w64-mingw32/../../libexec/i686-w64-mingw32/gcc/i686-w64-mingw32/13.1.1/ld:
>>> ../obj_s/lib_kernel.o:lib_kernel.c:(.text+0x5c): undefined reference to
>>> `_nc_mingw_tcflush'
>>>> /home/pokybuild/yocto-worker/meta-mingw/build/build/tmp/work/i686-nativesdk-mingw32-w64-mingw32/nativesdk-ncurses/6.4-r0/recipe-sysroot-native/usr/bin/i686-w64-mingw32/../../libexec/i686-w64-mingw32/gcc/i686-w64-mingw32/13.1.1/ld:
>>> ../obj_s/lib_ttyflags.o:lib_ttyflags.c:(.text+0x2c): undefined reference to
>>> `_nc_mingw_tcgetattr'
>>>> /home/pokybuild/yocto-worker/meta-mingw/build/build/tmp/work/i686-nativesdk-mingw32-w64-mingw32/nativesdk-ncurses/6.4-r0/recipe-sysroot-native/usr/bin/i686-w64-mingw32/../../libexec/i686-w64-mingw32/gcc/i686-w64-mingw32/13.1.1/ld:
>>> ../obj_s/lib_ttyflags.o:lib_ttyflags.c:(.text+0xbb): undefined reference to
>>> `_nc_mingw_tcsetattr'
>>>
>>
>> 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.

Thanks,

> 
> Cheers,
> 
> Richard