xf86UDelay was used - removal breaks drivers
Luc Verhaegen
libv at skynet.be
Fri Nov 6 17:58:45 PST 2009
On Fri, Nov 06, 2009 at 05:50:35PM -0800, Alan Coopersmith wrote:
> Luc Verhaegen wrote:
> > On Fri, Nov 06, 2009 at 03:31:27PM -0800, Alan Coopersmith wrote:
> >> This commit:
> >>
> >> commit 9522ab9a98e98bc394703038e34b1c12aa39a179
> >> Author: Matt Turner <mattst88 at gmail.com>
> >> Date: Mon Aug 31 16:12:29 2009 -0400
> >>
> >> Remove unused Delay.c
> >>
> >> has broken a number of drivers that used the previously exported
> >> xf86UDelay function, including:
> >> xf86-video-ast
> >> xf86-video-chips
> >> xf86-video-glint
> >> xf86-video-neomagic
> >> xf86-video-rendition
> >>
> >> Are updates going to be issued to all these drivers to use a different
> >> function, or should the removal be reverted?
> >
> > Apart from the fact that i find changes like this horribly shortsighted
> > and stupid, i wonder why tinderbox did not catch this.
>
> We caught it while working on the updates to our packages, since we force
> resolution of all symbols when building drivers, but I don't think the
> X.Org upstream builds do that. (For it to work for us on Solaris, we
> have to generate a linker mapfile listing the exported symbols from the
> Xorg binary that's used to satisfy the linker when checking.)
>
> It does appear some distros have had bugs reported on it though:
> http://bugs.gentoo.org/291825
> https://bugzilla.redhat.com/show_bug.cgi?id=523800
Sounds like something useful that tinderbox could do too then :)
Not that that in any way should allow for changes like this to happen
gratuitously.
Luc Verhaegen.
More information about the xorg-devel
mailing list