Ubuntu
virt-manager package

  1. Bug #187048
  2. Comment #29

Comment 29 for bug 187048

Revision history for this message
Martin Pitt (pitti) wrote :

You need to fix the i18n'ing of the string:

 msg = "..."
 error(_(msg))

does not work, since xgettext only scans for _('string'), and does not track variables. You need to use

  msg = _('...')
  error(msg)

This is really an edge case of an SRU, but I agree that unhelpful error messages drive users up the wall, so I'm ok with smuggling this in (it's a small patch),

However, Dustin's latest patch only tests for kvm membership. Weren't some of the reports due to not being in "libvirtd"? Or is http://launchpadlibrarian.net/13880679/virt-manager-check-groups.debdiff required in addition to http://launchpadlibrarian.net/14055096/virt-manager_test_kvm_perms.debdiff ?