[Buildroot] [PATCH 1/2] system/skeleton: add mail group

[Luca Ceresoli <luca@lucaceresoli.net>]

Hi Yann,

Yann E. MORIN wrote:
> Luca, All,
>
> On 2014-03-06 17:54 +0100, Luca Ceresoli spake thusly:
>> ---
>>   system/skeleton/etc/group | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/system/skeleton/etc/group b/system/skeleton/etc/group
>> index 864d1db..85a1fa2 100644
>> --- a/system/skeleton/etc/group
>> +++ b/system/skeleton/etc/group
>> @@ -6,6 +6,7 @@ adm:x:4:
>>   tty:x:5:
>>   disk:x:6:
>>   lp:x:7:
>> +mail:x:8:
>>   kmem:x:9:
>>   wheel:x:10:root
>>   cdrom:x:11:
>
> There is no reason to add the 'mail' group to the default skeleton. If
> it is missing, 'mkuser' is expected to create it (if not, that's a bug.)
>
> So the EXIM_USER in your second patch should be enough.

Technically speaking you're perfectly right.

I just felt the mail group is one of those groups that are supposed to
be used by multiple packages. At least an MTA and a mail server, maybe
a spam filter, a virus scanner... and we might have multiple
alternatives for each of them.

So it looks bad to me to refer to the same group in (potentially) many
packages and not having the group well-defined in "the" group list.

Of course this is my POV, and it's not very strong now that we have such
a small number of mail-related packages in Buildroot...

Oh, following my idea I could as well change:

  define EXIM_USERS
-exim 88 mail 8 * - - - exim
+exim 88 mail -1 * - - - exim
  endef

and your script would get the group ID, right?

-- 
Luca