Accepting request 105038 from home:aspiers:branches:openSUSE:Tools

As discussed with Adrian, there's no reason to ever keep the temporary uncompressed file.

OBS-URL: https://build.opensuse.org/request/show/105038
OBS-URL: https://build.opensuse.org/package/show/openSUSE:Tools/obs-service-recompress?expand=0&rev=9

obs-service-recompress.changes

@@ -0,0 +1,5 @@
+-------------------------------------------------------------------
+Thu Feb 14 15:52:17 UTC 2012 - aspiers@suse.com
+
+- always remove uncompressed files
+- fix rpmlint warnings

obs-service-recompress.spec

@@ -1,7 +1,7 @@
 #
 # spec file for package obs-service-recompress
 #
-# Copyright (c) 2011 SUSE LINUX Products GmbH, Nuernberg, Germany.
+# Copyright (c) 2012 SUSE LINUX Products GmbH, Nuernberg, Germany.
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -18,11 +18,12 @@
 
 
 Name:           obs-service-recompress
-License:        GPL v2 or later
+License:        GPL-2.0+
 Group:          Development/Tools/Building
 Summary:        An OBS source service: Recompress files
-Version:        0.1
+Version:        0.2
 Release:        1
+Url:            https://build.opensuse.org/package/show?package=obs-service-recompress&project=openSUSE%3ATools
 Source:         recompress
 Source1:        recompress.service
 Requires:       xz gzip bzip2

recompress

@@ -91,10 +91,8 @@ for i in $FILES; do
   # do the real work
   $UNCOMPRESS "$FILE" | $COMPRESS > "$MYOUTDIR/$NEWFILE" || exit 1
 
-  if [ "${FILE#_service:}" != "$FILE" ]; then
+  # we can remove service files, no need to store them twice
-    # we can remove service files, no need to store them twice
+  rm -f "$FILE"
-    rm -f "$FILE"
-  fi
 done
 
 exit 0