On Wed, 4 Nov 2015 08:53:11 +0100 Gilles Chanteperdrix <[email protected]> wrote:
> On Wed, Oct 28, 2015 at 12:15:16PM +0100, Henning Schild wrote: > > The environment variable TMPDIR is optional and not always set. We > > could test for it and fall back to /tmp/ or just use the tool > > mktemp. > > > > Signed-off-by: Henning Schild <[email protected]> > > --- > > scripts/prepare-kernel.sh | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/scripts/prepare-kernel.sh b/scripts/prepare-kernel.sh > > index 8e960e8..333d11c 100755 > > --- a/scripts/prepare-kernel.sh > > +++ b/scripts/prepare-kernel.sh > > @@ -238,9 +238,9 @@ fi > > # Create an empty output patch file, and initialize the temporary > > tree. if test "x$output_patch" != "x"; then > > > > - temp_tree=$TMPDIR/prepare-kernel-$$ > > - if ! mkdir $temp_tree; then > > - echo Temporary directory $temp_tree already exists, > > aborting. > > + temp_tree=`mktemp -d prepare-kernel-XXX --tmpdir` > > + if [ $? -ne 0 ]; then > > + echo Temporary directory could not be created. > > exit 1 > > fi > > The version of mktemp installed on my machine requires 6 X in the > template. That is unfortunate but easy enough to fix. The patch has already been applied, so we will need another one on top. I suggest you go ahead and expand the template since you found the problem. The man-pages on my boxes all say "at least 3". Henning _______________________________________________ Xenomai mailing list [email protected] http://xenomai.org/mailman/listinfo/xenomai
