Page MenuHomeSolus

Update libvirt to 6.1.0
AbandonedPublic

Authored by YakoYakoYokuYoku on Jan 10 2020, 2:05 PM.

Details

Reviewers
JoshStrobl
Group Reviewers
Triage Team
Summary

The update of libvirt to 6.1.0 adds lots of changes since version 4.8.0 (browseable changes).
Dropped the build and installation of the documentation as it cannot be installed any more and also it's already available at the official site.

Signed-off-by: Martin Reboredo <gc1000ll@gmail.com>

Test Plan

Launch a Debian VM with MATE desktop and run a lua hello world in virt-manager.

Diff Detail

Repository
R1891 libvirt
Branch
arcpatch-D8029
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

YakoYakoYokuYoku requested review of this revision.Jan 10 2020, 2:05 PM
JoshStrobl requested changes to this revision.EditedJan 10 2020, 2:47 PM
JoshStrobl added a subscriber: JoshStrobl.

docs package has been removed without explanation, that should probably be restored. There's also ABI removals so you need to basically rebuild everything against this and validate the changes. We also don't do 80 columns and use Markdown in changelogs.

package.yml
48

No you should be using the reconfigure macro.

49

Why are you doing this in a build dir now?

This revision now requires changes to proceed.Jan 10 2020, 2:47 PM
YakoYakoYokuYoku edited the summary of this revision. (Show Details)

Formatted properly the message

JoshStrobl requested changes to this revision.Jan 10 2020, 4:05 PM

Their change to enforce separate build directories is dumb, their reasoning is dumb (basically "another build system we're not even using does this, so let's do it too"), so let's revert it. Please revert the changes based in https://github.com/libvirt/libvirt/commit/f96395e78eaccffbf128336382c74b1250f04032 in a patch. Also you didn't address my issues with the summary or the docs.

This revision now requires changes to proceed.Jan 10 2020, 4:05 PM
YakoYakoYokuYoku edited the summary of this revision. (Show Details)

Dropped the packaging of documentation as it's now pointless

YakoYakoYokuYoku retitled this revision from Update libvirt to 5.10.0 to Update libvirt to 6.0.0.
YakoYakoYokuYoku edited the summary of this revision. (Show Details)

Update to 6.0.0

YakoYakoYokuYoku marked 2 inline comments as done.Feb 5 2020, 1:23 PM
YakoYakoYokuYoku retitled this revision from Update libvirt to 6.0.0 to Update libvirt to 6.1.0.
YakoYakoYokuYoku edited the summary of this revision. (Show Details)

Updated libvirt to 6.1.0

JoshStrobl abandoned this revision.Mar 12 2020, 2:38 AM

Closing since I have this local given it's a requirement for a new gnome-boxes update. I appreciate the time you took to package this though.