On Thu, Nov 05, 2015 at 11:31:28AM +0100, Henning Schild wrote:
> 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".
I suggest your fix it.
--
Gilles.
https://click-hack.org
_______________________________________________
Xenomai mailing list
[email protected]
http://xenomai.org/mailman/listinfo/xenomai
