Re: [Xen-devel] [PATCH v2 3/9] OssTest: Add target_dir_exists

[Konrad Rzeszutek Wilk <konrad@xxxxxxxxxx>]

On Tue, Dec 13, 2016 at 04:24:56PM +0000, Ian Jackson wrote:
> Konrad Rzeszutek Wilk writes ("[PATCH v2 3/9] OssTest: Add 
> target_dir_exists"):
> > We have target_file_exists but not an equivalant one for directories.
> > This adds it in and is used in the "ts-xen-build: Make {xen|}dist.tar.gz
> > only if $builddir/install/{$xen|}install" patch.
> 
> Do you care about the distinction between a file and a directory ?

Yes. I just need to mke sure that the directory exists, while there
may be multiple files (in it).

> IOW I don't understand why target_file_exists wouldn't do.
> 
> If you think the word "file" in its name is confusing, please feel
> free to add a doc comment or to rename that function.

The functions are pretty similar. The file one does 'test -e' while
this one does 'test -d'.

> 
> Thanks,
> Ian.

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