vncterm fails because qemu is in different location than in Debian

[Mike McClurg]

In debian, the qemu directory is /usr/share/qemu, but in Ubuntu that directory is /usr/share/qemu-linaro. The workaround for this has been to manually create a symlink from /usr/share/qemu pointing to qemu-linaro. We should instead modify the Ubuntu package to point to the correct location.

[Mike McClurg (mike-mcclurg)]

Debdiff patch attached.

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "xcp-vncterm.debdiff" of this bug report has been identified as being a patch in the form of a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-sponsors team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

[Marc Deslauriers (mdeslaur)]

NACK. On Ubuntu, the path is also /usr/share/qemu in the qemu-common package:

/usr/share/qemu/keymaps/ar
/usr/share/qemu/keymaps/bepo
/usr/share/qemu/keymaps/common
/usr/share/qemu/keymaps/da
/usr/share/qemu/keymaps/de
/usr/share/qemu/keymaps/de-ch
/usr/share/qemu/keymaps/en-gb
/usr/share/qemu/keymaps/en-us
/usr/share/qemu/keymaps/es
/usr/share/qemu/keymaps/et
/usr/share/qemu/keymaps/fi
/usr/share/qemu/keymaps/fo
/usr/share/qemu/keymaps/fr
/usr/share/qemu/keymaps/fr-be
/usr/share/qemu/keymaps/fr-ca
/usr/share/qemu/keymaps/fr-ch
/usr/share/qemu/keymaps/hr
/usr/share/qemu/keymaps/hu
/usr/share/qemu/keymaps/is
/usr/share/qemu/keymaps/it
/usr/share/qemu/keymaps/ja
/usr/share/qemu/keymaps/lt
/usr/share/qemu/keymaps/lv
/usr/share/qemu/keymaps/mk
/usr/share/qemu/keymaps/modifiers
/usr/share/qemu/keymaps/nl
/usr/share/qemu/keymaps/nl-be
/usr/share/qemu/keymaps/no
/usr/share/qemu/keymaps/pl
/usr/share/qemu/keymaps/pt
/usr/share/qemu/keymaps/pt-br
/usr/share/qemu/keymaps/ru
/usr/share/qemu/keymaps/sl
/usr/share/qemu/keymaps/sv
/usr/share/qemu/keymaps/th
/usr/share/qemu/keymaps/tr

[Mike McClurg (mike-mcclurg)]

Ah, thanks Marc! I didn't know that. Why are the keymaps duplicated between these packages? There must have been a reason to deviate from Debian's qemu.

I'll rework the patch so that we have different depends based on the vendor, instead of using two different patch series.

[Marc Deslauriers (mdeslaur)]

What exactly doesn't work in xcp-vncterm that requires this change?

[Mike McClurg (mike-mcclurg)]

xcp-vncterm fails to start because it can't find the qemu keymaps, because it hardcodes the location of the keymaps. We have a patch that points xcp-vncterm at /usr/share/qemu/keymaps on Debian, but that wasn't working on Ubuntu because qemu-keymaps on Ubuntu puts the keymaps at /usr/share/qemu-linaro/keymaps.

The qemu-keymaps package is an indirect dependency of xcp-vncterm for both Debian and Ubuntu (by virtue of it needing to be installed alongside xen-hypervisor). On Debian, qemu-keymaps installs keymaps to /usr/share/qemu/keymaps. On Ubuntu, it looks like I'll need to make xcp-vncterm depend directly on qemu-common (which doesn't exist in Debian). I suppose that this is more maintainable than having two separate quilt patchqueues.

[Marc Deslauriers (mdeslaur)]

Ah! I see now.

The qemu-keymaps package on Ubuntu is an optional, mostly-unused package in universe. qemu-common would be the right dependency for the majority of users.

[Mike McClurg (mike-mcclurg)]

So xen-hypervisor-amd64 suggests xen-utils-4.1 suggests qemu-keymaps and qemu-utils in both Debian and Ubuntu. Perhaps one of the three packages (xen-utils-4.1, qemu-keymaps, qemu-utils) should also suggest qemu-common? It seems the only package that suggests/requires qemu-common is qemu-kvm. Is this correct?

[Mike McClurg (mike-mcclurg)]

Moving this from Invalid to In Progress. We still need to fix this package so that it depends on qemu-common when installing on Ubuntu.