Hi Phil,
Phil Dibowitz wrote:
> Poking around a see a few things:
>
> 1. I don't see a single entry in the man page that leaves off the 'age'
> column.
Never heard of that until now.
> I wonder if newer systemd is ignoring this entry because of
> that. From the man page, they have a screen example:
O.o
> Example 1. Create directories with specific mode and ownership
>
> screen(1), needs two directories created at boot with specific modes
> and ownership:
>
> # /usr/lib/tmpfiles.d/screen.conf
> d /run/screens 1777 root screen 10d
> d /run/uscreens 0755 root screen 10d12h
>
> I wouldn't have an age on them that seems scary,
Definitely.
> but you could have a '-' to imply no age, changing your config file to:
>
> d /run/screen 1777 root utmp -
Yes. Would you mind checking if modifying the file fixes the issue for
you in the case of it being reproducible?
Will check at the next occasion of rebooting one of my few
systemd-based machines, too.
Then again, "systemd-tmpfiles --cat-config" outputs, beyond others:
# /etc/tmpfiles.d/screen-cleanup.conf
d /run/screen 1777 root utmp
So it doesn't seem to ignore it.
> Relatedly, since your config file comes in your package, it should
> *probably* be in /usr/lib/tmpfiles.d/ not /etc/tmpfiles.d. :)
Hrm, actually both, /etc/tmpfiles.d/screen-cleanup.conf (generated by
postinst) and /usr/lib/tmpfiles.d/screen-cleanup.conf (part of the
package), exist here.
Currently trying to figure out if this was intended that way or if one
of them still being there is an error, as in the beginning of the
package's systemd support, there was a little bit of a whirl about how
/etc/tmpfiles.d/screen-cleanup.conf and especially
/lib/systemd/system/screen-cleanup.service exactly are shipped
respectively generated.
I guess the point is that it is not completely redundant:
* /usr/lib/tmpfiles.d/screen-cleanup.conf is used when the package is
installed, but not yet configured.
* /etc/tmpfiles.d/screen-cleanup.conf is generated by postinst with
contents depending on the permissions of /usr/bin/screen and used as
soon as the package is configured.
* Depending on the permissions of /usr/bin/screen,
/etc/tmpfiles.d/screen-cleanup.conf either overrides
/usr/lib/tmpfiles.d/screen-cleanup.conf (with statoverride) or just
states the same (default). In the latter case one of them is
redundant.
Relevant changelog entries seem so far:
screen (4.5.1-3) experimental; urgency=low
* Also change permissions (wrt. to utempter usage) in
/usr/lib/tmpfiles.d/screen-cleanup.conf. Fixes "Directory
'/run/screen' must have mode 777." error when running under systemd.
-- Axel Beckert <a...@debian.org> Wed, 08 Mar 2017 23:20:15 +0100
screen (4.3.0-2) unstable; urgency=low
[...]
* Re-add debian/dirs with /etc/tmpfiles.d/ and add a comment why screen
ships an empty directory.
+ Fixes regression introduced in 4.2.1-4: If systemd is not installed
and screen is either setuid or neither setuid nor setgid,
/var/lib/dpkg/info/screen.postinst bailed out with "16:
/var/lib/dpkg/info/screen.postinst: cannot create
/etc/tmpfiles.d/screen-cleanup.conf: Directory nonexistent".
+ See comment in debian/dirs for more detailed reasoning.
* No more ship /lib/systemd/system/screen-cleanup.service in the package
but link it to /dev/null in postinst and remove the link again in
postrm. (LP: #1462692)
[...]
-- Axel Beckert <a...@debian.org> Wed, 17 Jun 2015 21:57:18 +0200
screen (4.2.1-4) unstable; urgency=medium
[...]
* Remove debian/dirs:
+ /etc/tmpfiles.d/ is shipped by systemd and only needed with systemd
+ Explicitly creating /etc/ seems no more necessary.
[...]
-- Axel Beckert <a...@debian.org> Thu, 11 Jun 2015 01:02:06 +0200
screen (4.1.0~20120320gitdb59704-10) unstable; urgency=medium
[...]
* Add systemd variant of /etc/init.d/screen-cleanup (Closes: #740301)
+ Update README.Debian and NEWS.Debian accordingly
+ Bump versioned debhelper build-dependency to 9.20130504 to ensure
systemd support in dh_installinit.
Thanks Josh Triplett for help and patches!
[...]
-- Axel Beckert <a...@debian.org> Thu, 17 Apr 2014 11:45:17 +0200
Regards, Axel
--
,''`. | Axel Beckert <a...@debian.org>, https://people.debian.org/~abe/
: :' : | Debian Developer, ftp.ch.debian.org Admin
`. `' | 4096R: 2517 B724 C5F6 CA99 5329 6E61 2FF9 CD59 6126 16B5
`- | 1024D: F067 EA27 26B9 C3FC 1486 202E C09E 1D89 9593 0EDE
