[libvirt] [PATCH] qemuxml2xmloutdata: Use more symlinks

Ján Tomko jtomko at redhat.com
Thu Aug 9 15:05:41 UTC 2018


On Thu, Aug 09, 2018 at 04:50:58PM +0200, Michal Privoznik wrote:
>There are couple of files that are the same in both
>qemuxml2argvdata and qemuxml2xmloutdata directories. Link them
>instead of having full copy.
>
>Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>---
> tests/qemuxml2xmloutdata/boot-floppy-q35.xml       | 33 +---------------
> tests/qemuxml2xmloutdata/bootindex-floppy-q35.xml  | 33 +---------------

These two are a) identical to each other b) one of them will be made
redundant after Peter pushes the BOOTINDEX capability changes.
If you wait until he pushes it, there might be one less change needed in
this commit.

> tests/qemuxml2xmloutdata/disk-virtio-queues.xml    | 35 +---------------
> tests/qemuxml2xmloutdata/intel-iommu-machine.xml   | 27 +------------
> tests/qemuxml2xmloutdata/intel-iommu.xml           | 27 +------------

These are also identical. It makes sense for them to point to their
respective source counterparts. But the followup question is, whether we
want to make the source files point to each other.

> tests/qemuxml2xmloutdata/luks-disks.xml            | 46 +---------------------
> tests/qemuxml2xmloutdata/vcpu-placement-static.xml | 34 +---------------
> 7 files changed, 7 insertions(+), 228 deletions(-)
> mode change 100644 => 120000 tests/qemuxml2xmloutdata/boot-floppy-q35.xml
> mode change 100644 => 120000 tests/qemuxml2xmloutdata/bootindex-floppy-q35.xml
> mode change 100644 => 120000 tests/qemuxml2xmloutdata/disk-virtio-queues.xml
> mode change 100644 => 120000 tests/qemuxml2xmloutdata/intel-iommu-machine.xml
> mode change 100644 => 120000 tests/qemuxml2xmloutdata/intel-iommu.xml
> mode change 100644 => 120000 tests/qemuxml2xmloutdata/luks-disks.xml
> mode change 100644 => 120000 tests/qemuxml2xmloutdata/vcpu-placement-static.xml
>

One way or another:
Reviewed-by: Ján Tomko <jtomko at redhat.com>

Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180809/3372fc6b/attachment-0001.sig>


More information about the libvir-list mailing list