Accepting request 151489 from home:jsmeix:branches:Printing

Added explanation why pid, socket and lock files are moved to /run by cups-move-everything-to-run.patch and have it as Patch108 unconditioned in the spec file.

OBS-URL: https://build.opensuse.org/request/show/151489
OBS-URL: https://build.opensuse.org/package/show/Printing/cups?expand=0&rev=255

cups.changes

@@ -2,7 +2,9 @@
 Wed Jan 30 13:24:43 UTC 2013 - rmilasan@suse.com
 
 - Move everything (pid, lock, socket files) to /run only for 12.3.
-  Added: cups-move-everything-to-run.patch 
+  Added: cups-move-everything-to-run.patch
+  See also
+  http://lists.opensuse.org/opensuse-factory/2013-01/msg00578.html
 
 -------------------------------------------------------------------
 Thu Dec 20 12:38:47 CET 2012 - jsmeix@suse.de

cups.spec

@@ -15,7 +15,6 @@
 # Please submit bugfixes or comments via http://bugs.opensuse.org/
 #
 
-
 Name:           cups
 BuildRequires:  dbus-1-devel
 BuildRequires:  fdupes
@@ -152,10 +151,17 @@ Patch105:       cups-0001-systemd-add-systemd-socket-activation-and-unit-files.p
 Patch106:       cups-config-libs.patch
 # Patch107 provides cupsd.service in cups.service
 Patch107:       cups-provides-cupsd-service.patch
-%if %suse_version > 1220
 # Patch108 cups-move-everything-to-run.patch (move pid, socket and lock files to /run)
+# Explanation that I got via mail from Robert Milasan (quotation):
+#   the changes are due to systemd. Systemd needs /run and in our case
+#   openSUSE we bind /run to /var/run which at boot time can created
+#   issues, meaning /var/run is used before being binded to /run.
+#   All apps should use /run and /run/lock instead of /var/run and /var/lock.
+#   In other distro's they links /run to /var/run and /run/lock
+#   to /var/lock, but we don't, so it's better to move everything
+#   and yes upstream should do the same. Please also check:
+#   http://lists.opensuse.org/opensuse-factory/2013-01/msg00578.html
 Patch108:       cups-move-everything-to-run.patch
-%endif
 # Install into this non-root directory (required when norootforbuild is used):
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build