[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] tests: Avoid nonportable "mktemp" usage.
From: |
Ben Pfaff |
Subject: |
Re: [PATCH] tests: Avoid nonportable "mktemp" usage. |
Date: |
Tue, 12 Jul 2011 21:55:47 -0700 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.2 (gnu/linux) |
Thanks, I pushed this to master.
John Darrington <address@hidden> writes:
> This patch looks fine to me.
>
> J'
>
> On Sun, Jul 10, 2011 at 03:52:18PM -0700, Ben Pfaff wrote:
> Jeremy Lavergne <address@hidden> reported that
> mktemp on Mac OS X does not have a -p option. But I don't see
> a reason to use mktemp at all here, so this commit uses a fixed
> name instead.
> ---
> This works for me. John, did you have a reason to want to use a random
> name here? mktemp doesn't generate very interesting file names in any
> case.
>
> diff --git a/tests/libpspp/zip.at b/tests/libpspp/zip.at
> index a725049..55bf563 100644
> --- a/tests/libpspp/zip.at
> +++ b/tests/libpspp/zip.at
> @@ -14,7 +14,7 @@ mkdir -p $dir1
> names=""
> s=1;
> while test $s -le 8192 ; do
> - name=`mktemp -p $dir1`;
> + name=$dir1/$s
> dd if=/dev/urandom of=$name count=1 bs=$s 2> /dev/null
> s=$(($s * 2));
> bn=`basename $name`;
> --
> 1.7.2.5
--
"...dans ce pays-ci il est bon de tuer de temps en temps un amiral
pour encourager les autres."
--Voltaire, _Candide_