Bug 1038544

Summary: qemu-system-* packages should not depend on brlapi
Product: [Fedora] Fedora Reporter: Christian Stadelmann <fedora>
Component: qemuAssignee: Fedora Virtualization Maintainers <virt-maint>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 20CC: amit.shah, berrange, cfergeau, dwmw2, fedora, itamar, pbonzini, rjones, scottt.tw, virt-maint
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-12-05 11:09:23 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Christian Stadelmann 2013-12-05 10:10:47 UTC 
Description of problem:
Currently qemu-system-* packages depend on the brlapi package. I don't see why.
According to the spec file [1] brlapi is only a builddep (which is ok).

According to package metadata it depends on brlapi:
$ yum deplist qemu-system-x86 | grep brl
  Dependency: libbrlapi.so.0.6()(64bit)
   provider: brlapi.x86_64 0.6.0-9.fc20

[1] http://pkgs.fedoraproject.org/cgit/qemu.git/tree/qemu.spec
Comment 1 Richard W.M. Jones 2013-12-05 10:18:35 UTC 
qemu can emulate a Braille device:

http://git.qemu.org/?p=qemu.git;a=blob;f=backends/baum.c;hb=HEAD

It appears to be compiled in:

$ qemu-system-x86_64 -device \? |& grep -i bra
name "usb-braille", bus usb-bus
$ qemu-system-x86_64 -help |& grep braille
-chardev braille,id=id[,mux=on|off]

Is there some reason why we wouldn't want to use brlapi?
Comment 2 Christian Stadelmann 2013-12-05 10:29:33 UTC 
No, I just don't know why we would want to depend on it.
Comment 3 Richard W.M. Jones 2013-12-05 10:33:12 UTC 
Because qemu can emulate a Braille device ...

Is there some problem with depending on brlapi?
Comment 4 Christian Stadelmann 2013-12-05 11:04:48 UTC 
It's just another package (brlapi, brltty) installed but only needed in rare cases.
Comment 5 Richard W.M. Jones 2013-12-05 11:09:23 UTC 
qemu isn't designed in a way that allows us to split out this
functionality into a separate package.  I suggest you work
with upstream so that qemu can use loadable modules to implement
functionality.
Comment 6 Christian Stadelmann 2013-12-05 11:42:58 UTC 
Ok.