Hello,

Thanks for the patch! Would you mind not making an upload of SDL right now? I will package a new revision of the current one with the build dependencies fixed ( i already fixed that bug here in 1.2.8 that I am working on... it was simply a 'configure' parameter ). I will try to release a -4 revision tonight with the fixes already included.

Would you be able to upload it for me?

Lawrence

----Original Message Follows----
From: Steve Langasek <[EMAIL PROTECTED]>
To: Lawrence Williams <[EMAIL PROTECTED]>
CC: [EMAIL PROTECTED]
Subject: Re: Bug#285074: Bug#278582: marked as done (SDL Parachute Deployed when using aRts version of libsdl)
Date: Mon, 10 Jan 2005 03:33:29 -0800


tags 285074 patch
severity 285074 important
thanks

On Fri, Jan 07, 2005 at 07:53:55AM -0330, Lawrence Williams wrote:
> I am working on packaging SDL 1.2.8. Among other things, I will disable
> dlopen() so bugs like this won't occur.

I don't know what your timetable for the new version is, but in the
meantime, we worked this bug a bit at the BSP this weekend, and the attached
patch is the result.  This may be nothing different than what you already
had in mind, but I figured it wouldn't hurt to get it into the BTS.

In looking around at the source, I noticed that the alsa driver was not the
only one that had references to dlopen(); the esd and arts drivers also have
support for dlopen, only in the case of these two drivers, they *only*
support dlopen -- they are not linked against the underlying libs, and as a
result the packages are also missing dependencies on the packages providing
them!  I'm almost positive this is not the intended behavior -- AIUI, much
of the purpose of having split SDL audio packages is to ensure there's at
least one working audio driver -- but I didn't make any changes here without
confirming with you first.

I'm not NMUing for this bug at this time, because upon reflection, I think
this bug should be downgraded; and because the entire issue arose because of
a misbuild in the *maintainer's* build environment, so an NMU doesn't
actually fix that.  The chances of a security upload tripping across this
bug in testing or stable is very slim, and the immediate breakage in the
archive has already been fixed by a recompile-only NMU, so I don't think
there's any reason left to consider this RC.

OTOH, if it's determined that libsdl1.2debian-arts and libsdl1.2debian-esd
are missing dependencies on the corresponding audio libs, that would be an
RC bug.

Thanks,
--
Steve Langasek
postmodern programmer
<< libsdl1.2-285074.diff >>
<< signature.asc >>




-- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Reply via email to