(In reply to comment #0)
> The current /etc/init.d/screen-cleanup script doesn't check whether SCREENDIR
> (/var/run/screen) exists, which generates a spurious error if it doesn't exist.
> (Screen works without a problem because it creates the directory later on.) This
> situation occurs when /var/run is on a tmpfs (which works with the other init
> scripts).
> A proposed patch is included.
Your patch is not sufficient, screen does _not_ work without the directory:
$ screen
Cannot make directory '/var/run/screen': Permission denied
Which is understandable since screen is only setgid utmp, and /var/run is
root:root 0750. So the patch should be modified to create the directory if it is
not present already and the directory should not be shipped in the deb any more.
Hi!
(In reply to comment #0) d/screen- cleanup script doesn't check whether SCREENDIR
> The current /etc/init.
> (/var/run/screen) exists, which generates a spurious error if it doesn't exist.
> (Screen works without a problem because it creates the directory later on.) This
> situation occurs when /var/run is on a tmpfs (which works with the other init
> scripts).
> A proposed patch is included.
Your patch is not sufficient, screen does _not_ work without the directory:
$ screen
Cannot make directory '/var/run/screen': Permission denied
Which is understandable since screen is only setgid utmp, and /var/run is
root:root 0750. So the patch should be modified to create the directory if it is
not present already and the directory should not be shipped in the deb any more.