diff --git a/rsync.changes b/rsync.changes
index a9a9a56..59cedad 100644
--- a/rsync.changes
+++ b/rsync.changes
@@ -1,3 +1,12 @@
+-------------------------------------------------------------------
+Tue Aug 12 10:10:06 CEST 2008 - puzel@suse.cz
+
+- rsyncd.rc: use Should-Start instead of X-UnitedLinux-Should-Start 
+- specfile fixes:
+  * added Should-Stop
+  * 'make test' moved to %check
+  * recommend logrotate
+
 -------------------------------------------------------------------
 Mon Apr 14 11:43:03 CEST 2008 - ro@suse.de
 
diff --git a/rsync.spec b/rsync.spec
index 3571839..e5343d5 100644
--- a/rsync.spec
+++ b/rsync.spec
@@ -2,9 +2,16 @@
 # spec file for package rsync (Version 3.0.2)
 #
 # Copyright (c) 2008 SUSE LINUX Products GmbH, Nuernberg, Germany.
-# This file and all modifications and additions to the pristine
-# package are under the same license as the package itself.
 #
+# All modifications and additions to the file contributed by third parties
+# remain the property of their copyright owners, unless otherwise agreed
+# upon. The license for this file, and modifications and additions to the
+# file, is the same license as for the pristine package itself (unless the
+# license for the pristine package is not an Open Source License, in which
+# case the license is the MIT License). An "Open Source License" is a
+# license that conforms to the Open Source Definition (Version 1.9)
+# published by the Open Source Initiative.
+
 # Please submit bugfixes or comments via http://bugs.opensuse.org/
 #
 
@@ -19,12 +26,13 @@ BuildRequires:  texlive-latex
 BuildRequires:  te_ams te_latex
 %endif
 BuildRequires:  popt-devel zlib-devel
+Recommends:     logrotate
 License:        GPL v3 or later
 Group:          Productivity/Networking/Other
 PreReq:         %fillup_prereq %insserv_prereq sed grep
 AutoReqProv:    on
 Version:        3.0.2
-Release:        1
+Release:        38
 Summary:        Replacement for RCP/mirror that has Many More Features
 Source:         samba.org/ftp/rsync/rsync-%{version}.tar.bz2
 Source1:        samba.org/ftp/rsync/rsync-patches-%{version}.tar.bz2
@@ -83,14 +91,16 @@ CFLAGS=$RPM_OPT_FLAGS \
 	--enable-acl-support \
 	--enable-xattr-support
 make %{?jobs:-j %jobs}
+latex tech_report
+latex tech_report
+dvips tech_report -o tech_report.ps
+
+%check
 %ifarch s390x
 make test || :
 %else
 make test
 %endif
-latex tech_report
-latex tech_report
-dvips tech_report -o tech_report.ps
 
 %install
 make install DESTDIR=$RPM_BUILD_ROOT
@@ -137,6 +147,12 @@ fi
 %doc COPYING NEWS README tech_report.ps tech_report.tex
 
 %changelog
+* Tue Aug 12 2008 puzel@suse.cz
+- rsyncd.rc: use Should-Start instead of X-UnitedLinux-Should-Start
+- specfile fixes:
+  * added Should-Stop
+  * 'make test' moved to %%check
+  * recommend logrotate
 * Mon Apr 14 2008 ro@suse.de
 - update to version 3.0.2
   - Fixed a potential buffer overflow in the xattr code.
diff --git a/rsyncd.rc b/rsyncd.rc
index 3cfe612..65feea4 100644
--- a/rsyncd.rc
+++ b/rsyncd.rc
@@ -17,7 +17,8 @@
 # Provides: rsync
 # Required-Start: $remote_fs $syslog
 # Required-Stop:  $remote_fs $syslog
-# X-UnitedLinux-Should-Start: slpd
+# Should-Start: slpd
+# Should-Stop:  slpd
 # Default-Start:  3 5
 # Default-Stop:   0 1 2 6
 # Description:    Start the rsync server daemon