Change request; tmpfile cleanup

[Nigel Jones]

+1

- Nigel
----- "Jesse Keating" <jkeating at redhat.com> wrote:

> 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
> 
> _______________________________________________
> Fedora-infrastructure-list mailing list
> Fedora-infrastructure-list at redhat.com
> https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list