[PATCH 1/5] qemuOpenFileAs: Move into util/virqemu.c

Ján Tomko jtomko at redhat.com
Tue Aug 25 09:15:11 UTC 2020


On a Monday in 2020, Daniel P. Berrangé wrote:
>On Mon, Aug 24, 2020 at 05:24:51PM +0200, Peter Krempa wrote:
>> On Mon, Aug 24, 2020 at 16:18:40 +0100, Daniel Berrange wrote:
>> > On Thu, Aug 06, 2020 at 11:55:12AM +0200, Peter Krempa wrote:
>> > > Signed-off-by: Peter Krempa <pkrempa at redhat.com>
>> > > ---
>> > >  src/libvirt_private.syms |   1 +
>> > >  src/qemu/qemu_driver.c   | 138 ++-------------------------------------
>> > >  src/util/virqemu.c       | 130 ++++++++++++++++++++++++++++++++++++
>> > >  src/util/virqemu.h       |   7 ++
>> > >  4 files changed, 144 insertions(+), 132 deletions(-)
>> >
>> > The original source files were not built on Win32, the new
>> > source files are. This causes a build failure due to...
>>
>> Sigh. I thought that I could make it to be an universal helper. Not very
>> useful, but universal.
>>
>> An alternative solution could be to dump it to qemu_domain.c ...
>

How about qemu_util.c?

qemu_domain.c already is a dump.

Jano

>Yeah, I actually thought about this after sending the mail. It isn't
>used anywhere else outside of QEMU.
>
>
>Regards,
>Daniel
>-- 
>|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
>|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
>|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200825/186909ac/attachment-0001.sig>


More information about the libvir-list mailing list