Date: Tue, 11 Apr 2006 22:20:12 GMT From: Joerg Pulz <Joerg.Pulz@frm2.tum.de> To: freebsd-ports-bugs@FreeBSD.org Subject: Re: ports/95422: devel/pwlib: unbreak sdl detection, repair bsdvideo plugin, ... Message-ID: <200604112220.k3BMKCPg078728@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
The following reply was made to PR ports/95422; it has been noted by GNATS. From: Joerg Pulz <Joerg.Pulz@frm2.tum.de> To: bug-followup@FreeBSD.org, Joerg.Pulz@frm2.tum.de, Steve Ames <steve@energistic.com>, Boris Samorodov <bsam@ipt.ru> Cc: Subject: Re: ports/95422: devel/pwlib: unbreak sdl detection, repair bsdvideo plugin, ... Date: Wed, 12 Apr 2006 00:12:24 +0200 (CEST) -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hi, regarding to the latest follow-ups for PR/95424 which where kindly provided by Boris Samorodov, here is a more refined version for files/patch-configureac in devel/pwlib. The problem is actually, that the configure script adds a line "#define P_SDL 0" if one uses the WITHOUT_SDL knob or SDL is not available at all. This is completely wrong as this actually defines P_SDL with value "0" instead of undefining it. I double checked the configure.ac file and found no other point beside the P_SDL where this is done in this way. I think the upstream maintainers made an error there. Unfortunately, this error prevents the patched version (PR/95424) of net/ohpone and maybe other ports too from building correctly on systems where SDL is not installed at allor disabled during the build of devel/pwlib by using the WITHOUT_SDL knob. The procedure for fixing this PR should now be: 1. apply the initial patch and add new files to CVS 2. apply the following patch to files/patch-configureac Thanks a lot Joerg - -- The beginning is the most important part of the work. -Plato - --- patch-configureac.diff begins here --- - --- patch-configureac.orig Wed Apr 12 00:07:09 2006 +++ patch-configureac Wed Apr 12 00:06:14 2006 @@ -1,5 +1,5 @@ --- configure.ac.orig Wed Feb 22 12:53:28 2006 - -+++ configure.ac Wed Apr 12 00:06:55 2006 ++++ configure.ac Tue Apr 11 23:47:45 2006 @@ -136,7 +136,7 @@ x86 | i686 | i586 | i486 | i386 ) MACHTYPE=x86 ;; @@ -107,6 +107,15 @@ #include <netdb.h>], [getnameinfo(NULL, 0, NULL, 0, NULL, 0, 0);], HAS_IPV6=yes, HAS_IPV6=no) if test $HAS_IPV6 = no ; then +@@ -1485,7 +1486,7 @@ + HAS_SDL= + if test "$enable_sdl" = "no" ; then + AC_MSG_NOTICE(SDL disabled) +- AC_DEFINE(P_SDL, 0) ++# AC_DEFINE(P_SDL, 0) + else + dnl SDL checks + if test "${SDLLIB:-unset}" = "unset" ; then @@ -1501,9 +1502,9 @@ AC_CHECK_LIB(SDL, SDL_CreateYUVOverlay, HAS_SDL=1, HAS_SDL=, -L${X11LIBDIR} -lpthread -lX11 -lXext) fi - --- patch-configureac.diff ends here --- -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.3 (FreeBSD) iD8DBQFEPCnLSPOsGF+KA+MRAstMAJ9VVHVizRkrQaq16QBGUi+/Rm8sSwCffBez RwCbQZfq9qwok2Z061KBamQ= =i0LY -----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200604112220.k3BMKCPg078728>