[Gossip-dev] Remove some functions from GossipProtocol that can't be implemented in TELEPATHY

[Mikael Hallendal]

26 dec 2006 kl. 12.28 skrev Martyn Russell:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Xavier Claessens wrote:

>> * _get_default_server is used to extract the server from an ID and  
>> take

> I have not looked in detail how this works, but I would assume that  
> all
> you need to do is store a list of GossipProtocols against some  
> telepathy
> thing which you use (a connection?) and that type of connection  
> (Jabber,
> MSN, etc) should be known in the backend surely? Otherwise how to do
> create the different types of GossipAccounts? If this is known, then I
> don't see why you can't do specific things in the telepathy backend
> based on that information per protocol?
>
> It, too me, just sounds like a little bit of work, maybe I am wrong?

I think one misconception is that this has to come from Telepathy.  
However, some of this can be done in the Gossip backend for  
Telepathy. So even if Telepathy itself doesn't support it it can  
still be supported by the backend.

The only feature I can come up with here is link local, as far as I  
understand all others would have a default server and a default port.

>> I really want to fix those 2 points, for others it's not very  
>> important.
>>
>> For get_example_* maybe it make sense to have something in the  
>> telepathy
>> spec to get an example for each parameter.
>
> Yes, think we have discussed this before and as I recall the response
> was divided. I would like to see the backend tell me the UI what  
> example
> to use and make the UI as unaware of protocols as possible, but I  
> think
> Micke and Robert thought it should be done by the UI.

However, you did bring up a good point here Martyn in that it's used  
in several places (for example to add an account but also to add a  
contact).

We could also look at putting this in libgossip but not as part of  
the protocol backend.

Best Regards,
   Mikael Hallendal

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