On söndag 18 maj 2008, Ron wrote:
[...]
> There was discussion and full rationale for this the last time we did it.
> It should be in the linuxwacom-discuss archives if this isn't sufficient.

Thanks, I'll search for it.

> I don't know what else you were trying to change here, and the commit
> message just says "rework" it,

Mmm... I was making it more linear and readable (that was the intention).. 
Also added some options and parts that are available in 'modern' systems.

> but I don't see anything that suggests a 
> need to change the option semantics from the code above, do you?

No, the semantics was not something I wanted to change.

> Oh wait, I see, you've moved WCM_OPTION_DLLOADER=yes to an unconditonal
> test outside all of this...  Ok, gotta sleep, but please let's start with
> the above as a known working reference then try to figure out what it was
> that you really wanted to change from that ...

That's exactly what I did for this patch :)

The default is to use the dlloader, --with-xmoduledir only overrides the path 
where the .so-file is put. Prior to this patch there was code that could 
automatically disable the dlloader, the patch remove this code so if you 
don't want an .so-file you will have to explicitly state that with 
the --disable-dlloader option.

Before my rework the configure script could get a false negative on the 
dlloader option if the --with-xmoduledir was not defined and one of the 
default paths existed but wacom_drv.so was not previously installed.

> Thanks for the quick response though ...
> Ron

Cheers
  Magnus

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to