Home | Docs | Forums | Lists | Bugs | Planet | Store | GMN | Get Gentoo!

Full Text Bug Listing

postfix 2.5.0 introduced $data_directory.  2.5.1 ebuild does:
[...]
    dodir /var/lib/postfix
    keepdir /var/lib/postfix
    fowners postfix:postfix /var/lib/postfix
    fperms 0750 /var/lib/postfix
[...]
which results in a root owned keepdir file which in turn results in:

# postfix check
postfix/postfix-script: warning: not owned by postfix:
/var/lib/postfix/./.keep_mail-mta_postfix-0

One way to get rid of the warning would be to add:

fowners postfix:postfix /var/lib/postfix/.keep_${CATEGORY}_${PN}-${SLOT} 

after the keepdir /var/lib/postifx command.

Btw, do we need to keepdir() this directory (just asking) as it contains:
"...files that maintained by single-writer
Postfix process: the address verification cache that is maintained
by the verify service; the pseudo-random pool and TLS session keys
that are maintained by the tlsmgr service; and other cache files
that are maintained by the proxywrite service."

Reproducible: Always



Expected Results:  
no warnings after upgrade/installation

# emerge --info
Portage 2.1.4.4 (default-linux/x86/2007.0, gcc-4.1.2, glibc-2.6.1-r0,
2.6.23-gentoo-r6 i686)
=================================================================
System uname: 2.6.23-gentoo-r6 i686 Pentium III (Coppermine)
Timestamp of tree: Sun, 23 Mar 2008 05:17:01 +0000
app-shells/bash:     3.2_p17-r1
dev-lang/python:     2.4.4-r9
dev-python/pycrypto: 2.0.1-r6
sys-apps/baselayout: 1.12.11.1
sys-apps/sandbox:    1.2.18.1-r2
sys-devel/autoconf:  2.13, 2.61-r1
sys-devel/automake:  1.4_p6, 1.5, 1.6.3, 1.7.9-r1, 1.8.5-r3, 1.9.6-r2, 1.10
sys-devel/binutils:  2.18-r1
sys-devel/gcc-config: 1.4.0-r4
sys-devel/libtool:   1.5.26
virtual/os-headers:  2.6.23-r3
ACCEPT_KEYWORDS="x86"
CBUILD="i686-pc-linux-gnu"
CFLAGS="-march=pentium3 -O2 -pipe"
CHOST="i686-pc-linux-gnu"
CONFIG_PROTECT="/etc /var/bind"
CONFIG_PROTECT_MASK="/etc/env.d /etc/gconf /etc/php/apache2-php5/ext-active/
/etc/php/cgi-php5/ext-active/ /etc/php/cli-php5/ext-active/ /etc/revdep-rebuild
/etc/terminfo /etc/udev/rules.d"
CXXFLAGS="-march=pentium3 -O2 -pipe"
DISTDIR="/usr/portage/distfiles"
FEATURES="distlocks metadata-transfer sandbox sfperms strict unmerge-orphans
userfetch"
GENTOO_MIRRORS="ftp://distro.ibiblio.org/pub/linux/distributions/gentoo/
ftp://ftp-stud.fht-esslingen.de/pub/Mirrors/gentoo/
ftp://ftp.ntua.gr/pub/linux/gentoo/ ftp://ftp.linux.org.tr/pub/mirrors/gentoo/"
MAKEOPTS="-j2"
PKGDIR="/usr/portage/packages"
PORTAGE_RSYNC_OPTS="--recursive --links --safe-links --perms --times --compress
--force --whole-file --delete --stats --timeout=180 --exclude=/distfiles
--exclude=/local --exclude=/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY="/usr/local/portage"
SYNC="rsync://rsync.europe.gentoo.org/gentoo-portage"
USE="acl apache2 berkdb bzip2 cjk cli cracklib crypt dri fortran gdbm gpm iconv
imap isdnlog ldap libwww maildir midi mudflap mysql ncurses nls nptl nptlonly
openmp pam pcre perl pppd python readline reflection sasl session spl ssl tcpd
unicode usb x86 xorg zlib" ALSA_CARDS="ali5451 als4000 atiixp atiixp-modem
bt87x ca0106 cmipci emu10k1 emu10k1x ens1370 ens1371 es1938 es1968 fm801
hda-intel intel8x0 intel8x0m maestro3 trident usb-audio via82xx via82xx-modem
ymfpci" ALSA_PCM_PLUGINS="adpcm alaw asym copy dmix dshare dsnoop empty extplug
file hooks iec958 ioplug ladspa lfloat linear meter mulaw multi null plug rate
route share shm softvol" APACHE2_MODULES="actions alias auth_basic auth_digest
authn_anon authn_dbd authn_dbm authn_default authn_file authz_dbm authz_default
authz_groupfile authz_host authz_owner authz_user autoindex cache dav dav_fs
dav_lock dbd deflate dir disk_cache env expires ext_filter file_cache filter
headers ident imagemap include info log_config logio mem_cache mime mime_magic
negotiation proxy proxy_ajp proxy_balancer proxy_connect proxy_http rewrite
setenvif so speling status unique_id userdir usertrack vhost_alias"
ELIBC="glibc" INPUT_DEVICES="keyboard mouse evdev" KERNEL="linux"
LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216 lcdm001 mtxorb ncurses
text" USERLAND="GNU" VIDEO_CARDS="apm ark chips cirrus cyrix dummy fbdev glint
i128 i740 i810 imstt mach64 mga neomagic nsc nv r128 radeon rendition s3
s3virge savage siliconmotion sis sisusb tdfx tga trident tseng v4l vesa vga via
vmware voodoo"
Unset:  CPPFLAGS, CTARGET, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LANG, LC_ALL,
LDFLAGS, LINGUAS, PORTAGE_COMPRESS, PORTAGE_COMPRESS_FLAGS,
PORTAGE_RSYNC_EXTRA_OPTS

(In reply to comment #0)
> Btw, do we need to keepdir() this directory (just asking) as it contains:

How's that relevant here? The keepdir is there so that the directory doesn't
vanish on re-emerge if it's empty.

(In reply to comment #1)
> How's that relevant here? The keepdir is there so that the directory doesn't
> vanish on re-emerge if it's empty.

It is not.  Disregard the last paragraph please.

So... I'm confused...

What's the correct fix? It sounds like this file being owned by root is by
design... so is the solution simply that there is nothing to fix, but simply to
understand the reason for the warning?

Thanks...

(In reply to comment #3)
> What's the correct fix?

AFAIK, alternatives are:

1. Do nothing.  Let the postfix log a warning: I do not like unneccesary
warnings in my log files.  It messes up reporting.  I also do not like root
owned files in a directory where there should be none.
2. Patch postfix not to log a warning: Maintenance nightmare.  Also does not
address "root owned file in a directory..." stuff.
3. Have the keep file owned by postfix:postfix:  This is the correct solution
imho.  Adding fowners postfix:postfix
/var/lib/postfix/.keep_${CATEGORY}_${PN}-${SLOT} to the ebuild achieves this.

(In reply to comment #4)
> 3. Have the keep file owned by postfix:postfix:  This is the correct solution
> imho.  Adding fowners postfix:postfix
> /var/lib/postfix/.keep_${CATEGORY}_${PN}-${SLOT} to the ebuild achieves this.

Agreed, fixed this in 2.5.0, 2.5.1 and 2.5.2.

(In reply to comment #4)
> (In reply to comment #3)
> > What's the correct fix?
> 
> AFAIK, alternatives are:
> 
> [...]
4. Extend Portage to generate keepdir files as a specific user.
I.e. keepdir /the/dir user group