pspp-dev
[Top][All Lists]
Advanced

[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_



reply via email to

[Prev in Thread] Current Thread [Next in Thread]