Change request; tmpfile cleanup

Jesse Keating jkeating at redhat.com
Mon Feb 2 00:34:03 UTC 2009


I added some code just after FUDCon to generate a fullfilelist at the
top of /pub/fedora/.  I made an error in this leading to lots of /tmp/
files.

Subject: [PATCH] Fix an alias assignment.

This was causing /tmp/ to fill up with lots of tmpfiles of no use.
Also, use a more modern syntax.
---
 configs/system/fedora-updates-push |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/configs/system/fedora-updates-push
b/configs/system/fedora-updates-push
index 473048a..8c09e27 100755
--- a/configs/system/fedora-updates-push
+++ b/configs/system/fedora-updates-push
@@ -36,7 +36,7 @@ rsync -rlptDvHh --delay-updates $RSYNC_OPTS --delete
--delete-after \
 done
 
 pushd /pub/fedora/
-TMPFILE = `mktemp -p /tmp/`
+TMPFILE=$(mktemp -p /tmp/)
 find * -print > $TMPFILE
 diff $TMPFILE fullfilelist >/dev/null
 if [ "$?" = "1" ]; then
-- 
1.5.5.6

-- 
Jesse Keating
Fedora -- Freedom² is a feature!
identi.ca: http://identi.ca/jkeating
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
Url : http://lists.fedoraproject.org/pipermail/infrastructure/attachments/20090201/bde8ecc8/attachment.bin 


More information about the infrastructure mailing list