[meta-networking][PATCH] postfix: do NOT create directory /var/spool/mail

[meta-networking][PATCH] postfix: do NOT create directory /var/spool/mail

[kai.kang]

From: Kai Kang <kai.kang@windriver.com>

postfix installs directory /var/spool/mail which conflicts with package
shadow.

| Error: Transaction check error:
|  file /var/spool/mail conflicts between attempted installs of
|  postfix-3.1.1-r0.ppc64e6500 and shadow-4.2.1-r0.1.ppc64e6500

Make postfix not create /var/spool/mail that it does NOT create the
directory on Fedora and Ubuntu either.

Signed-off-by: Kai Kang <kai.kang@windriver.com>
---
 meta-networking/recipes-daemons/postfix/postfix.inc | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta-networking/recipes-daemons/postfix/postfix.inc b/meta-networking/recipes-daemons/postfix/postfix.inc
index 5bb2491..4c794ff 100644
--- a/meta-networking/recipes-daemons/postfix/postfix.inc
+++ b/meta-networking/recipes-daemons/postfix/postfix.inc
@@ -164,8 +164,6 @@ do_install () {
     install -m 770 -d ${D}${localstatedir}/spool/postfix
     chown postfix:postfix ${D}${localstatedir}/spool/postfix
 
-    install -m 2755 -d ${D}${localstatedir}/spool/mail
-    chown postfix:nogroup ${D}${localstatedir}/spool/mail
     install -m 0755 -d ${D}${localstatedir}/lib/postfix
     chown postfix:nogroup ${D}${localstatedir}/lib/postfix
     install -m 0755 -d ${D}${localstatedir}/spool/postfix
-- 
2.10.1

Re: [meta-networking][PATCH] postfix: do NOT create directory /var/spool/mail

[Mark Hatle]

On 4/9/17 9:18 PM, kai.kang@windriver.com wrote:
> From: Kai Kang <kai.kang@windriver.com>
> 
> postfix installs directory /var/spool/mail which conflicts with package
> shadow.
> 
> | Error: Transaction check error:
> |  file /var/spool/mail conflicts between attempted installs of
> |  postfix-3.1.1-r0.ppc64e6500 and shadow-4.2.1-r0.1.ppc64e6500
> 
> Make postfix not create /var/spool/mail that it does NOT create the
> directory on Fedora and Ubuntu either.

I am not sure this is correct.

The two packages have (according to DNF/RPM) different permissions, owner or
group assigned to that particular directory.

I think it's reasonable for both of the packages to create the directory,
however they must match in perms/owner/group.  I would defer to shadow as the
correct source for this information.

It may be necessary though to change shadow's default to something more
reasonable if 'root:root' is not right.

--Mark

> Signed-off-by: Kai Kang <kai.kang@windriver.com>
> ---
>  meta-networking/recipes-daemons/postfix/postfix.inc | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/meta-networking/recipes-daemons/postfix/postfix.inc b/meta-networking/recipes-daemons/postfix/postfix.inc
> index 5bb2491..4c794ff 100644
> --- a/meta-networking/recipes-daemons/postfix/postfix.inc
> +++ b/meta-networking/recipes-daemons/postfix/postfix.inc
> @@ -164,8 +164,6 @@ do_install () {
>      install -m 770 -d ${D}${localstatedir}/spool/postfix
>      chown postfix:postfix ${D}${localstatedir}/spool/postfix
>  
> -    install -m 2755 -d ${D}${localstatedir}/spool/mail
> -    chown postfix:nogroup ${D}${localstatedir}/spool/mail
>      install -m 0755 -d ${D}${localstatedir}/lib/postfix
>      chown postfix:nogroup ${D}${localstatedir}/lib/postfix
>      install -m 0755 -d ${D}${localstatedir}/spool/postfix
> 

Re: [meta-networking][PATCH] postfix: do NOT create directory /var/spool/mail

[Kang Kai]

On 2017年04月13日 03:32, Mark Hatle wrote:
> On 4/9/17 9:18 PM, kai.kang@windriver.com wrote:
>> From: Kai Kang <kai.kang@windriver.com>
>>
>> postfix installs directory /var/spool/mail which conflicts with package
>> shadow.
>>
>> | Error: Transaction check error:
>> |  file /var/spool/mail conflicts between attempted installs of
>> |  postfix-3.1.1-r0.ppc64e6500 and shadow-4.2.1-r0.1.ppc64e6500
>>
>> Make postfix not create /var/spool/mail that it does NOT create the
>> directory on Fedora and Ubuntu either.
> I am not sure this is correct.
>
> The two packages have (according to DNF/RPM) different permissions, owner or
> group assigned to that particular directory.
>
> I think it's reasonable for both of the packages to create the directory,
> however they must match in perms/owner/group.  I would defer to shadow as the
> correct source for this information.

Do you mean use to update-alternative to keep them both?

>
> It may be necessary though to change shadow's default to something more
> reasonable if 'root:root' is not right.

I checked on both Fedora and Ubuntu that /var/spool/mail is set with 
'root:mail'.  I will update it in recipe shadow.
And it also set with  'setgid' on Ubuntu, do we need 'setgid' too?

Thanks.



--Kai


>
> --Mark
>
>> Signed-off-by: Kai Kang <kai.kang@windriver.com>
>> ---
>>   meta-networking/recipes-daemons/postfix/postfix.inc | 2 --
>>   1 file changed, 2 deletions(-)
>>
>> diff --git a/meta-networking/recipes-daemons/postfix/postfix.inc b/meta-networking/recipes-daemons/postfix/postfix.inc
>> index 5bb2491..4c794ff 100644
>> --- a/meta-networking/recipes-daemons/postfix/postfix.inc
>> +++ b/meta-networking/recipes-daemons/postfix/postfix.inc
>> @@ -164,8 +164,6 @@ do_install () {
>>       install -m 770 -d ${D}${localstatedir}/spool/postfix
>>       chown postfix:postfix ${D}${localstatedir}/spool/postfix
>>   
>> -    install -m 2755 -d ${D}${localstatedir}/spool/mail
>> -    chown postfix:nogroup ${D}${localstatedir}/spool/mail
>>       install -m 0755 -d ${D}${localstatedir}/lib/postfix
>>       chown postfix:nogroup ${D}${localstatedir}/lib/postfix
>>       install -m 0755 -d ${D}${localstatedir}/spool/postfix
>>
>


-- 
Regards,
Neil | Kai Kang