On 2/8/19 9:43 AM, Alexander Kanavin wrote:
> Patch 3 changes things in meta-poky/, and patch 4 changes things in
> meta/, so they cannot be combined unfortunately. I can however swap
> their order, so that there is less confusion.
Swap the order and add a copy of the commit message (from 4 to 3) then. That
should lesson anyone's questions if they need to figure out why it changed in
the future.
Thanks!
--Mark
> Alex
>
> On Fri, 8 Feb 2019 at 16:31, Mark Hatle <mark.ha...@windriver.com> wrote:
>>
>> I may be nitpicking slightly, but this patch 4 and 3 should probably be
>> combined.. I looked at '3' and went "this looks wrong, I have no idea why"..
>> then saw '4' and it became clear. Since they're both tied together (same
>> purpose) it may make sense to combine them...
>>
>> On 2/8/19 8:45 AM, Alexander Kanavin wrote:
>>> As sdl is deprecated in favor of gtk+, it should not be the default.
>>
>> At a minimum the commit message above should be included in 3/12.
>>
>> --Mark
>>
>>> Signed-off-by: Alexander Kanavin <alex.kana...@gmail.com>
>>> ---
>>> meta/recipes-devtools/qemu/qemu.inc | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/meta/recipes-devtools/qemu/qemu.inc
>>> b/meta/recipes-devtools/qemu/qemu.inc
>>> index 95e9e0ede77..07f78058e7b 100644
>>> --- a/meta/recipes-devtools/qemu/qemu.inc
>>> +++ b/meta/recipes-devtools/qemu/qemu.inc
>>> @@ -71,11 +71,11 @@ do_install_append() {
>>> # END of qemu-mips workaround
>>>
>>> PACKAGECONFIG ??= " \
>>> - fdt sdl kvm \
>>> + fdt gtk+ kvm \
>>> ${@bb.utils.filter('DISTRO_FEATURES', 'alsa xen', d)} \
>>> "
>>> PACKAGECONFIG_class-native ??= "fdt alsa kvm virglrenderer glx"
>>> -PACKAGECONFIG_class-nativesdk ??= "fdt sdl kvm virglrenderer glx"
>>> +PACKAGECONFIG_class-nativesdk ??= "fdt kvm virglrenderer glx"
>>>
>>> # Handle distros such as CentOS 5 32-bit that do not have kvm support
>>> PACKAGECONFIG_class-native_remove = "${@'kvm' if not
>>> os.path.exists('/usr/include/linux/kvm.h') else ''}"
>>>
>>
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
