On 6/17/22 10:00, Daniel P. Berrangé wrote:
On Fri, Jun 17, 2022 at 09:09:09AM +0200, Michal Prívozník wrote:
> On 6/16/22 17:36, Daniel P. Berrangé wrote:
>> See commit 2 for the example usage
>>
>> Daniel P. Berrangé (2):
>> tools: add helper method for printing an XML document
>> tools: add '--xpath EXPRESSION --wrap' args to all dumpxml commands
>>
>> docs/manpages/virsh.rst | 138 ++++++++++++++++++++++++++++++++++-----
>> tools/virsh-backup.c | 17 ++++-
>> tools/virsh-checkpoint.c | 17 ++++-
>> tools/virsh-domain.c | 55 +++++++++++++---
>> tools/virsh-interface.c | 22 +++++--
>> tools/virsh-network.c | 47 +++++++++----
>> tools/virsh-nodedev.c | 17 ++++-
>> tools/virsh-nwfilter.c | 51 ++++++++++-----
>> tools/virsh-pool.c | 27 +++++---
>> tools/virsh-secret.c | 18 ++++-
>> tools/virsh-snapshot.c | 17 ++++-
>> tools/virsh-util.c | 59 +++++++++++++++++
>> tools/virsh-util.h | 7 ++
>> tools/virsh-volume.c | 28 +++++---
>> 14 files changed, 436 insertions(+), 84 deletions(-)
>>
>
> So this allows virsh dumpxml --wrap $dom, but I guess that's okay. No
> harm done.
I did wonder about calling it --xpwrap or --xpath-wrap too,
and could add error checking if desired
Nah, I think you're good. Push it as is. If somebody complains we can
add error checking later. Then again, it's harmless.
Michal