[systemd-devel] Regression in 219: tmpfiles.d order not respected any more

[Martin Pitt]

Hey Zbigniew, hey all,

our autopkgtests found a regression in 219; unfortunately only
yesterday when we flipped our cloud images to systemd by default,
before that the bug was hidden.

As per tmpfiles.d(5), the first match should win if there are several
tmpfiles.d lines for the same directory. Our rsyslog package ships a
/usr/lib/tmpfiles.d/00rsyslog.conf which sets /var/log to 0775, so
that it can write into it as the "syslog" system user. But now the
permissions of /var/log are 0755 as /usr/lib/tmpfiles.d/var.conf
overwrites it.

This can be reproduced with

| $ cat ~/test.conf
| d /var/log 0775 root syslog -
| d /var/log 0755 - - -
|
| $ sudo SYSTEMD_LOG_LEVEL=debug systemd-tmpfiles --create ~/test.conf
| [/home/martin/test.conf:2] Duplicate line for path "/var/log", ignoring.
| Running create action for entry d /var/log
| Found existing directory "/var/log".
| chmod "/var/log" to mode 775
| /var/log created successfully.
| Running remove action for entry d /var/log
| Running create action for entry d /var/log
| Found existing directory "/var/log".
| chmod "/var/log" to mode 755
| /var/log created successfully.
| Running remove action for entry d /var/log

(this command works well right out of the built tree, which is what I
used for git bisect run)

In 218 this still worked fine, but git bisect shows that
http://cgit.freedesktop.org/systemd/systemd/commit/?id=3f93da987961
introduced this regression: i. e. even though it still warns about the
duplicate line for /var/log/, the second line is now being processed
anyway.

Thanks,

Martin

-- 
Martin Pitt                        | http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.freedesktop.org/archives/systemd-devel/attachments/20150305/8aafc6c2/attachment-0001.sig>