Re: [Libguestfs] [PATCH] v2v: add --in-place mode

First off, thanks for the review. IIUC you are not opposed to the idea of adding the in-place operation mode in general, are you? I'll work through your comments to make the patch fit better with the rest of the code.

On Tue, Jul 28, 2015 at 11:54:59AM +0100, Richard W.M. Jones wrote: > On Mon, Jul 27, 2015 at 07:29:57PM +0300, Roman Kagan wrote: > > --- a/tests/guests/guests.xml.in > > +++ b/tests/guests/guests.xml.in > > Let's not add this to tests/guests, since this disk image is only used > for a single test in the v2v/ subdirectory. > > You can change the new test in the v2v/ subdirectory so it creates (or > is supplied with) the XML. There are various tests which work like > that already - eg: v2v/test-v2v-cdrom.* OK, makes sense, thanks for the suggestion. While at this: do you think adding the test for the newly added option should go in the same commit with the option itself? It may be easier to review separately, but YMMV, so please let me know which way you'd like it better.