On Mon, Jan 10, 2005 at 05:20:03PM -0330, Lawrence Williams wrote: > Thanks for the patch! Would you mind not making an upload of SDL right now?
Yes, as noted in the last message, I have no intention to NMU at this time. > 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? If the changeset is sufficiently small to permit me to easily review it, I have no problems doing that. Cheers, -- Steve Langasek postmodern programmer > ----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 >> >
signature.asc
Description: Digital signature
