backends need some OO loving

Jussi Kukkonen jhkukkon at cc.hut.fi
Thu Aug 30 10:09:40 PDT 2007


Matt Hoosier wrote:
>> BTW: If you're autotools-literate please take a look at
>> backend/common/Makefile.am -- it works but I'd like to see if automake
>> can make this more automatic...
>> http://gitweb.freedesktop.org/?p=geoclue.git;a=blob;f=backend/common/Makefile.am
> 
> Do you ever want some backend to link one of those, but not the other?
> I guess this is why you separated the two sets of CFLAGS and LDFLAGS
> out.

Yes, geoclue_position_error is for all position backends and
geoclue_web_service is for all backends that use web services.


> I'd recommend using a convenience library, to avoiding "guessing" the
> name of the object files. It'd be something like:
> 
>   noinst_LIBRARIES = libgeoclue_web_service.a libgeoclue_position_error.a
> 
>   libgeoclue_web_service_a_CFLAGS += ...
>   libgeoclue_position_error_a_CFLAGS ++ ...
> 
> Then of course you have to change the sites that currently do something like:
> 
>   my_backend_LIBADD = .../common/libgeoclue_web_service.a
> 

Thanks! "noinst" was what I was looking for.

-jussi

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 307 bytes
Desc: OpenPGP digital signature
Url : http://lists.freedesktop.org/archives/geoclue/attachments/20070830/b8911185/attachment.pgp 


More information about the GeoClue mailing list