Code review comment for ~jocave/plainbox-provider-checkbox:uvtkvm-test-on-ubuntucore

Revision history for this message
Jeff Lane  (bladernr) wrote :

Hey,

While youre at it, could you add a packaging.pxu file to explicitly install uvtool and uvtool-libvirt?

There is no packaging metadata that i can find, and turns out that on focal, installing uvtool also installs uvtool-libvirt, but on bionic, you have to explicitly install both. So a quick fix would be to just add a packaging metadata definition to explicitly install both regardless of release.

That's just a side request that could be done here... otherwise, everything below makes sense to me.

review: Needs Information

« Back to merge proposal