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