[Gossip-dev] Gnome Screensaver Support

Richard Hult richard at imendio.com
Fri Mar 3 20:55:05 CET 2006


Brian Pepple wrote:

> Yeah, the configure.ac section needs some work (though w/o dbus it sorta
> workless building against gss).  How I have it right now, you can't
> explicitly disable gss. I figure I'll look at getting the extended away
> feature figured out, then look at something more sane in the
> configure.ac.

I forgot to comment on the rest of the patch :)

I think it would be good to do it like this:

gossip-idle.h with the function headers

and

gossip-idle-xss.c
gossip-idle-gss.c
gossip-idle-<whatever>.c for other platforms when/if we port to those.

with the various implementations, which are conditionally compiled in,
just the one that is selected. Then there is no need to have ifdefs in
the main code.

It might be that we need to expand the "API" a bit that is in
gossip-idle.h to be able to use the various implementations.

/Richard

-- 
Imendio AB, http://www.imendio.com/


More information about the Gossip-dev mailing list