Jesse Keating wrote:
...
> diff --git a/configs/build/rawhide b/configs/build/rawhide
...
> +15 6 * * * masher TMPDIR=`mktemp -d /tmp/rawhide.XXXXXX` && cd $TMPDIR; git 
> clone -n git://git.fedorahosted.org/git/releng; cd releng; git checkout -b 
> rawhide-stable; LANG=en_US.UTF-8 ./scripts/buildrawhide $(date "+\%Y\%m\%d"); 
> sudo -u ftpsync /usr/local/bin/update-fullfilelist fedora

It'd be good to change the other semicolons to " &&", too.
Otherwise, failing mktemp will not stop the script, and
the git clone will run in $HOME.

An alternative would be to start with "set -e;" (stop on error)

15 6 * * * masher set -e; TMPDIR=`mktemp -d /tmp/rawhide.XXXXXX`; ...

or to do this:

15 6 * * * masher TMPDIR=`mktemp -d /tmp/rawhide.XXXXXX` || exit 1; cd 
$TMPDIR...

_______________________________________________
Fedora-infrastructure-list mailing list
Fedora-infrastructure-list@redhat.com
https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list

Reply via email to