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
signature.asc
Description: This is a digitally signed message part.