[Xorg] debrix

[Daniel Stone]

On Mon, Jun 28, 2004 at 03:41:45AM +0200, Jakub Piotr C??apa wrote:
> Daniel Stone wrote:
> >On Sun, Jun 27, 2004 at 10:35:06PM +0200, Jakub Piotr C?apa wrote:
> >>Maybe. Tha main problem is Xorg binary loading sth as a driver without 
> >>complaining about it not being a driver. :D It should be checked somehow 
> >>IMHO. A list of display drivers + compatibile hardware (better - PCI 
> >>ids) would also be cool. (or is it actually done in -configure? tt also 
> >>failed for me)
> >
> >Hm, interesting. I suppose we could error out if there's no relevant
> >DriverRec in the module we explicitly requested. I think
> >radeon/r128/whatever have an output DriverRec, tho.
> 
> IMHO we need something to distinguish module types. But don't ask me 
> what it should be... ;-) I'm not on this level of debrix hacking. :D

I'll put it on the TODO.

> >That list would be really, really long, BTW. And I don't think there's
> >any standard interface to get all the PCI IDs.
> 
> My /etc/pci.ids is only 256K and it contains lots more than graphic 
> cards (+ the names would not have to be duplicated and driver names are 
> shorter ;). Maybe it wouldn't be as long as you think?

Hm.

> >>Don't understand the difference (yet) but AFAIR fbdev is complaining 
> >>about missing fbdevhw.
> >
> >Right. What happens is that you can't have module A depending on a
> >*variable* in module B, only a function. So if module A needs a variable
> >from module B, that just won't work - it has to be hidden behind a
> >function. In this case, fbdev/nv need variables from fbdevhw and xaa, so
> >fbdevhw and xaa can't be built as modules - they need to be built in to
> >the binary.
> >
> >That's the executive summary.
> 
> Now I understand. What you have broken is modules like pcidata ;-)
> #v+
> (EE) Failed to load module "pcidata" (module does not exist, 0)
> 
> Fatal server error:
> Unable to load required base modules, Exiting...
> #v-
> Make them modules again or stop forcing to load them.

Ugh, that was a mistake. I need to fix the loader. But not now.

> >>So I have to change the way I'm used to use my e-mail program. (groups 
> >>I'm most active on set the Reply-To header) :D
> >
> >Ah, sorry. That was the way fd.o was set up, and the way I'm used to - I
> >don't really want to change it, personally. ;)
> 
> Not a big deal, really. :D
> 
> While building in a clear enviroment I've catched some other problems 
> (patch is available [1]) (ordered as fixes appear in the diff)
> 
> 1. compalloc.c - probably you forgot to commit it (it was commited into
>    xserver by mistake, then reverted); fixes building with
>    --enable-composite (haven't tried running because of the lack of
>    modules ;)

--enable-composite is really broken; waiting for a code drop from
anholt, who has it working.

> 2. #include <X11/DECkeysym.h> - seems redundand to me and we don't have
>    this in debrix anywhere (it is in the Xorg tree - should be
>    imported?)

I've already committed removing this, but thanks.

> 3. hw/xorg/include/X11/extensions/*.h header file weren't installed

Ah, fixed - ta.

> 5. we have X11/Xdmcp.h and pkgconfig --cflags xdcmp returns
>    /usr/X11R6/include, not /usr/X11R6/include/X11

I've already committed this one, too.

Thanks a lot!
:) d

-- 
Daniel Stone                                            <daniel at freedesktop.org>
freedesktop.org: powering your desktop                http://www.freedesktop.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg/attachments/20040628/f3b21793/attachment.pgp>