Opened 20 years ago

Closed 20 years ago

Last modified 5 years ago

#8308 closed patch (fixed)

A patch that allows compiling X11 backend on FreeBSD

Reported by: SF/niemisto Owned by: fingolfin
Priority: normal Component: Port: BSD
Version: Keywords:
Cc: Game:

Description

In file backends/x11/x11.cpp you included linux/soundcard.h. AFAIK Linux is the only system that puts any headers under linux/. So I made a little #ifdef modification that makes all other systems to include file sys/soundcard.h.

There is in fact another problem in the same file near line 1030 or so in method OSystem_X11::set_timer(): you use value timer that is not defined anywhere (at least on FreeBSD). I tried to replace it with the methods parameter interval but didn't include it in this patch because I think it may be wrong.

Ticket imported from: #862283. Ticket imported from: patches/413.

Attachments (1)

freebsd_fix.diff (563 bytes ) - added by SF/niemisto 20 years ago.
Patch against this morning's CVS

Download all attachments as: .zip

Change History (5)

by SF/niemisto, 20 years ago

Attachment: freebsd_fix.diff added

Patch against this morning's CVS

comment:1 by fingolfin, 20 years ago

Owner: set to fingolfin
Resolution: fixed
Status: newclosed

comment:2 by fingolfin, 20 years ago

Well "sys/soundcard.h" isn't really portable either... :-) But OK, if this helps you, I guess it won't hurt. The X11 backend is completly unmaintained currently, though. I.e. I am not surprised that it won't even compile out of the box.

comment:3 by digitall, 5 years ago

Component: Ports

comment:4 by digitall, 5 years ago

Component: PortsPort: BSD
Note: See TracTickets for help on using tickets.