libxf86config no longer usable

[Dan Nicholson]

On Tue, Apr 6, 2010 at 6:45 AM, Dan Nicholson <dbn.lists at gmail.com> wrote:
> On Mon, Apr 5, 2010 at 4:36 PM, Ruediger Oertel <ro at suse.de> wrote:
>> Hi,
>>
>> commit e1165632bdfbd720889ed1adf5f7ab338032c0ee
>> has added a call to xf86CheckBoolOption to Layout.c but this function (obviously)
>> does not end up in libxf86config since it's not in .../parser but in .../common/xf86Option.c
>>
>> so linking against libxf86config (if installed as possible via configure --enable-install-libxf86config)
>> will end up with: undefined symbol: xf86CheckBoolOption
>>
>> I just can't come up with a clean idea how to solve this ...
>
> I think the "cleanest" way would be to move a lot of xf86Option.c
> under parser/. It's really about parsing Options anyway, and half the
> implementations just wrap code from parser/Flags.c. So, moving
> xf86Opt.h and xf86Option.c under parser/ would seem to be the easiest.
>
> Another alternative is to move the action of automatically adding the
> InputDevices under the ServerLayout to xf86Config.c like the video
> config. It's a little cleaner to put it together immediately during
> the parsing, but you could assemble it later.

Another option is just to do a local hack in parser/Layout.c to
emulate xf86CheckBoolOption. Many of the necessary components are
already under parser/.

--
Dan