Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 87501

Summary: recent xmame ebuilds lack lirc support
Product: Gentoo Linux Reporter: Erik Hovland <erik>
Component: [OLD] GamesAssignee: Gentoo Games <games>
Status: RESOLVED FIXED    
Severity: enhancement    
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: adds lirc use to xmame ebuilds

Description Erik Hovland 2005-03-31 21:53:34 UTC
As of version 0.89 xmame now has support for lirc. This is really handy for those who run keyboardless systems (ie mythtv). But the ebuilds of versions after version 0.89 do not have the required bits to provide this feature if the lirc use flag is given.

Reproducible: Always
Steps to Reproduce:
1. ACCEPT_KEYWORDS='~x86' USE='lirc' emerge xmame
2. ldd /usr/games/bin/xmame

Actual Results:  
There is no lirc_client library linked in to xmame and so no action by xmame
when a lirc command is sent.

Expected Results:  
emerge should have added lirc_client into the libraries that xmame links against
and should have enabled the LIRC variable in the makefile.unix
Comment 1 Erik Hovland 2005-03-31 21:54:50 UTC
Created attachment 54994 [details, diff]
adds lirc use to xmame ebuilds

This patch will add lirc support to the xmame 0.94 and 0.95 ebuilds. This will
make it possible to use lirc with xmame and make (mostly) keyboardless
operation of xmame possible.
Comment 2 Mr. Bones. (RETIRED) gentoo-dev 2005-04-01 09:02:20 UTC
added lirc support to xmame and xmess.

It would have been nice if I had been able to apply your patch directly.  In the future, please limit your patch to the feature/bug fix in question and leave the formatting of the IUSE variable alone.  Thanks.

No rev bump since these things are such large compiles.  For lirc support, resync and remerge.

Thanks for the bug report.
Comment 3 Erik Hovland 2005-04-01 09:43:09 UTC
Sorry about the reformatting. I erred on the side of 80 columns or less (I know better now). Thanks for the quick resolution!