Re: CVS commit: pkgsrc/devel/SDL

View: New views
3 Messages — Rating Filter:   Alert me  

Parent Message unknown Re: CVS commit: pkgsrc/devel/SDL

by Joerg Sonnenberger :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

It seems like the new version doesn't link against libX11 by default
anymore, but uses dlopen instead. This breaks a number of packages
in pkgsrc. It certainly breaks the ABI.

How do we want to continue with this?

Joerg

On Fri, Oct 23, 2009 at 06:05:07AM +0000, Adam Ciarcinski wrote:

> Module Name: pkgsrc
> Committed By: adam
> Date: Fri Oct 23 06:05:07 UTC 2009
>
> Modified Files:
> pkgsrc/devel/SDL: Makefile buildlink3.mk distinfo
> Removed Files:
> pkgsrc/devel/SDL/patches: patch-ab patch-pulse
>
> Log Message:
> Changes 1.2.14:
> * Added cast macros for correct usage with C++:
> SDL_reinterpret_cast(type, expression)
> SDL_static_cast(type, expression)
> * Added SDL_VIDEO_FULLSCREEN_DISPLAY as a preferred synonym for
>   SDL_VIDEO_FULLSCREEN_HEAD on X11.
> * Added SDL_DISABLE_LOCK_KEYS environment variable to enable normal
>   up/down events for Caps-Lock and Num-Lock keys.
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.86 -r1.87 pkgsrc/devel/SDL/Makefile
> cvs rdiff -u -r1.30 -r1.31 pkgsrc/devel/SDL/buildlink3.mk
> cvs rdiff -u -r1.53 -r1.54 pkgsrc/devel/SDL/distinfo
> cvs rdiff -u -r1.8 -r0 pkgsrc/devel/SDL/patches/patch-ab
> cvs rdiff -u -r1.1 -r0 pkgsrc/devel/SDL/patches/patch-pulse
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.

Re: CVS commit: pkgsrc/devel/SDL

by Tobias Nygren-4 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Thu, 29 Oct 2009 19:04:45 +0100
Joerg Sonnenberger <joerg@...> wrote:

> It seems like the new version doesn't link against libX11 by default
> anymore, but uses dlopen instead. This breaks a number of packages
> in pkgsrc. It certainly breaks the ABI.
>
> How do we want to continue with this?

Let's just bite the bullet add -lX11 where needed and bump revisions.
Fortunately breakage seems mostly confined to games/ and leaf packages.

-Tobias

Re: CVS commit: pkgsrc/devel/SDL

by Adam CiarciƄski :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

>> It seems like the new version doesn't link against libX11 by default
>> anymore, but uses dlopen instead. This breaks a number of packages
>> in pkgsrc. It certainly breaks the ABI.
>>
>> How do we want to continue with this?
>
> Let's just bite the bullet add -lX11 where needed and bump revisions.
> Fortunately breakage seems mostly confined to games/ and leaf  
> packages.

I have added --disable-x11-shared to CONFIGURE_ARGS.
Please, check if that fixes the problem with other packages.

Kind regards,
Adam