[OE-core] [PATCH v2] base-passwd: Add kvm group

[Jacob Kroon]

Hi Joshua, Mark,

>> In general, I don't think we should be "patching in" users and groups
>> like this to base-passwd. If upstream doesn't have them, they need to
>> be added through the normal useradd.bbclass mechanism so that users of
>> useradd-staticids can choose what ID they get mapped to.
>>
>> Is it possible to revert this and add the kvm group via the useradd.bbclass?
>>
>>>
>>>    udevd[<pid>]: specified group 'kvm' unknown
> 
> As for this particular patch, why is adding a specific KVM group required?
> Unless something is hardcoded in KVM to use group '47', a regular system type
> useradd call should be done instead.  This would avoid the problem that Joshua
> is experiencing.
> 

The groups "shutdown" and "input" were already patched in in the same 
way, so I figured that was the way to do it. But it sounds like you're 
pointing out that we can drop those three patches, and instead use 
useradd.bbclass in base-passwd recipe ? Sounds like a nice cleanup.

Cheers,
Jacob