Re: [Xen-devel] [OSSTEST PATCH] standalone-reset: use mkdir -p

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Mon, Feb 06, 2017 at 03:13:17PM +0000, Ian Jackson wrote:
> Wei Liu writes ("Re: [OSSTEST PATCH] standalone-reset: use mkdir -p"):
> > On Mon, Feb 06, 2017 at 11:49:23AM +0000, Ian Jackson wrote:
> > > Wei Liu writes ("[OSSTEST PATCH] standalone-reset: use mkdir -p"):
> > > ...
> > > >  ensure_dir () {
> > > >         if test -d "$1"; then return; fi
> > > > -       mkdir "$1"
> > > > +       mkdir -p "$1"
> > > 
> > > This deliberately doesn't do this.  Otherwise wrong (or partially
> > > missing) configuration can create strangely-named directories.
> > > 
> > > What missing parent directory did you want it to create ?
> > 
> > /tmp/wei/osstest/
> 
> I guess I mean, which configuration option ?  What was your
> configuration ?

tftptmp=`getconfig TftpTmpDir`
ensure_dir "$tftp$tftptmp"

And $tftp is set to /tmp/ in my standalone config.

>  Did /tmp/wei exist ?

No.

> 
> Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel