Opened 16 years ago

Closed 16 years ago

Last modified 7 months 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
Keywords: Cc:


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

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 16 years ago.
Patch against this morning's CVS

Download all attachments as: .zip

Change History (5)

Changed 16 years ago by SF/niemisto

Attachment: freebsd_fix.diff added

Patch against this morning's CVS

comment:1 Changed 16 years ago by fingolfin

Owner: set to fingolfin
Resolution: fixed
Status: newclosed

comment:2 Changed 16 years ago by fingolfin

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 Changed 7 months ago by digitall

Component: Ports

comment:4 Changed 7 months ago by digitall

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