04316: MAME/SDL/Linux/PowerPC does not build

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
04316	Compiling	Major	Always	Apr 22, 2011, 16:57	Apr 23, 2011, 21:21

Tester	jordi	View Status	Public	Platform	SDLMAME
Assigned To		Resolution	Fixed	OS	Linux
Status [?]	Resolved			Driver
Version	0.141	Fixed in Version	0.142	Build	PowerPC
		Fixed in Git Commit		Github Pull Request #

Summary	04316: MAME/SDL/Linux/PowerPC does not build
Description	0.141 does not build for ppclinux, due to a wrongly declared ifdef construct. Compiling src/mame/drivers/aleck64.c... cc1plus: warnings being treated as errors In file included from src/mame/includes/n64.h:5:0, from src/mame/drivers/aleck64.c:173: src/mame/video/n64.h:43:5: error: "LSB_FIRST" is not defined make[2]: *** [obj/sdl/mame/mame/drivers/aleck64.o] Error 1 The attached patch, now available in the Debian package, fixes it.
Steps To Reproduce
Additional Information
Github Commit

Flags
Regression Version
Affected Sets / Systems

Attached Files	fix_ppc_build.txt (611 bytes) Apr 22, 2011, 16:57 Uploaded by jordi Fix sdlmame linuxppc build [Show Content]

No.07391 Fujix Administrator Apr 22, 2011, 17:27	I allowed a file with the .patch extension. BTW, please DO NOT use Mantis for reporting your request. Please remember THIS IS ONLY FOR MAME. You can use a report note if you want to ask me to allow a .patch file. In addition, your report is for 0.141. Please reconfirm that you have the same issue with the latest version of MAME.
No.07392 Tafoid Administrator Apr 22, 2011, 17:32	Also, since this appears to be an SDL specific issue, you would be better served by posting at the SDLMAME forum if the problem persists with 0.142 (current working base is 0.142u1): http://forums.bannister.org/ubbthreads.php?ubb=postlist&Board=8
No.07394 R. Belmont Developer Apr 22, 2011, 23:11	As far as I can tell, this has already been fixed in 0.142. I don't quite get Fuji's objection since this bug is for MAME :)
No.07396 jordi Tester Apr 23, 2011, 00:50	Hi! Sorry Fujix, as you know I'm very new to the MAME community and still need to learn my ways. :) I had no idea where to report the patch thing, I still did not know who the administrators are or how to reach them. Sorry for the noise! R. Belmont, thanks for the info. In Debian we still have 0.141 because we had been working on some other issues before updating to the new version. FYI, mame hit debian unstable today and will appear soonish in Ubuntu distributions.
No.07397 Fujix Administrator Apr 23, 2011, 02:45 edited on: Apr 23, 2011, 05:44	RB, in fact jordi made an another single bug report that Mantis doesn't allow adding a file with .patch extension. I already deleted it. jordi, I understand your situation. Our Guide Line page and FAQ page will help making a better report. http://mametesters.org/rules.html http://mametesters.org/faq.html Enjoy testing.
No.07398 R. Belmont Developer Apr 23, 2011, 21:21	Jordi: glad to hear MAME is hitting unstable! Many distro and OS guys contact me directly rather than through MAMETesters - the bannister.org forum is a good place (you'll see posts from Belegdol, who makes the RPMs for Fedora/Red Hat/CentOS) or you can PM me to get an email. Now, I have a NetBSD patch to apply :)