ports/107447: [patch:enhancement] devel/sdl12 - Add devel/directfb support

[Mario Sergio Fujikawa Ferreira]

>Number:         107447
>Category:       ports
>Synopsis:       [patch:enhancement] devel/sdl12 - Add devel/directfb support
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Wed Jan 03 08:10:17 GMT 2007
>Closed-Date:
>Last-Modified:
>Originator:     Mario Sergio Fujikawa Ferreira
>Release:        FreeBSD 6.2-PRERELEASE i386
>Organization:
>Environment:
System: FreeBSD exxodus.fedaykin.here 6.2-PRERELEASE FreeBSD 6.2-PRERELEASE #19: Thu Dec 21 19:00:25 BRST 2006 lioux at exxodus:/usr/src-6.x/sys/i386/compile/LIOUX i386

>Description:

	devel/sdl12 neglects to build if devel/directfb is installed.
Furthermore, it seems to be a shame that we do not take the opportunity
to try this support. Therefore, I wrote a simple set of patches to allow
sdl12 to build with directfb support.

	Have in mind that I did not test if the support actually
works. I just got it to compile. Therefore, tests are required.

>How-To-Repeat:

n.a.

>Fix:

	Patch Makefile
	Add patch patch-src__video__directfb__SDL_DirectFB_video.c to files


--- patch begins here ---
Index: Makefile
===================================================================
RCS file: /home/pcvs/ports/devel/sdl12/Makefile,v
retrieving revision 1.104
diff -d -u -u -r1.104 Makefile
--- Makefile	21 Nov 2006 16:47:00 -0000	1.104
+++ Makefile	3 Jan 2007 07:54:10 -0000
@@ -7,6 +7,7 @@
 
 PORTNAME=	sdl
 PORTVERSION=	1.2.11
+PORTREVISION=	1
 PORTEPOCH=	2
 CATEGORIES=	devel
 MASTER_SITES=	http://www.libsdl.org/release/
@@ -49,6 +50,7 @@
 
 OPTIONS=	NAS	"Include support for the Network Audio System"	Off	\
 		AALIB	"Include support for AA-lib"			On	\
+		DIRECTFB	"Include support for DirectFB"		Off	\
 		GL	"Include support for OpenGL"			On	\
 		GGI	"Include support for GGI"			Off	\
 		SVGALIB	"Include support for SVGALIB"			On
@@ -119,6 +121,10 @@
 CONFIGURE_ARGS+=--disable-arts
 .endif
 
+.if (exists(${LOCALBASE}/directfb-config) || defined(WITH_DIRECTFB)) && !defined(WITHOUT_DIRECTFB)
+LIB_DEPENDS+=	directfb-0.9.16:${PORTSDIR}/devel/directfb
+.endif
+
 pre-configure:
 	@${FIND} ${WRKSRC} -type f -name Makefile.in | ${XARGS} ${TOUCH}
 
--- patch ends here ---

--- patch-src__video__directfb__SDL_DirectFB_video.c begins here ---
--- src/video/directfb/SDL_DirectFB_video.c.orig	Wed Jan  3 04:46:41 2007
+++ src/video/directfb/SDL_DirectFB_video.c	Wed Jan  3 05:29:06 2007
@@ -32,8 +32,8 @@
 #include <unistd.h>
 #include <sys/mman.h>
 
-#include <directfb.h>
-#include <directfb_version.h>
+#include <directfb/directfb.h>
+#include <directfb-internal/directfb_version.h>
 
 #include "SDL_video.h"
 #include "SDL_mouse.h"
@@ -181,9 +181,9 @@
   return DSPF_UNKNOWN;
 }
 
-static DFBEnumerationResult EnumModesCallback (int  width,
-                                               int  height,
-                                               int  bpp,
+static DFBEnumerationResult EnumModesCallback (unsigned int  width,
+                                               unsigned int  height,
+                                               unsigned int  bpp,
                                                void *data)
 {
   SDL_VideoDevice *this = (SDL_VideoDevice *)data;
--- patch-src__video__directfb__SDL_DirectFB_video.c ends here ---


>Release-Note:
>Audit-Trail:
>Unformatted: