[Libguestfs] [PATCH v2 4/5] v2v:tests: use fake virtio-win drivers
Roman Kagan
rkagan at virtuozzo.com
Thu Oct 8 15:16:18 UTC 2015
On Thu, Oct 08, 2015 at 03:53:24PM +0100, Richard W.M. Jones wrote:
> I'm just testing patches 1-3, and if they pass I will push them.
>
> On Thu, Oct 08, 2015 at 04:50:35PM +0300, Roman Kagan wrote:
> > +fake_virtio_win_dir = fake-virtio-win
> > +FAKE_VIRTIO_WIN = $(wildcard $(fake_virtio_win_dir)/*/*/*/*)
>
> Why the fake_virtio_win_dir variable? Unfortunately automake treats
> variables that have '*dir' in the name specially, but in this case I
> don't think you intended that.
Absolutely not. I'm basically uninitiated in auto-foo and had no idea
this could mean anything special to automake; I just needed a simple
make variable. I've checked that it propagated as I expected into
Makefile.in and then into Makefile, and that looked like enough to me.
BTW patch 1 introduces fake_virt_tools_dir which probably has the same
concern, doesn't it?
Roman.
More information about the Libguestfs
mailing list